Re: [PATCH] dt-bindings: Drop redundant minItems/maxItems

Philipp Zabel
2021-06-16T11:05:09Z

On Tue, 2021-06-15 at 13:15 -0600, Rob Herring wrote:
> If a property has an 'items' list, then a 'minItems' or 'maxItems' with the
> same size as the list is redundant and can be dropped. Note that is DT
> schema specific behavior and not standard json-schema behavior. The tooling
> will fixup the final schema adding any unspecified minItems/maxItems.
>
> This condition is partially checked with the meta-schema already, but
> only if both 'minItems' and 'maxItems' are equal to the 'items' length.
> An improved meta-schema is pending.
[...]
> Documentation/devicetree/bindings/reset/fsl,imx-src.yaml | 1 -
[...]
> diff --git a/Documentation/devicetree/bindings/reset/fsl,imx-src.yaml b/Documentation/devicetree/bindings/reset/fsl,imx-src.yaml
> index 27c5e34a3ac6..b11ac533f914 100644
> --- a/Documentation/devicetree/bindings/reset/fsl,imx-src.yaml
> +++ b/Documentation/devicetree/bindings/reset/fsl,imx-src.yaml
> @@ -59,7 +59,6 @@ properties:
> - description: SRC interrupt
> - description: CPU WDOG interrupts out of SRC
> minItems: 1
> - maxItems: 2
>
> '#reset-cells':
> const: 1

Acked-by: Philipp Zabel <p.zabel@pengutronix.de>

regards
Philipp