Re: [PATCH 3/5] dt-bindings: clock: drop minItems equal to maxItems

From: Laurent Pinchart
Date: Thu Aug 25 2022 - 13:01:55 EST


Hi Krzysztof,

Thank you for the patch.

On Thu, Aug 25, 2022 at 02:33:32PM +0300, Krzysztof Kozlowski wrote:
> minItems, if missing, are implicitly equal to maxItems, so drop
> redundant piece to reduce size of code.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>

Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx>

> ---
> Documentation/devicetree/bindings/clock/cirrus,cs2000-cp.yaml | 1 -
> .../devicetree/bindings/clock/renesas,rcar-usb2-clock-sel.yaml | 2 --
> Documentation/devicetree/bindings/clock/samsung,s2mps11.yaml | 1 -
> 3 files changed, 4 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/clock/cirrus,cs2000-cp.yaml b/Documentation/devicetree/bindings/clock/cirrus,cs2000-cp.yaml
> index 0abd6ba82dfd..82836086cac1 100644
> --- a/Documentation/devicetree/bindings/clock/cirrus,cs2000-cp.yaml
> +++ b/Documentation/devicetree/bindings/clock/cirrus,cs2000-cp.yaml
> @@ -23,7 +23,6 @@ properties:
> clocks:
> description:
> Common clock binding for CLK_IN, XTI/REF_CLK
> - minItems: 2
> maxItems: 2
>
> clock-names:
> diff --git a/Documentation/devicetree/bindings/clock/renesas,rcar-usb2-clock-sel.yaml b/Documentation/devicetree/bindings/clock/renesas,rcar-usb2-clock-sel.yaml
> index 6eaabb4d82ec..81f09df7147e 100644
> --- a/Documentation/devicetree/bindings/clock/renesas,rcar-usb2-clock-sel.yaml
> +++ b/Documentation/devicetree/bindings/clock/renesas,rcar-usb2-clock-sel.yaml
> @@ -47,7 +47,6 @@ properties:
> maxItems: 1
>
> clocks:
> - minItems: 4
> maxItems: 4
>
> clock-names:
> @@ -64,7 +63,6 @@ properties:
> maxItems: 1
>
> resets:
> - minItems: 2
> maxItems: 2
>
> reset-names:
> diff --git a/Documentation/devicetree/bindings/clock/samsung,s2mps11.yaml b/Documentation/devicetree/bindings/clock/samsung,s2mps11.yaml
> index 9248bfc16d48..d5296e6053a1 100644
> --- a/Documentation/devicetree/bindings/clock/samsung,s2mps11.yaml
> +++ b/Documentation/devicetree/bindings/clock/samsung,s2mps11.yaml
> @@ -34,7 +34,6 @@ properties:
> const: 1
>
> clock-output-names:
> - minItems: 3
> maxItems: 3
> description: Names for AP, CP and BT clocks.
>

--
Regards,

Laurent Pinchart