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

From: Alim Akhtar
Date: Thu Aug 25 2022 - 12:59:10 EST




>-----Original Message-----
>From: Krzysztof Kozlowski [mailto:krzysztof.kozlowski@xxxxxxxxxx]
>Sent: Thursday, August 25, 2022 5:04 PM
>To: Rob Herring <robh+dt@xxxxxxxxxx>; Krzysztof Kozlowski
><krzysztof.kozlowski+dt@xxxxxxxxxx>; Kunihiko Hayashi
><hayashi.kunihiko@xxxxxxxxxxxxx>; Masami Hiramatsu
><mhiramat@xxxxxxxxxx>; Damien Le Moal
><damien.lemoal@xxxxxxxxxxxxxxxxxx>; Michael Turquette
><mturquette@xxxxxxxxxxxx>; Stephen Boyd <sboyd@xxxxxxxxxx>; Geert
>Uytterhoeven <geert+renesas@xxxxxxxxx>; Sylwester Nawrocki
><s.nawrocki@xxxxxxxxxxx>; Tomasz Figa <tomasz.figa@xxxxxxxxx>;
>Chanwoo Choi <cw00.choi@xxxxxxxxxxx>; Alim Akhtar
><alim.akhtar@xxxxxxxxxxx>; Vladimir Zapolskiy <vz@xxxxxxxxx>; Herbert
>Xu <herbert@xxxxxxxxxxxxxxxxxxx>; David S. Miller
><davem@xxxxxxxxxxxxx>; Andrzej Hajda <andrzej.hajda@xxxxxxxxx>; Neil
>Armstrong <neil.armstrong@xxxxxxxxxx>; Robert Foss
><robert.foss@xxxxxxxxxx>; Laurent Pinchart
><Laurent.pinchart@xxxxxxxxxxxxxxxx>; Jonas Karlman <jonas@xxxxxxxxx>;
>Jernej Skrabec <jernej.skrabec@xxxxxxxxx>; David Airlie <airlied@xxxxxxxx>;
>Daniel Vetter <daniel@xxxxxxxx>; Rob Clark <robdclark@xxxxxxxxx>; Abhinav
>Kumar <quic_abhinavk@xxxxxxxxxxx>; Dmitry Baryshkov
><dmitry.baryshkov@xxxxxxxxxx>; Sean Paul <sean@xxxxxxxxxx>; Inki Dae
><inki.dae@xxxxxxxxxxx>; Seung-Woo Kim <sw0312.kim@xxxxxxxxxxx>;
>Kyungmin Park <kyungmin.park@xxxxxxxxxxx>; Thierry Reding
><thierry.reding@xxxxxxxxx>; Jonathan Hunter <jonathanh@xxxxxxxxxx>;
>Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx>; Florian Fainelli
><f.fainelli@xxxxxxxxx>; Linus Walleij <linus.walleij@xxxxxxxxxx>; Andre
>Przywara <andre.przywara@xxxxxxx>; Kuninori Morimoto
><kuninori.morimoto.gx@xxxxxxxxxxx>; Yoshihiro Shimoda
><yoshihiro.shimoda.uh@xxxxxxxxxxx>; Marek Vasut <marex@xxxxxxx>;
>Krishna Manikandan <quic_mkrishn@xxxxxxxxxxx>;
>devicetree@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux-
>kernel@xxxxxxxxxxxxxxx; linux-ide@xxxxxxxxxxxxxxx; linux-
>clk@xxxxxxxxxxxxxxx; linux-renesas-soc@xxxxxxxxxxxxxxx; linux-samsung-
>soc@xxxxxxxxxxxxxxx; linux-crypto@xxxxxxxxxxxxxxx; dri-
>devel@xxxxxxxxxxxxxxxxxxxxx; linux-arm-msm@xxxxxxxxxxxxxxx;
>freedreno@xxxxxxxxxxxxxxxxxxxxx; linux-tegra@xxxxxxxxxxxxxxx
>Cc: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
>Subject: [PATCH 3/5] dt-bindings: clock: drop minItems equal to maxItems
>
>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: Alim Akhtar <alim.akhtar@xxxxxxxxxxx>

> 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-se
>+++ l.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.
>
>--
>2.34.1