* [PATCH 2/5] dt-bindings: ata: drop minItems equal to maxItems
2022-08-25 11:33 [PATCH 1/5] dt-bindings: socionext,uniphier-system-cache: drop minItems equal to maxItems Krzysztof Kozlowski
@ 2022-08-25 11:33 ` Krzysztof Kozlowski
2022-08-25 17:00 ` Laurent Pinchart
` (2 more replies)
2022-08-25 11:33 ` [PATCH 3/5] dt-bindings: clock: " Krzysztof Kozlowski
` (4 subsequent siblings)
5 siblings, 3 replies; 20+ messages in thread
From: Krzysztof Kozlowski @ 2022-08-25 11:33 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Kunihiko Hayashi,
Masami Hiramatsu, Damien Le Moal, Michael Turquette,
Stephen Boyd, Geert Uytterhoeven, Sylwester Nawrocki,
Tomasz Figa, Chanwoo Choi, Alim Akhtar, Vladimir Zapolskiy,
Herbert Xu, David S. Miller, Andrzej Hajda, Neil Armstrong,
Robert Foss, Laurent Pinchart, Jonas Karlman, Jernej Skrabec,
David Airlie, Daniel Vetter, Rob Clark, Abhinav Kumar,
Dmitry Baryshkov, Sean Paul, Inki Dae, Seung-Woo Kim,
Kyungmin Park, Thierry Reding, Jonathan Hunter, Masahiro Yamada,
Florian Fainelli, Linus Walleij, Andre Przywara,
Kuninori Morimoto, Yoshihiro Shimoda, Marek Vasut,
Krishna Manikandan, devicetree, linux-arm-kernel, linux-kernel,
linux-ide, linux-clk, linux-renesas-soc, linux-samsung-soc,
linux-crypto, dri-devel, linux-arm-msm, freedreno, linux-tegra
Cc: Krzysztof Kozlowski
minItems, if missing, are implicitly equal to maxItems, so drop
redundant piece to reduce size of code.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
Documentation/devicetree/bindings/ata/brcm,sata-brcm.yaml | 1 -
.../devicetree/bindings/ata/cortina,gemini-sata-bridge.yaml | 2 --
Documentation/devicetree/bindings/ata/sata_highbank.yaml | 1 -
3 files changed, 4 deletions(-)
diff --git a/Documentation/devicetree/bindings/ata/brcm,sata-brcm.yaml b/Documentation/devicetree/bindings/ata/brcm,sata-brcm.yaml
index 235a93ac86b0..3766cc80cb17 100644
--- a/Documentation/devicetree/bindings/ata/brcm,sata-brcm.yaml
+++ b/Documentation/devicetree/bindings/ata/brcm,sata-brcm.yaml
@@ -30,7 +30,6 @@ properties:
- const: brcm,bcm-nsp-ahci
reg:
- minItems: 2
maxItems: 2
reg-names:
diff --git a/Documentation/devicetree/bindings/ata/cortina,gemini-sata-bridge.yaml b/Documentation/devicetree/bindings/ata/cortina,gemini-sata-bridge.yaml
index 21a90975593b..529093666508 100644
--- a/Documentation/devicetree/bindings/ata/cortina,gemini-sata-bridge.yaml
+++ b/Documentation/devicetree/bindings/ata/cortina,gemini-sata-bridge.yaml
@@ -22,7 +22,6 @@ properties:
maxItems: 1
resets:
- minItems: 2
maxItems: 2
description: phandles to the reset lines for both SATA bridges
@@ -32,7 +31,6 @@ properties:
- const: sata1
clocks:
- minItems: 2
maxItems: 2
description: phandles to the compulsory peripheral clocks
diff --git a/Documentation/devicetree/bindings/ata/sata_highbank.yaml b/Documentation/devicetree/bindings/ata/sata_highbank.yaml
index 49679b58041c..f23f26a8f21c 100644
--- a/Documentation/devicetree/bindings/ata/sata_highbank.yaml
+++ b/Documentation/devicetree/bindings/ata/sata_highbank.yaml
@@ -52,7 +52,6 @@ properties:
minItems: 1
maxItems: 8
items:
- minItems: 2
maxItems: 2
calxeda,tx-atten:
--
2.34.1
^ permalink raw reply related [flat|nested] 20+ messages in thread* Re: [PATCH 2/5] dt-bindings: ata: drop minItems equal to maxItems
2022-08-25 11:33 ` [PATCH 2/5] dt-bindings: ata: " Krzysztof Kozlowski
@ 2022-08-25 17:00 ` Laurent Pinchart
2022-08-25 22:31 ` Damien Le Moal
2022-08-29 19:56 ` Rob Herring
2 siblings, 0 replies; 20+ messages in thread
From: Laurent Pinchart @ 2022-08-25 17:00 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Rob Herring, Krzysztof Kozlowski, Kunihiko Hayashi,
Masami Hiramatsu, Damien Le Moal, Michael Turquette,
Stephen Boyd, Geert Uytterhoeven, Sylwester Nawrocki,
Tomasz Figa, Chanwoo Choi, Alim Akhtar, Vladimir Zapolskiy,
Herbert Xu, David S. Miller, Andrzej Hajda, Neil Armstrong,
Robert Foss, Jonas Karlman, Jernej Skrabec, David Airlie,
Daniel Vetter, Rob Clark, Abhinav Kumar, Dmitry Baryshkov,
Sean Paul, Inki Dae, Seung-Woo Kim, Kyungmin Park,
Thierry Reding, Jonathan Hunter, Masahiro Yamada,
Florian Fainelli, Linus Walleij, Andre Przywara,
Kuninori Morimoto, Yoshihiro Shimoda, Marek Vasut,
Krishna Manikandan, devicetree, linux-arm-kernel, linux-kernel,
linux-ide, linux-clk, linux-renesas-soc, linux-samsung-soc,
linux-crypto, dri-devel, linux-arm-msm, freedreno, linux-tegra
Hi Krzysztof,
Thank you for the patch.
On Thu, Aug 25, 2022 at 02:33:31PM +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@linaro.org>
Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> ---
> Documentation/devicetree/bindings/ata/brcm,sata-brcm.yaml | 1 -
> .../devicetree/bindings/ata/cortina,gemini-sata-bridge.yaml | 2 --
> Documentation/devicetree/bindings/ata/sata_highbank.yaml | 1 -
> 3 files changed, 4 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/ata/brcm,sata-brcm.yaml b/Documentation/devicetree/bindings/ata/brcm,sata-brcm.yaml
> index 235a93ac86b0..3766cc80cb17 100644
> --- a/Documentation/devicetree/bindings/ata/brcm,sata-brcm.yaml
> +++ b/Documentation/devicetree/bindings/ata/brcm,sata-brcm.yaml
> @@ -30,7 +30,6 @@ properties:
> - const: brcm,bcm-nsp-ahci
>
> reg:
> - minItems: 2
> maxItems: 2
>
> reg-names:
> diff --git a/Documentation/devicetree/bindings/ata/cortina,gemini-sata-bridge.yaml b/Documentation/devicetree/bindings/ata/cortina,gemini-sata-bridge.yaml
> index 21a90975593b..529093666508 100644
> --- a/Documentation/devicetree/bindings/ata/cortina,gemini-sata-bridge.yaml
> +++ b/Documentation/devicetree/bindings/ata/cortina,gemini-sata-bridge.yaml
> @@ -22,7 +22,6 @@ properties:
> maxItems: 1
>
> resets:
> - minItems: 2
> maxItems: 2
> description: phandles to the reset lines for both SATA bridges
>
> @@ -32,7 +31,6 @@ properties:
> - const: sata1
>
> clocks:
> - minItems: 2
> maxItems: 2
> description: phandles to the compulsory peripheral clocks
>
> diff --git a/Documentation/devicetree/bindings/ata/sata_highbank.yaml b/Documentation/devicetree/bindings/ata/sata_highbank.yaml
> index 49679b58041c..f23f26a8f21c 100644
> --- a/Documentation/devicetree/bindings/ata/sata_highbank.yaml
> +++ b/Documentation/devicetree/bindings/ata/sata_highbank.yaml
> @@ -52,7 +52,6 @@ properties:
> minItems: 1
> maxItems: 8
> items:
> - minItems: 2
> maxItems: 2
>
> calxeda,tx-atten:
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 2/5] dt-bindings: ata: drop minItems equal to maxItems
2022-08-25 11:33 ` [PATCH 2/5] dt-bindings: ata: " Krzysztof Kozlowski
2022-08-25 17:00 ` Laurent Pinchart
@ 2022-08-25 22:31 ` Damien Le Moal
2022-08-29 19:56 ` Rob Herring
2 siblings, 0 replies; 20+ messages in thread
From: Damien Le Moal @ 2022-08-25 22:31 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Krzysztof Kozlowski,
Kunihiko Hayashi, Masami Hiramatsu, Michael Turquette,
Stephen Boyd, Geert Uytterhoeven, Sylwester Nawrocki,
Tomasz Figa, Chanwoo Choi, Alim Akhtar, Vladimir Zapolskiy,
Herbert Xu, David S. Miller, Andrzej Hajda, Neil Armstrong,
Robert Foss, Laurent Pinchart, Jonas Karlman, Jernej Skrabec,
David Airlie, Daniel Vetter, Rob Clark, Abhinav Kumar,
Dmitry Baryshkov, Sean Paul, Inki Dae, Seung-Woo Kim,
Kyungmin Park, Thierry Reding, Jonathan Hunter, Masahiro Yamada,
Florian Fainelli, Linus Walleij, Andre Przywara,
Kuninori Morimoto, Yoshihiro Shimoda, Marek Vasut,
Krishna Manikandan, devicetree, linux-arm-kernel, linux-kernel,
linux-ide, linux-clk, linux-renesas-soc, linux-samsung-soc,
linux-crypto, dri-devel, linux-arm-msm, freedreno, linux-tegra
On 8/25/22 20:33, 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@linaro.org>
Acked-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
> ---
> Documentation/devicetree/bindings/ata/brcm,sata-brcm.yaml | 1 -
> .../devicetree/bindings/ata/cortina,gemini-sata-bridge.yaml | 2 --
> Documentation/devicetree/bindings/ata/sata_highbank.yaml | 1 -
> 3 files changed, 4 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/ata/brcm,sata-brcm.yaml b/Documentation/devicetree/bindings/ata/brcm,sata-brcm.yaml
> index 235a93ac86b0..3766cc80cb17 100644
> --- a/Documentation/devicetree/bindings/ata/brcm,sata-brcm.yaml
> +++ b/Documentation/devicetree/bindings/ata/brcm,sata-brcm.yaml
> @@ -30,7 +30,6 @@ properties:
> - const: brcm,bcm-nsp-ahci
>
> reg:
> - minItems: 2
> maxItems: 2
>
> reg-names:
> diff --git a/Documentation/devicetree/bindings/ata/cortina,gemini-sata-bridge.yaml b/Documentation/devicetree/bindings/ata/cortina,gemini-sata-bridge.yaml
> index 21a90975593b..529093666508 100644
> --- a/Documentation/devicetree/bindings/ata/cortina,gemini-sata-bridge.yaml
> +++ b/Documentation/devicetree/bindings/ata/cortina,gemini-sata-bridge.yaml
> @@ -22,7 +22,6 @@ properties:
> maxItems: 1
>
> resets:
> - minItems: 2
> maxItems: 2
> description: phandles to the reset lines for both SATA bridges
>
> @@ -32,7 +31,6 @@ properties:
> - const: sata1
>
> clocks:
> - minItems: 2
> maxItems: 2
> description: phandles to the compulsory peripheral clocks
>
> diff --git a/Documentation/devicetree/bindings/ata/sata_highbank.yaml b/Documentation/devicetree/bindings/ata/sata_highbank.yaml
> index 49679b58041c..f23f26a8f21c 100644
> --- a/Documentation/devicetree/bindings/ata/sata_highbank.yaml
> +++ b/Documentation/devicetree/bindings/ata/sata_highbank.yaml
> @@ -52,7 +52,6 @@ properties:
> minItems: 1
> maxItems: 8
> items:
> - minItems: 2
> maxItems: 2
>
> calxeda,tx-atten:
--
Damien Le Moal
Western Digital Research
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 2/5] dt-bindings: ata: drop minItems equal to maxItems
2022-08-25 11:33 ` [PATCH 2/5] dt-bindings: ata: " Krzysztof Kozlowski
2022-08-25 17:00 ` Laurent Pinchart
2022-08-25 22:31 ` Damien Le Moal
@ 2022-08-29 19:56 ` Rob Herring
2 siblings, 0 replies; 20+ messages in thread
From: Rob Herring @ 2022-08-29 19:56 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Kunihiko Hayashi, freedreno, Vladimir Zapolskiy, Chanwoo Choi,
linux-renesas-soc, Masami Hiramatsu, David Airlie,
Masahiro Yamada, Kuninori Morimoto, linux-crypto,
Geert Uytterhoeven, linux-arm-kernel, Jernej Skrabec,
Seung-Woo Kim, Laurent Pinchart, linux-samsung-soc, Rob Herring,
David S. Miller, linux-clk, Abhinav Kumar, Krishna Manikandan,
Inki Dae, linux-kernel, Kyungmin Park, Krzysztof Kozlowski,
Sylwester Nawrocki, Robert Foss, Rob Clark, Sean Paul,
linux-tegra, Linus Walleij, Alim Akhtar, Daniel Vetter,
Marek Vasut, Tomasz Figa, Florian Fainelli, Yoshihiro Shimoda,
linux-arm-msm, Herbert Xu, Neil Armstrong, dri-devel,
Jonathan Hunter, Stephen Boyd, Jonas Karlman, Dmitry Baryshkov,
linux-ide, Thierry Reding, Andre Przywara, Michael Turquette,
Andrzej Hajda, devicetree, Damien Le Moal
On Thu, 25 Aug 2022 14:33:31 +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@linaro.org>
> ---
> Documentation/devicetree/bindings/ata/brcm,sata-brcm.yaml | 1 -
> .../devicetree/bindings/ata/cortina,gemini-sata-bridge.yaml | 2 --
> Documentation/devicetree/bindings/ata/sata_highbank.yaml | 1 -
> 3 files changed, 4 deletions(-)
>
Applied, thanks!
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH 3/5] dt-bindings: clock: drop minItems equal to maxItems
2022-08-25 11:33 [PATCH 1/5] dt-bindings: socionext,uniphier-system-cache: drop minItems equal to maxItems Krzysztof Kozlowski
2022-08-25 11:33 ` [PATCH 2/5] dt-bindings: ata: " Krzysztof Kozlowski
@ 2022-08-25 11:33 ` Krzysztof Kozlowski
2022-08-25 16:58 ` Alim Akhtar
` (2 more replies)
2022-08-25 11:33 ` [PATCH 4/5] dt-bindings: crypto: " Krzysztof Kozlowski
` (3 subsequent siblings)
5 siblings, 3 replies; 20+ messages in thread
From: Krzysztof Kozlowski @ 2022-08-25 11:33 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Kunihiko Hayashi,
Masami Hiramatsu, Damien Le Moal, Michael Turquette,
Stephen Boyd, Geert Uytterhoeven, Sylwester Nawrocki,
Tomasz Figa, Chanwoo Choi, Alim Akhtar, Vladimir Zapolskiy,
Herbert Xu, David S. Miller, Andrzej Hajda, Neil Armstrong,
Robert Foss, Laurent Pinchart, Jonas Karlman, Jernej Skrabec,
David Airlie, Daniel Vetter, Rob Clark, Abhinav Kumar,
Dmitry Baryshkov, Sean Paul, Inki Dae, Seung-Woo Kim,
Kyungmin Park, Thierry Reding, Jonathan Hunter, Masahiro Yamada,
Florian Fainelli, Linus Walleij, Andre Przywara,
Kuninori Morimoto, Yoshihiro Shimoda, Marek Vasut,
Krishna Manikandan, devicetree, linux-arm-kernel, linux-kernel,
linux-ide, linux-clk, linux-renesas-soc, linux-samsung-soc,
linux-crypto, dri-devel, linux-arm-msm, freedreno, linux-tegra
Cc: Krzysztof Kozlowski
minItems, if missing, are implicitly equal to maxItems, so drop
redundant piece to reduce size of code.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
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.
--
2.34.1
^ permalink raw reply related [flat|nested] 20+ messages in thread* RE: [PATCH 3/5] dt-bindings: clock: drop minItems equal to maxItems
2022-08-25 11:33 ` [PATCH 3/5] dt-bindings: clock: " Krzysztof Kozlowski
@ 2022-08-25 16:58 ` Alim Akhtar
2022-08-25 17:01 ` Laurent Pinchart
2022-08-29 20:01 ` Rob Herring
2 siblings, 0 replies; 20+ messages in thread
From: Alim Akhtar @ 2022-08-25 16:58 UTC (permalink / raw)
To: 'Krzysztof Kozlowski', 'Rob Herring',
'Krzysztof Kozlowski', 'Kunihiko Hayashi',
'Masami Hiramatsu', 'Damien Le Moal',
'Michael Turquette', 'Stephen Boyd',
'Geert Uytterhoeven', 'Sylwester Nawrocki',
'Tomasz Figa', 'Chanwoo Choi',
'Vladimir Zapolskiy', 'Herbert Xu',
'David S. Miller', 'Andrzej Hajda',
'Neil Armstrong', 'Robert Foss',
'Laurent Pinchart', 'Jonas Karlman',
'Jernej Skrabec', 'David Airlie',
'Daniel Vetter', 'Rob Clark',
'Abhinav Kumar', 'Dmitry Baryshkov',
'Sean Paul', 'Inki Dae', 'Seung-Woo Kim',
'Kyungmin Park', 'Thierry Reding',
'Jonathan Hunter', 'Masahiro Yamada',
'Florian Fainelli', 'Linus Walleij',
'Andre Przywara', 'Kuninori Morimoto',
'Yoshihiro Shimoda', 'Marek Vasut',
'Krishna Manikandan',
devicetree, linux-arm-kernel, linux-kernel, linux-ide, linux-clk,
linux-renesas-soc, linux-samsung-soc, linux-crypto, dri-devel,
linux-arm-msm, freedreno, linux-tegra
>-----Original Message-----
>From: Krzysztof Kozlowski [mailto:krzysztof.kozlowski@linaro.org]
>Sent: Thursday, August 25, 2022 5:04 PM
>To: Rob Herring <robh+dt@kernel.org>; Krzysztof Kozlowski
><krzysztof.kozlowski+dt@linaro.org>; Kunihiko Hayashi
><hayashi.kunihiko@socionext.com>; Masami Hiramatsu
><mhiramat@kernel.org>; Damien Le Moal
><damien.lemoal@opensource.wdc.com>; Michael Turquette
><mturquette@baylibre.com>; Stephen Boyd <sboyd@kernel.org>; Geert
>Uytterhoeven <geert+renesas@glider.be>; Sylwester Nawrocki
><s.nawrocki@samsung.com>; Tomasz Figa <tomasz.figa@gmail.com>;
>Chanwoo Choi <cw00.choi@samsung.com>; Alim Akhtar
><alim.akhtar@samsung.com>; Vladimir Zapolskiy <vz@mleia.com>; Herbert
>Xu <herbert@gondor.apana.org.au>; David S. Miller
><davem@davemloft.net>; Andrzej Hajda <andrzej.hajda@intel.com>; Neil
>Armstrong <neil.armstrong@linaro.org>; Robert Foss
><robert.foss@linaro.org>; Laurent Pinchart
><Laurent.pinchart@ideasonboard.com>; Jonas Karlman <jonas@kwiboo.se>;
>Jernej Skrabec <jernej.skrabec@gmail.com>; David Airlie <airlied@linux.ie>;
>Daniel Vetter <daniel@ffwll.ch>; Rob Clark <robdclark@gmail.com>; Abhinav
>Kumar <quic_abhinavk@quicinc.com>; Dmitry Baryshkov
><dmitry.baryshkov@linaro.org>; Sean Paul <sean@poorly.run>; Inki Dae
><inki.dae@samsung.com>; Seung-Woo Kim <sw0312.kim@samsung.com>;
>Kyungmin Park <kyungmin.park@samsung.com>; Thierry Reding
><thierry.reding@gmail.com>; Jonathan Hunter <jonathanh@nvidia.com>;
>Masahiro Yamada <yamada.masahiro@socionext.com>; Florian Fainelli
><f.fainelli@gmail.com>; Linus Walleij <linus.walleij@linaro.org>; Andre
>Przywara <andre.przywara@arm.com>; Kuninori Morimoto
><kuninori.morimoto.gx@renesas.com>; Yoshihiro Shimoda
><yoshihiro.shimoda.uh@renesas.com>; Marek Vasut <marex@denx.de>;
>Krishna Manikandan <quic_mkrishn@quicinc.com>;
>devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-
>kernel@vger.kernel.org; linux-ide@vger.kernel.org; linux-
>clk@vger.kernel.org; linux-renesas-soc@vger.kernel.org; linux-samsung-
>soc@vger.kernel.org; linux-crypto@vger.kernel.org; dri-
>devel@lists.freedesktop.org; linux-arm-msm@vger.kernel.org;
>freedreno@lists.freedesktop.org; linux-tegra@vger.kernel.org
>Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>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@linaro.org>
>---
Reviewed-by: Alim Akhtar <alim.akhtar@samsung.com>
> 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
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 3/5] dt-bindings: clock: drop minItems equal to maxItems
2022-08-25 11:33 ` [PATCH 3/5] dt-bindings: clock: " Krzysztof Kozlowski
2022-08-25 16:58 ` Alim Akhtar
@ 2022-08-25 17:01 ` Laurent Pinchart
2022-08-29 20:01 ` Rob Herring
2 siblings, 0 replies; 20+ messages in thread
From: Laurent Pinchart @ 2022-08-25 17:01 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Rob Herring, Krzysztof Kozlowski, Kunihiko Hayashi,
Masami Hiramatsu, Damien Le Moal, Michael Turquette,
Stephen Boyd, Geert Uytterhoeven, Sylwester Nawrocki,
Tomasz Figa, Chanwoo Choi, Alim Akhtar, Vladimir Zapolskiy,
Herbert Xu, David S. Miller, Andrzej Hajda, Neil Armstrong,
Robert Foss, Jonas Karlman, Jernej Skrabec, David Airlie,
Daniel Vetter, Rob Clark, Abhinav Kumar, Dmitry Baryshkov,
Sean Paul, Inki Dae, Seung-Woo Kim, Kyungmin Park,
Thierry Reding, Jonathan Hunter, Masahiro Yamada,
Florian Fainelli, Linus Walleij, Andre Przywara,
Kuninori Morimoto, Yoshihiro Shimoda, Marek Vasut,
Krishna Manikandan, devicetree, linux-arm-kernel, linux-kernel,
linux-ide, linux-clk, linux-renesas-soc, linux-samsung-soc,
linux-crypto, dri-devel, linux-arm-msm, freedreno, linux-tegra
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@linaro.org>
Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> ---
> 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
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 3/5] dt-bindings: clock: drop minItems equal to maxItems
2022-08-25 11:33 ` [PATCH 3/5] dt-bindings: clock: " Krzysztof Kozlowski
2022-08-25 16:58 ` Alim Akhtar
2022-08-25 17:01 ` Laurent Pinchart
@ 2022-08-29 20:01 ` Rob Herring
2 siblings, 0 replies; 20+ messages in thread
From: Rob Herring @ 2022-08-29 20:01 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Jonathan Hunter, linux-ide, Vladimir Zapolskiy,
linux-samsung-soc, Rob Herring, Daniel Vetter, Laurent Pinchart,
Geert Uytterhoeven, Robert Foss, linux-clk, linux-crypto,
Damien Le Moal, Andre Przywara, Kuninori Morimoto, devicetree,
Masahiro Yamada, Stephen Boyd, Kyungmin Park, Thierry Reding,
Linus Walleij, Masami Hiramatsu, Seung-Woo Kim, freedreno,
Tomasz Figa, Florian Fainelli, linux-arm-msm, linux-arm-kernel,
Jonas Karlman, Krishna Manikandan, linux-kernel, Neil Armstrong,
Dmitry Baryshkov, Andrzej Hajda, Kunihiko Hayashi, David Airlie,
Rob Clark, Abhinav Kumar, linux-renesas-soc, Marek Vasut,
Chanwoo Choi, Krzysztof Kozlowski, Alim Akhtar, Sean Paul,
David S. Miller, Michael Turquette, dri-devel,
Sylwester Nawrocki, Herbert Xu, Yoshihiro Shimoda,
Jernej Skrabec, linux-tegra, Inki Dae
On Thu, 25 Aug 2022 14:33:32 +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@linaro.org>
> ---
> 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(-)
>
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH 4/5] dt-bindings: crypto: drop minItems equal to maxItems
2022-08-25 11:33 [PATCH 1/5] dt-bindings: socionext,uniphier-system-cache: drop minItems equal to maxItems Krzysztof Kozlowski
2022-08-25 11:33 ` [PATCH 2/5] dt-bindings: ata: " Krzysztof Kozlowski
2022-08-25 11:33 ` [PATCH 3/5] dt-bindings: clock: " Krzysztof Kozlowski
@ 2022-08-25 11:33 ` Krzysztof Kozlowski
2022-08-25 17:01 ` Laurent Pinchart
` (2 more replies)
2022-08-25 11:33 ` [PATCH 5/5] dt-bindings: display: " Krzysztof Kozlowski
` (2 subsequent siblings)
5 siblings, 3 replies; 20+ messages in thread
From: Krzysztof Kozlowski @ 2022-08-25 11:33 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Kunihiko Hayashi,
Masami Hiramatsu, Damien Le Moal, Michael Turquette,
Stephen Boyd, Geert Uytterhoeven, Sylwester Nawrocki,
Tomasz Figa, Chanwoo Choi, Alim Akhtar, Vladimir Zapolskiy,
Herbert Xu, David S. Miller, Andrzej Hajda, Neil Armstrong,
Robert Foss, Laurent Pinchart, Jonas Karlman, Jernej Skrabec,
David Airlie, Daniel Vetter, Rob Clark, Abhinav Kumar,
Dmitry Baryshkov, Sean Paul, Inki Dae, Seung-Woo Kim,
Kyungmin Park, Thierry Reding, Jonathan Hunter, Masahiro Yamada,
Florian Fainelli, Linus Walleij, Andre Przywara,
Kuninori Morimoto, Yoshihiro Shimoda, Marek Vasut,
Krishna Manikandan, devicetree, linux-arm-kernel, linux-kernel,
linux-ide, linux-clk, linux-renesas-soc, linux-samsung-soc,
linux-crypto, dri-devel, linux-arm-msm, freedreno, linux-tegra
Cc: Krzysztof Kozlowski
minItems, if missing, are implicitly equal to maxItems, so drop
redundant piece to reduce size of code.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
Documentation/devicetree/bindings/crypto/samsung-slimsss.yaml | 1 -
1 file changed, 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/crypto/samsung-slimsss.yaml b/Documentation/devicetree/bindings/crypto/samsung-slimsss.yaml
index 676950bb7b37..5b31891c97fe 100644
--- a/Documentation/devicetree/bindings/crypto/samsung-slimsss.yaml
+++ b/Documentation/devicetree/bindings/crypto/samsung-slimsss.yaml
@@ -24,7 +24,6 @@ properties:
maxItems: 1
clocks:
- minItems: 2
maxItems: 2
clock-names:
--
2.34.1
^ permalink raw reply related [flat|nested] 20+ messages in thread* Re: [PATCH 4/5] dt-bindings: crypto: drop minItems equal to maxItems
2022-08-25 11:33 ` [PATCH 4/5] dt-bindings: crypto: " Krzysztof Kozlowski
@ 2022-08-25 17:01 ` Laurent Pinchart
2022-08-25 17:02 ` Alim Akhtar
2022-08-29 19:56 ` Rob Herring
2 siblings, 0 replies; 20+ messages in thread
From: Laurent Pinchart @ 2022-08-25 17:01 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Rob Herring, Krzysztof Kozlowski, Kunihiko Hayashi,
Masami Hiramatsu, Damien Le Moal, Michael Turquette,
Stephen Boyd, Geert Uytterhoeven, Sylwester Nawrocki,
Tomasz Figa, Chanwoo Choi, Alim Akhtar, Vladimir Zapolskiy,
Herbert Xu, David S. Miller, Andrzej Hajda, Neil Armstrong,
Robert Foss, Jonas Karlman, Jernej Skrabec, David Airlie,
Daniel Vetter, Rob Clark, Abhinav Kumar, Dmitry Baryshkov,
Sean Paul, Inki Dae, Seung-Woo Kim, Kyungmin Park,
Thierry Reding, Jonathan Hunter, Masahiro Yamada,
Florian Fainelli, Linus Walleij, Andre Przywara,
Kuninori Morimoto, Yoshihiro Shimoda, Marek Vasut,
Krishna Manikandan, devicetree, linux-arm-kernel, linux-kernel,
linux-ide, linux-clk, linux-renesas-soc, linux-samsung-soc,
linux-crypto, dri-devel, linux-arm-msm, freedreno, linux-tegra
Hi Krzysztof,
Thank you for the patch.
On Thu, Aug 25, 2022 at 02:33:33PM +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@linaro.org>
Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> ---
> Documentation/devicetree/bindings/crypto/samsung-slimsss.yaml | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/crypto/samsung-slimsss.yaml b/Documentation/devicetree/bindings/crypto/samsung-slimsss.yaml
> index 676950bb7b37..5b31891c97fe 100644
> --- a/Documentation/devicetree/bindings/crypto/samsung-slimsss.yaml
> +++ b/Documentation/devicetree/bindings/crypto/samsung-slimsss.yaml
> @@ -24,7 +24,6 @@ properties:
> maxItems: 1
>
> clocks:
> - minItems: 2
> maxItems: 2
>
> clock-names:
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 20+ messages in thread
* RE: [PATCH 4/5] dt-bindings: crypto: drop minItems equal to maxItems
2022-08-25 11:33 ` [PATCH 4/5] dt-bindings: crypto: " Krzysztof Kozlowski
2022-08-25 17:01 ` Laurent Pinchart
@ 2022-08-25 17:02 ` Alim Akhtar
2022-08-29 19:56 ` Rob Herring
2 siblings, 0 replies; 20+ messages in thread
From: Alim Akhtar @ 2022-08-25 17:02 UTC (permalink / raw)
To: 'Krzysztof Kozlowski', 'Rob Herring',
'Krzysztof Kozlowski', 'Kunihiko Hayashi',
'Masami Hiramatsu', 'Damien Le Moal',
'Michael Turquette', 'Stephen Boyd',
'Geert Uytterhoeven', 'Sylwester Nawrocki',
'Tomasz Figa', 'Chanwoo Choi',
'Vladimir Zapolskiy', 'Herbert Xu',
'David S. Miller', 'Andrzej Hajda',
'Neil Armstrong', 'Robert Foss',
'Laurent Pinchart', 'Jonas Karlman',
'Jernej Skrabec', 'David Airlie',
'Daniel Vetter', 'Rob Clark',
'Abhinav Kumar', 'Dmitry Baryshkov',
'Sean Paul', 'Inki Dae', 'Seung-Woo Kim',
'Kyungmin Park', 'Thierry Reding',
'Jonathan Hunter', 'Masahiro Yamada',
'Florian Fainelli', 'Linus Walleij',
'Andre Przywara', 'Kuninori Morimoto',
'Yoshihiro Shimoda', 'Marek Vasut',
'Krishna Manikandan',
devicetree, linux-arm-kernel, linux-kernel, linux-ide, linux-clk,
linux-renesas-soc, linux-samsung-soc, linux-crypto, dri-devel,
linux-arm-msm, freedreno, linux-tegra
Hi Krzysztof
>-----Original Message-----
>From: Krzysztof Kozlowski [mailto:krzysztof.kozlowski@linaro.org]
>Sent: Thursday, August 25, 2022 5:04 PM
>To: Rob Herring <robh+dt@kernel.org>; Krzysztof Kozlowski
><krzysztof.kozlowski+dt@linaro.org>; Kunihiko Hayashi
><hayashi.kunihiko@socionext.com>; Masami Hiramatsu
><mhiramat@kernel.org>; Damien Le Moal
><damien.lemoal@opensource.wdc.com>; Michael Turquette
><mturquette@baylibre.com>; Stephen Boyd <sboyd@kernel.org>; Geert
>Uytterhoeven <geert+renesas@glider.be>; Sylwester Nawrocki
><s.nawrocki@samsung.com>; Tomasz Figa <tomasz.figa@gmail.com>;
>Chanwoo Choi <cw00.choi@samsung.com>; Alim Akhtar
><alim.akhtar@samsung.com>; Vladimir Zapolskiy <vz@mleia.com>; Herbert
>Xu <herbert@gondor.apana.org.au>; David S. Miller
><davem@davemloft.net>; Andrzej Hajda <andrzej.hajda@intel.com>; Neil
>Armstrong <neil.armstrong@linaro.org>; Robert Foss
><robert.foss@linaro.org>; Laurent Pinchart
><Laurent.pinchart@ideasonboard.com>; Jonas Karlman <jonas@kwiboo.se>;
>Jernej Skrabec <jernej.skrabec@gmail.com>; David Airlie <airlied@linux.ie>;
>Daniel Vetter <daniel@ffwll.ch>; Rob Clark <robdclark@gmail.com>; Abhinav
>Kumar <quic_abhinavk@quicinc.com>; Dmitry Baryshkov
><dmitry.baryshkov@linaro.org>; Sean Paul <sean@poorly.run>; Inki Dae
><inki.dae@samsung.com>; Seung-Woo Kim <sw0312.kim@samsung.com>;
>Kyungmin Park <kyungmin.park@samsung.com>; Thierry Reding
><thierry.reding@gmail.com>; Jonathan Hunter <jonathanh@nvidia.com>;
>Masahiro Yamada <yamada.masahiro@socionext.com>; Florian Fainelli
><f.fainelli@gmail.com>; Linus Walleij <linus.walleij@linaro.org>; Andre
>Przywara <andre.przywara@arm.com>; Kuninori Morimoto
><kuninori.morimoto.gx@renesas.com>; Yoshihiro Shimoda
><yoshihiro.shimoda.uh@renesas.com>; Marek Vasut <marex@denx.de>;
>Krishna Manikandan <quic_mkrishn@quicinc.com>;
>devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-
>kernel@vger.kernel.org; linux-ide@vger.kernel.org; linux-
>clk@vger.kernel.org; linux-renesas-soc@vger.kernel.org; linux-samsung-
>soc@vger.kernel.org; linux-crypto@vger.kernel.org; dri-
>devel@lists.freedesktop.org; linux-arm-msm@vger.kernel.org;
>freedreno@lists.freedesktop.org; linux-tegra@vger.kernel.org
>Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>Subject: [PATCH 4/5] dt-bindings: crypto: 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@linaro.org>
>---
Reviewed-by: Alim Akhtar <alim.akhtar@samsung.com>
> Documentation/devicetree/bindings/crypto/samsung-slimsss.yaml | 1 -
> 1 file changed, 1 deletion(-)
>
>diff --git a/Documentation/devicetree/bindings/crypto/samsung-slimsss.yaml
>b/Documentation/devicetree/bindings/crypto/samsung-slimsss.yaml
>index 676950bb7b37..5b31891c97fe 100644
>--- a/Documentation/devicetree/bindings/crypto/samsung-slimsss.yaml
>+++ b/Documentation/devicetree/bindings/crypto/samsung-slimsss.yaml
>@@ -24,7 +24,6 @@ properties:
> maxItems: 1
>
> clocks:
>- minItems: 2
> maxItems: 2
>
> clock-names:
>--
>2.34.1
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 4/5] dt-bindings: crypto: drop minItems equal to maxItems
2022-08-25 11:33 ` [PATCH 4/5] dt-bindings: crypto: " Krzysztof Kozlowski
2022-08-25 17:01 ` Laurent Pinchart
2022-08-25 17:02 ` Alim Akhtar
@ 2022-08-29 19:56 ` Rob Herring
2 siblings, 0 replies; 20+ messages in thread
From: Rob Herring @ 2022-08-29 19:56 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Herbert Xu, Abhinav Kumar, David S. Miller, Thierry Reding,
linux-samsung-soc, Jernej Skrabec, Sean Paul, Neil Armstrong,
freedreno, Andre Przywara, Jonas Karlman, Dmitry Baryshkov,
Kuninori Morimoto, linux-arm-msm, linux-ide, Krzysztof Kozlowski,
devicetree, Damien Le Moal, linux-crypto, Kunihiko Hayashi,
Linus Walleij, Stephen Boyd, Robert Foss, Chanwoo Choi,
Masahiro Yamada, Alim Akhtar, Florian Fainelli, linux-kernel,
dri-devel, linux-tegra, Laurent Pinchart, Seung-Woo Kim,
Rob Herring, Tomasz Figa, Sylwester Nawrocki, Yoshihiro Shimoda,
Michael Turquette, linux-clk, Geert Uytterhoeven, Inki Dae,
Vladimir Zapolskiy, Jonathan Hunter, Kyungmin Park, Marek Vasut,
Andrzej Hajda, Krishna Manikandan, Masami Hiramatsu,
linux-renesas-soc, David Airlie, Daniel Vetter, linux-arm-kernel,
Rob Clark
On Thu, 25 Aug 2022 14:33:33 +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@linaro.org>
> ---
> Documentation/devicetree/bindings/crypto/samsung-slimsss.yaml | 1 -
> 1 file changed, 1 deletion(-)
>
Applied, thanks!
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH 5/5] dt-bindings: display: drop minItems equal to maxItems
2022-08-25 11:33 [PATCH 1/5] dt-bindings: socionext,uniphier-system-cache: drop minItems equal to maxItems Krzysztof Kozlowski
` (2 preceding siblings ...)
2022-08-25 11:33 ` [PATCH 4/5] dt-bindings: crypto: " Krzysztof Kozlowski
@ 2022-08-25 11:33 ` Krzysztof Kozlowski
2022-08-25 17:01 ` Laurent Pinchart
` (3 more replies)
2022-08-25 16:27 ` [PATCH 1/5] dt-bindings: socionext,uniphier-system-cache: " Laurent Pinchart
2022-08-29 19:55 ` Rob Herring
5 siblings, 4 replies; 20+ messages in thread
From: Krzysztof Kozlowski @ 2022-08-25 11:33 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Kunihiko Hayashi,
Masami Hiramatsu, Damien Le Moal, Michael Turquette,
Stephen Boyd, Geert Uytterhoeven, Sylwester Nawrocki,
Tomasz Figa, Chanwoo Choi, Alim Akhtar, Vladimir Zapolskiy,
Herbert Xu, David S. Miller, Andrzej Hajda, Neil Armstrong,
Robert Foss, Laurent Pinchart, Jonas Karlman, Jernej Skrabec,
David Airlie, Daniel Vetter, Rob Clark, Abhinav Kumar,
Dmitry Baryshkov, Sean Paul, Inki Dae, Seung-Woo Kim,
Kyungmin Park, Thierry Reding, Jonathan Hunter, Masahiro Yamada,
Florian Fainelli, Linus Walleij, Andre Przywara,
Kuninori Morimoto, Yoshihiro Shimoda, Marek Vasut,
Krishna Manikandan, devicetree, linux-arm-kernel, linux-kernel,
linux-ide, linux-clk, linux-renesas-soc, linux-samsung-soc,
linux-crypto, dri-devel, linux-arm-msm, freedreno, linux-tegra
Cc: Krzysztof Kozlowski
minItems, if missing, are implicitly equal to maxItems, so drop
redundant piece to reduce size of code.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml | 1 -
.../devicetree/bindings/display/msm/dsi-controller-main.yaml | 2 --
Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml | 2 --
.../bindings/display/samsung/samsung,exynos5433-decon.yaml | 2 --
.../bindings/display/samsung/samsung,exynos5433-mic.yaml | 1 -
.../bindings/display/samsung/samsung,exynos7-decon.yaml | 1 -
.../devicetree/bindings/display/samsung/samsung,fimd.yaml | 1 -
.../devicetree/bindings/display/tegra/nvidia,tegra20-gr3d.yaml | 1 -
.../devicetree/bindings/display/tegra/nvidia,tegra20-mpe.yaml | 2 --
9 files changed, 13 deletions(-)
diff --git a/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml b/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml
index 2ebaa43eb62e..b19be0804abe 100644
--- a/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml
@@ -25,7 +25,6 @@ properties:
const: ldb
reg:
- minItems: 2
maxItems: 2
reg-names:
diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
index 880bfe930830..3b609c19e0bc 100644
--- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
@@ -66,13 +66,11 @@ properties:
2 DSI links.
assigned-clocks:
- minItems: 2
maxItems: 2
description: |
Parents of "byte" and "pixel" for the given platform.
assigned-clock-parents:
- minItems: 2
maxItems: 2
description: |
The Byte clock and Pixel clock PLL outputs provided by a DSI PHY block.
diff --git a/Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml b/Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml
index 716f921e3532..d9ad8b659f58 100644
--- a/Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml
@@ -37,7 +37,6 @@ properties:
qcom,phy-rescode-offset-top:
$ref: /schemas/types.yaml#/definitions/int8-array
- minItems: 5
maxItems: 5
description:
Integer array of offset for pull-up legs rescode for all five lanes.
@@ -49,7 +48,6 @@ properties:
qcom,phy-rescode-offset-bot:
$ref: /schemas/types.yaml#/definitions/int8-array
- minItems: 5
maxItems: 5
description:
Integer array of offset for pull-down legs rescode for all five lanes.
diff --git a/Documentation/devicetree/bindings/display/samsung/samsung,exynos5433-decon.yaml b/Documentation/devicetree/bindings/display/samsung/samsung,exynos5433-decon.yaml
index 921bfe925cd6..6380eeebb073 100644
--- a/Documentation/devicetree/bindings/display/samsung/samsung,exynos5433-decon.yaml
+++ b/Documentation/devicetree/bindings/display/samsung/samsung,exynos5433-decon.yaml
@@ -24,7 +24,6 @@ properties:
- samsung,exynos5433-decon-tv
clocks:
- minItems: 11
maxItems: 11
clock-names:
@@ -59,7 +58,6 @@ properties:
- const: te
iommus:
- minItems: 2
maxItems: 2
iommu-names:
diff --git a/Documentation/devicetree/bindings/display/samsung/samsung,exynos5433-mic.yaml b/Documentation/devicetree/bindings/display/samsung/samsung,exynos5433-mic.yaml
index 7d405f2febcd..26e5017737a3 100644
--- a/Documentation/devicetree/bindings/display/samsung/samsung,exynos5433-mic.yaml
+++ b/Documentation/devicetree/bindings/display/samsung/samsung,exynos5433-mic.yaml
@@ -24,7 +24,6 @@ properties:
const: samsung,exynos5433-mic
clocks:
- minItems: 2
maxItems: 2
clock-names:
diff --git a/Documentation/devicetree/bindings/display/samsung/samsung,exynos7-decon.yaml b/Documentation/devicetree/bindings/display/samsung/samsung,exynos7-decon.yaml
index 969bd8c563a5..c06f306e8d14 100644
--- a/Documentation/devicetree/bindings/display/samsung/samsung,exynos7-decon.yaml
+++ b/Documentation/devicetree/bindings/display/samsung/samsung,exynos7-decon.yaml
@@ -22,7 +22,6 @@ properties:
const: samsung,exynos7-decon
clocks:
- minItems: 4
maxItems: 4
clock-names:
diff --git a/Documentation/devicetree/bindings/display/samsung/samsung,fimd.yaml b/Documentation/devicetree/bindings/display/samsung/samsung,fimd.yaml
index 5d5cc220f78a..210d856b3b57 100644
--- a/Documentation/devicetree/bindings/display/samsung/samsung,fimd.yaml
+++ b/Documentation/devicetree/bindings/display/samsung/samsung,fimd.yaml
@@ -27,7 +27,6 @@ properties:
const: 1
clocks:
- minItems: 2
maxItems: 2
clock-names:
diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-gr3d.yaml b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-gr3d.yaml
index dbdf0229d9f6..4755a73473c7 100644
--- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-gr3d.yaml
+++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-gr3d.yaml
@@ -59,7 +59,6 @@ properties:
maxItems: 2
power-domain-names:
- minItems: 2
maxItems: 2
allOf:
diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-mpe.yaml b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-mpe.yaml
index 4154ae01ad13..5f4f0fb4b692 100644
--- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-mpe.yaml
+++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-mpe.yaml
@@ -42,11 +42,9 @@ properties:
maxItems: 1
interconnects:
- minItems: 6
maxItems: 6
interconnect-names:
- minItems: 6
maxItems: 6
operating-points-v2:
--
2.34.1
^ permalink raw reply related [flat|nested] 20+ messages in thread* Re: [PATCH 5/5] dt-bindings: display: drop minItems equal to maxItems
2022-08-25 11:33 ` [PATCH 5/5] dt-bindings: display: " Krzysztof Kozlowski
@ 2022-08-25 17:01 ` Laurent Pinchart
2022-08-25 17:09 ` Alim Akhtar
` (2 subsequent siblings)
3 siblings, 0 replies; 20+ messages in thread
From: Laurent Pinchart @ 2022-08-25 17:01 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Rob Herring, Krzysztof Kozlowski, Kunihiko Hayashi,
Masami Hiramatsu, Damien Le Moal, Michael Turquette,
Stephen Boyd, Geert Uytterhoeven, Sylwester Nawrocki,
Tomasz Figa, Chanwoo Choi, Alim Akhtar, Vladimir Zapolskiy,
Herbert Xu, David S. Miller, Andrzej Hajda, Neil Armstrong,
Robert Foss, Jonas Karlman, Jernej Skrabec, David Airlie,
Daniel Vetter, Rob Clark, Abhinav Kumar, Dmitry Baryshkov,
Sean Paul, Inki Dae, Seung-Woo Kim, Kyungmin Park,
Thierry Reding, Jonathan Hunter, Masahiro Yamada,
Florian Fainelli, Linus Walleij, Andre Przywara,
Kuninori Morimoto, Yoshihiro Shimoda, Marek Vasut,
Krishna Manikandan, devicetree, linux-arm-kernel, linux-kernel,
linux-ide, linux-clk, linux-renesas-soc, linux-samsung-soc,
linux-crypto, dri-devel, linux-arm-msm, freedreno, linux-tegra
Hi Krzysztof,
Thank you for the patch.
On Thu, Aug 25, 2022 at 02:33:34PM +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@linaro.org>
Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> ---
> Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml | 1 -
> .../devicetree/bindings/display/msm/dsi-controller-main.yaml | 2 --
> Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml | 2 --
> .../bindings/display/samsung/samsung,exynos5433-decon.yaml | 2 --
> .../bindings/display/samsung/samsung,exynos5433-mic.yaml | 1 -
> .../bindings/display/samsung/samsung,exynos7-decon.yaml | 1 -
> .../devicetree/bindings/display/samsung/samsung,fimd.yaml | 1 -
> .../devicetree/bindings/display/tegra/nvidia,tegra20-gr3d.yaml | 1 -
> .../devicetree/bindings/display/tegra/nvidia,tegra20-mpe.yaml | 2 --
> 9 files changed, 13 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml b/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml
> index 2ebaa43eb62e..b19be0804abe 100644
> --- a/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml
> @@ -25,7 +25,6 @@ properties:
> const: ldb
>
> reg:
> - minItems: 2
> maxItems: 2
>
> reg-names:
> diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> index 880bfe930830..3b609c19e0bc 100644
> --- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> @@ -66,13 +66,11 @@ properties:
> 2 DSI links.
>
> assigned-clocks:
> - minItems: 2
> maxItems: 2
> description: |
> Parents of "byte" and "pixel" for the given platform.
>
> assigned-clock-parents:
> - minItems: 2
> maxItems: 2
> description: |
> The Byte clock and Pixel clock PLL outputs provided by a DSI PHY block.
> diff --git a/Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml b/Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml
> index 716f921e3532..d9ad8b659f58 100644
> --- a/Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml
> @@ -37,7 +37,6 @@ properties:
>
> qcom,phy-rescode-offset-top:
> $ref: /schemas/types.yaml#/definitions/int8-array
> - minItems: 5
> maxItems: 5
> description:
> Integer array of offset for pull-up legs rescode for all five lanes.
> @@ -49,7 +48,6 @@ properties:
>
> qcom,phy-rescode-offset-bot:
> $ref: /schemas/types.yaml#/definitions/int8-array
> - minItems: 5
> maxItems: 5
> description:
> Integer array of offset for pull-down legs rescode for all five lanes.
> diff --git a/Documentation/devicetree/bindings/display/samsung/samsung,exynos5433-decon.yaml b/Documentation/devicetree/bindings/display/samsung/samsung,exynos5433-decon.yaml
> index 921bfe925cd6..6380eeebb073 100644
> --- a/Documentation/devicetree/bindings/display/samsung/samsung,exynos5433-decon.yaml
> +++ b/Documentation/devicetree/bindings/display/samsung/samsung,exynos5433-decon.yaml
> @@ -24,7 +24,6 @@ properties:
> - samsung,exynos5433-decon-tv
>
> clocks:
> - minItems: 11
> maxItems: 11
>
> clock-names:
> @@ -59,7 +58,6 @@ properties:
> - const: te
>
> iommus:
> - minItems: 2
> maxItems: 2
>
> iommu-names:
> diff --git a/Documentation/devicetree/bindings/display/samsung/samsung,exynos5433-mic.yaml b/Documentation/devicetree/bindings/display/samsung/samsung,exynos5433-mic.yaml
> index 7d405f2febcd..26e5017737a3 100644
> --- a/Documentation/devicetree/bindings/display/samsung/samsung,exynos5433-mic.yaml
> +++ b/Documentation/devicetree/bindings/display/samsung/samsung,exynos5433-mic.yaml
> @@ -24,7 +24,6 @@ properties:
> const: samsung,exynos5433-mic
>
> clocks:
> - minItems: 2
> maxItems: 2
>
> clock-names:
> diff --git a/Documentation/devicetree/bindings/display/samsung/samsung,exynos7-decon.yaml b/Documentation/devicetree/bindings/display/samsung/samsung,exynos7-decon.yaml
> index 969bd8c563a5..c06f306e8d14 100644
> --- a/Documentation/devicetree/bindings/display/samsung/samsung,exynos7-decon.yaml
> +++ b/Documentation/devicetree/bindings/display/samsung/samsung,exynos7-decon.yaml
> @@ -22,7 +22,6 @@ properties:
> const: samsung,exynos7-decon
>
> clocks:
> - minItems: 4
> maxItems: 4
>
> clock-names:
> diff --git a/Documentation/devicetree/bindings/display/samsung/samsung,fimd.yaml b/Documentation/devicetree/bindings/display/samsung/samsung,fimd.yaml
> index 5d5cc220f78a..210d856b3b57 100644
> --- a/Documentation/devicetree/bindings/display/samsung/samsung,fimd.yaml
> +++ b/Documentation/devicetree/bindings/display/samsung/samsung,fimd.yaml
> @@ -27,7 +27,6 @@ properties:
> const: 1
>
> clocks:
> - minItems: 2
> maxItems: 2
>
> clock-names:
> diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-gr3d.yaml b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-gr3d.yaml
> index dbdf0229d9f6..4755a73473c7 100644
> --- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-gr3d.yaml
> +++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-gr3d.yaml
> @@ -59,7 +59,6 @@ properties:
> maxItems: 2
>
> power-domain-names:
> - minItems: 2
> maxItems: 2
>
> allOf:
> diff --git a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-mpe.yaml b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-mpe.yaml
> index 4154ae01ad13..5f4f0fb4b692 100644
> --- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-mpe.yaml
> +++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-mpe.yaml
> @@ -42,11 +42,9 @@ properties:
> maxItems: 1
>
> interconnects:
> - minItems: 6
> maxItems: 6
>
> interconnect-names:
> - minItems: 6
> maxItems: 6
>
> operating-points-v2:
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 20+ messages in thread* RE: [PATCH 5/5] dt-bindings: display: drop minItems equal to maxItems
2022-08-25 11:33 ` [PATCH 5/5] dt-bindings: display: " Krzysztof Kozlowski
2022-08-25 17:01 ` Laurent Pinchart
@ 2022-08-25 17:09 ` Alim Akhtar
2022-08-26 9:47 ` Dmitry Baryshkov
2022-08-29 19:56 ` Rob Herring
3 siblings, 0 replies; 20+ messages in thread
From: Alim Akhtar @ 2022-08-25 17:09 UTC (permalink / raw)
To: 'Krzysztof Kozlowski', 'Rob Herring',
'Krzysztof Kozlowski', 'Kunihiko Hayashi',
'Masami Hiramatsu', 'Damien Le Moal',
'Michael Turquette', 'Stephen Boyd',
'Geert Uytterhoeven', 'Sylwester Nawrocki',
'Tomasz Figa', 'Chanwoo Choi',
'Vladimir Zapolskiy', 'Herbert Xu',
'David S. Miller', 'Andrzej Hajda',
'Neil Armstrong', 'Robert Foss',
'Laurent Pinchart', 'Jonas Karlman',
'Jernej Skrabec', 'David Airlie',
'Daniel Vetter', 'Rob Clark',
'Abhinav Kumar', 'Dmitry Baryshkov',
'Sean Paul', 'Inki Dae', 'Seung-Woo Kim',
'Kyungmin Park', 'Thierry Reding',
'Jonathan Hunter', 'Masahiro Yamada',
'Florian Fainelli', 'Linus Walleij',
'Andre Przywara', 'Kuninori Morimoto',
'Yoshihiro Shimoda', 'Marek Vasut',
'Krishna Manikandan',
devicetree, linux-arm-kernel, linux-kernel, linux-ide, linux-clk,
linux-renesas-soc, linux-samsung-soc, linux-crypto, dri-devel,
linux-arm-msm, freedreno, linux-tegra
Hi Krzysztof
I got below messages when replied to all
"
Your mail to 'linux-arm-kernel' with the subject
RE: [PATCH 4/5] dt-bindings: crypto: drop minItems equal to
maxItems and for [PATCH 3/5] as well
Is being held until the list moderator can review it for approval.
The reason it is being held:
Too many recipients to the message
"
Are these many "To" addresses are really needed?
>-----Original Message-----
>From: Krzysztof Kozlowski [mailto:krzysztof.kozlowski@linaro.org]
>Sent: Thursday, August 25, 2022 5:04 PM
>To: Rob Herring <robh+dt@kernel.org>; Krzysztof Kozlowski
><krzysztof.kozlowski+dt@linaro.org>; Kunihiko Hayashi
><hayashi.kunihiko@socionext.com>; Masami Hiramatsu
><mhiramat@kernel.org>; Damien Le Moal
><damien.lemoal@opensource.wdc.com>; Michael Turquette
><mturquette@baylibre.com>; Stephen Boyd <sboyd@kernel.org>; Geert
>Uytterhoeven <geert+renesas@glider.be>; Sylwester Nawrocki
><s.nawrocki@samsung.com>; Tomasz Figa <tomasz.figa@gmail.com>;
>Chanwoo Choi <cw00.choi@samsung.com>; Alim Akhtar
><alim.akhtar@samsung.com>; Vladimir Zapolskiy <vz@mleia.com>; Herbert
>Xu <herbert@gondor.apana.org.au>; David S. Miller
><davem@davemloft.net>; Andrzej Hajda <andrzej.hajda@intel.com>; Neil
>Armstrong <neil.armstrong@linaro.org>; Robert Foss
><robert.foss@linaro.org>; Laurent Pinchart
><Laurent.pinchart@ideasonboard.com>; Jonas Karlman <jonas@kwiboo.se>;
>Jernej Skrabec <jernej.skrabec@gmail.com>; David Airlie <airlied@linux.ie>;
>Daniel Vetter <daniel@ffwll.ch>; Rob Clark <robdclark@gmail.com>; Abhinav
>Kumar <quic_abhinavk@quicinc.com>; Dmitry Baryshkov
><dmitry.baryshkov@linaro.org>; Sean Paul <sean@poorly.run>; Inki Dae
><inki.dae@samsung.com>; Seung-Woo Kim <sw0312.kim@samsung.com>;
>Kyungmin Park <kyungmin.park@samsung.com>; Thierry Reding
><thierry.reding@gmail.com>; Jonathan Hunter <jonathanh@nvidia.com>;
>Masahiro Yamada <yamada.masahiro@socionext.com>; Florian Fainelli
><f.fainelli@gmail.com>; Linus Walleij <linus.walleij@linaro.org>; Andre
>Przywara <andre.przywara@arm.com>; Kuninori Morimoto
><kuninori.morimoto.gx@renesas.com>; Yoshihiro Shimoda
><yoshihiro.shimoda.uh@renesas.com>; Marek Vasut <marex@denx.de>;
>Krishna Manikandan <quic_mkrishn@quicinc.com>;
>devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-
>kernel@vger.kernel.org; linux-ide@vger.kernel.org; linux-
>clk@vger.kernel.org; linux-renesas-soc@vger.kernel.org; linux-samsung-
>soc@vger.kernel.org; linux-crypto@vger.kernel.org; dri-
>devel@lists.freedesktop.org; linux-arm-msm@vger.kernel.org;
>freedreno@lists.freedesktop.org; linux-tegra@vger.kernel.org
>Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>Subject: [PATCH 5/5] dt-bindings: display: 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@linaro.org>
>---
Feel free to add
Reviewed-by: Alim Akhtar <alim.akhtar@samsung.com>
> Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml | 1 -
> .../devicetree/bindings/display/msm/dsi-controller-main.yaml | 2 --
> Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml | 2 --
> .../bindings/display/samsung/samsung,exynos5433-decon.yaml | 2 --
> .../bindings/display/samsung/samsung,exynos5433-mic.yaml | 1 -
> .../bindings/display/samsung/samsung,exynos7-decon.yaml | 1 -
> .../devicetree/bindings/display/samsung/samsung,fimd.yaml | 1 -
> .../devicetree/bindings/display/tegra/nvidia,tegra20-gr3d.yaml | 1 -
> .../devicetree/bindings/display/tegra/nvidia,tegra20-mpe.yaml | 2 --
> 9 files changed, 13 deletions(-)
>
>diff --git a/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml
>b/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml
>index 2ebaa43eb62e..b19be0804abe 100644
>--- a/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml
>+++ b/Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml
>@@ -25,7 +25,6 @@ properties:
> const: ldb
>
> reg:
>- minItems: 2
> maxItems: 2
>
> reg-names:
>diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-
>main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-
>controller-main.yaml
>index 880bfe930830..3b609c19e0bc 100644
>--- a/Documentation/devicetree/bindings/display/msm/dsi-controller-
>main.yaml
>+++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.
>+++ yaml
>@@ -66,13 +66,11 @@ properties:
> 2 DSI links.
>
> assigned-clocks:
>- minItems: 2
> maxItems: 2
> description: |
> Parents of "byte" and "pixel" for the given platform.
>
> assigned-clock-parents:
>- minItems: 2
> maxItems: 2
> description: |
> The Byte clock and Pixel clock PLL outputs provided by a DSI PHY
block.
>diff --git a/Documentation/devicetree/bindings/display/msm/dsi-phy-
>10nm.yaml b/Documentation/devicetree/bindings/display/msm/dsi-phy-
>10nm.yaml
>index 716f921e3532..d9ad8b659f58 100644
>--- a/Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml
>+++ b/Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml
>@@ -37,7 +37,6 @@ properties:
>
> qcom,phy-rescode-offset-top:
> $ref: /schemas/types.yaml#/definitions/int8-array
>- minItems: 5
> maxItems: 5
> description:
> Integer array of offset for pull-up legs rescode for all five lanes.
>@@ -49,7 +48,6 @@ properties:
>
> qcom,phy-rescode-offset-bot:
> $ref: /schemas/types.yaml#/definitions/int8-array
>- minItems: 5
> maxItems: 5
> description:
> Integer array of offset for pull-down legs rescode for all five
lanes.
>diff --git
>a/Documentation/devicetree/bindings/display/samsung/samsung,exynos543
>3-decon.yaml
>b/Documentation/devicetree/bindings/display/samsung/samsung,exynos543
>3-decon.yaml
>index 921bfe925cd6..6380eeebb073 100644
>---
>a/Documentation/devicetree/bindings/display/samsung/samsung,exynos543
>3-decon.yaml
>+++
>b/Documentation/devicetree/bindings/display/samsung/samsung,exynos54
>+++ 33-decon.yaml
>@@ -24,7 +24,6 @@ properties:
> - samsung,exynos5433-decon-tv
>
> clocks:
>- minItems: 11
> maxItems: 11
>
> clock-names:
>@@ -59,7 +58,6 @@ properties:
> - const: te
>
> iommus:
>- minItems: 2
> maxItems: 2
>
> iommu-names:
>diff --git
>a/Documentation/devicetree/bindings/display/samsung/samsung,exynos543
>3-mic.yaml
>b/Documentation/devicetree/bindings/display/samsung/samsung,exynos543
>3-mic.yaml
>index 7d405f2febcd..26e5017737a3 100644
>---
>a/Documentation/devicetree/bindings/display/samsung/samsung,exynos543
>3-mic.yaml
>+++
>b/Documentation/devicetree/bindings/display/samsung/samsung,exynos54
>+++ 33-mic.yaml
>@@ -24,7 +24,6 @@ properties:
> const: samsung,exynos5433-mic
>
> clocks:
>- minItems: 2
> maxItems: 2
>
> clock-names:
>diff --git
>a/Documentation/devicetree/bindings/display/samsung/samsung,exynos7-
>decon.yaml
>b/Documentation/devicetree/bindings/display/samsung/samsung,exynos7-
>decon.yaml
>index 969bd8c563a5..c06f306e8d14 100644
>---
>a/Documentation/devicetree/bindings/display/samsung/samsung,exynos7-
>decon.yaml
>+++
>b/Documentation/devicetree/bindings/display/samsung/samsung,exynos7-
>+++ decon.yaml
>@@ -22,7 +22,6 @@ properties:
> const: samsung,exynos7-decon
>
> clocks:
>- minItems: 4
> maxItems: 4
>
> clock-names:
>diff --git
>a/Documentation/devicetree/bindings/display/samsung/samsung,fimd.yaml
>b/Documentation/devicetree/bindings/display/samsung/samsung,fimd.yaml
>index 5d5cc220f78a..210d856b3b57 100644
>---
>a/Documentation/devicetree/bindings/display/samsung/samsung,fimd.yaml
>+++
>b/Documentation/devicetree/bindings/display/samsung/samsung,fimd.yam
>+++ l
>@@ -27,7 +27,6 @@ properties:
> const: 1
>
> clocks:
>- minItems: 2
> maxItems: 2
>
> clock-names:
>diff --git
a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-
>gr3d.yaml
>b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-
>gr3d.yaml
>index dbdf0229d9f6..4755a73473c7 100644
>--- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-
>gr3d.yaml
>+++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-gr3
>+++ d.yaml
>@@ -59,7 +59,6 @@ properties:
> maxItems: 2
>
> power-domain-names:
>- minItems: 2
> maxItems: 2
>
> allOf:
>diff --git
a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-
>mpe.yaml
>b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-
>mpe.yaml
>index 4154ae01ad13..5f4f0fb4b692 100644
>--- a/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-
>mpe.yaml
>+++ b/Documentation/devicetree/bindings/display/tegra/nvidia,tegra20-mpe
>+++ .yaml
>@@ -42,11 +42,9 @@ properties:
> maxItems: 1
>
> interconnects:
>- minItems: 6
> maxItems: 6
>
> interconnect-names:
>- minItems: 6
> maxItems: 6
>
> operating-points-v2:
>--
>2.34.1
^ permalink raw reply [flat|nested] 20+ messages in thread* Re: [PATCH 5/5] dt-bindings: display: drop minItems equal to maxItems
2022-08-25 11:33 ` [PATCH 5/5] dt-bindings: display: " Krzysztof Kozlowski
2022-08-25 17:01 ` Laurent Pinchart
2022-08-25 17:09 ` Alim Akhtar
@ 2022-08-26 9:47 ` Dmitry Baryshkov
2022-08-29 19:56 ` Rob Herring
3 siblings, 0 replies; 20+ messages in thread
From: Dmitry Baryshkov @ 2022-08-26 9:47 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Krzysztof Kozlowski,
Kunihiko Hayashi, Masami Hiramatsu, Damien Le Moal,
Michael Turquette, Stephen Boyd, Geert Uytterhoeven,
Sylwester Nawrocki, Tomasz Figa, Chanwoo Choi, Alim Akhtar,
Vladimir Zapolskiy, Herbert Xu, David S. Miller, Andrzej Hajda,
Neil Armstrong, Robert Foss, Laurent Pinchart, Jonas Karlman,
Jernej Skrabec, David Airlie, Daniel Vetter, Rob Clark,
Abhinav Kumar, Sean Paul, Inki Dae, Seung-Woo Kim, Kyungmin Park,
Thierry Reding, Jonathan Hunter, Masahiro Yamada,
Florian Fainelli, Linus Walleij, Andre Przywara,
Kuninori Morimoto, Yoshihiro Shimoda, Marek Vasut,
Krishna Manikandan, devicetree, linux-arm-kernel, linux-kernel,
linux-ide, linux-clk, linux-renesas-soc, linux-samsung-soc,
linux-crypto, dri-devel, linux-arm-msm, freedreno, linux-tegra
On 25/08/2022 14:33, 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@linaro.org>
> ---
> Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml | 1 -
> .../devicetree/bindings/display/msm/dsi-controller-main.yaml | 2 --
> Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml | 2 --
For msm changes:
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> .../bindings/display/samsung/samsung,exynos5433-decon.yaml | 2 --
> .../bindings/display/samsung/samsung,exynos5433-mic.yaml | 1 -
> .../bindings/display/samsung/samsung,exynos7-decon.yaml | 1 -
> .../devicetree/bindings/display/samsung/samsung,fimd.yaml | 1 -
> .../devicetree/bindings/display/tegra/nvidia,tegra20-gr3d.yaml | 1 -
> .../devicetree/bindings/display/tegra/nvidia,tegra20-mpe.yaml | 2 --
> 9 files changed, 13 deletions(-)
>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 5/5] dt-bindings: display: drop minItems equal to maxItems
2022-08-25 11:33 ` [PATCH 5/5] dt-bindings: display: " Krzysztof Kozlowski
` (2 preceding siblings ...)
2022-08-26 9:47 ` Dmitry Baryshkov
@ 2022-08-29 19:56 ` Rob Herring
3 siblings, 0 replies; 20+ messages in thread
From: Rob Herring @ 2022-08-29 19:56 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Vladimir Zapolskiy, linux-tegra, freedreno, Rob Herring,
Robert Foss, linux-ide, Jernej Skrabec, Krzysztof Kozlowski,
Chanwoo Choi, Dmitry Baryshkov, Masami Hiramatsu, Andre Przywara,
Michael Turquette, Sean Paul, Stephen Boyd, linux-arm-kernel,
linux-arm-msm, Damien Le Moal, Neil Armstrong, Daniel Vetter,
Florian Fainelli, linux-samsung-soc, Geert Uytterhoeven,
Herbert Xu, David S. Miller, Rob Clark, Kyungmin Park, linux-clk,
dri-devel, devicetree, Andrzej Hajda, Jonathan Hunter,
Linus Walleij, Seung-Woo Kim, Jonas Karlman, Alim Akhtar,
David Airlie, Krishna Manikandan, Thierry Reding, Marek Vasut,
linux-renesas-soc, Laurent Pinchart, Inki Dae, Kunihiko Hayashi,
Tomasz Figa, Masahiro Yamada, Abhinav Kumar, Kuninori Morimoto,
linux-kernel, linux-crypto, Yoshihiro Shimoda,
Sylwester Nawrocki
On Thu, 25 Aug 2022 14:33:34 +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@linaro.org>
> ---
> Documentation/devicetree/bindings/display/bridge/fsl,ldb.yaml | 1 -
> .../devicetree/bindings/display/msm/dsi-controller-main.yaml | 2 --
> Documentation/devicetree/bindings/display/msm/dsi-phy-10nm.yaml | 2 --
> .../bindings/display/samsung/samsung,exynos5433-decon.yaml | 2 --
> .../bindings/display/samsung/samsung,exynos5433-mic.yaml | 1 -
> .../bindings/display/samsung/samsung,exynos7-decon.yaml | 1 -
> .../devicetree/bindings/display/samsung/samsung,fimd.yaml | 1 -
> .../devicetree/bindings/display/tegra/nvidia,tegra20-gr3d.yaml | 1 -
> .../devicetree/bindings/display/tegra/nvidia,tegra20-mpe.yaml | 2 --
> 9 files changed, 13 deletions(-)
>
Applied, thanks!
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 1/5] dt-bindings: socionext,uniphier-system-cache: drop minItems equal to maxItems
2022-08-25 11:33 [PATCH 1/5] dt-bindings: socionext,uniphier-system-cache: drop minItems equal to maxItems Krzysztof Kozlowski
` (3 preceding siblings ...)
2022-08-25 11:33 ` [PATCH 5/5] dt-bindings: display: " Krzysztof Kozlowski
@ 2022-08-25 16:27 ` Laurent Pinchart
2022-08-29 19:55 ` Rob Herring
5 siblings, 0 replies; 20+ messages in thread
From: Laurent Pinchart @ 2022-08-25 16:27 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Rob Herring, Krzysztof Kozlowski, Kunihiko Hayashi,
Masami Hiramatsu, Damien Le Moal, Michael Turquette,
Stephen Boyd, Geert Uytterhoeven, Sylwester Nawrocki,
Tomasz Figa, Chanwoo Choi, Alim Akhtar, Vladimir Zapolskiy,
Herbert Xu, David S. Miller, Andrzej Hajda, Neil Armstrong,
Robert Foss, Jonas Karlman, Jernej Skrabec, David Airlie,
Daniel Vetter, Rob Clark, Abhinav Kumar, Dmitry Baryshkov,
Sean Paul, Inki Dae, Seung-Woo Kim, Kyungmin Park,
Thierry Reding, Jonathan Hunter, Masahiro Yamada,
Florian Fainelli, Linus Walleij, Andre Przywara,
Kuninori Morimoto, Yoshihiro Shimoda, Marek Vasut,
Krishna Manikandan, devicetree, linux-arm-kernel, linux-kernel,
linux-ide, linux-clk, linux-renesas-soc, linux-samsung-soc,
linux-crypto, dri-devel, linux-arm-msm, freedreno, linux-tegra
Hi Krzysztof,
Thank you for the patch.
On Thu, Aug 25, 2022 at 02:33:30PM +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@linaro.org>
Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> ---
> .../bindings/arm/socionext/socionext,uniphier-system-cache.yaml | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/arm/socionext/socionext,uniphier-system-cache.yaml b/Documentation/devicetree/bindings/arm/socionext/socionext,uniphier-system-cache.yaml
> index 7ca5375f278f..6096c082d56d 100644
> --- a/Documentation/devicetree/bindings/arm/socionext/socionext,uniphier-system-cache.yaml
> +++ b/Documentation/devicetree/bindings/arm/socionext/socionext,uniphier-system-cache.yaml
> @@ -22,7 +22,6 @@ properties:
> description: |
> should contain 3 regions: control register, revision register,
> operation register, in this order.
> - minItems: 3
> maxItems: 3
>
> interrupts:
--
Regards,
Laurent Pinchart
^ permalink raw reply [flat|nested] 20+ messages in thread* Re: [PATCH 1/5] dt-bindings: socionext,uniphier-system-cache: drop minItems equal to maxItems
2022-08-25 11:33 [PATCH 1/5] dt-bindings: socionext,uniphier-system-cache: drop minItems equal to maxItems Krzysztof Kozlowski
` (4 preceding siblings ...)
2022-08-25 16:27 ` [PATCH 1/5] dt-bindings: socionext,uniphier-system-cache: " Laurent Pinchart
@ 2022-08-29 19:55 ` Rob Herring
5 siblings, 0 replies; 20+ messages in thread
From: Rob Herring @ 2022-08-29 19:55 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Laurent Pinchart, Andre Przywara, Sylwester Nawrocki,
Daniel Vetter, linux-kernel, Jonas Karlman, dri-devel,
linux-arm-msm, Andrzej Hajda, Florian Fainelli, Inki Dae,
linux-arm-kernel, Masahiro Yamada, linux-ide, Herbert Xu,
Kunihiko Hayashi, Masami Hiramatsu, Jonathan Hunter, devicetree,
David S. Miller, linux-tegra, Sean Paul, Stephen Boyd,
David Airlie, Thierry Reding, linux-samsung-soc, Seung-Woo Kim,
Rob Clark, linux-renesas-soc, freedreno, Abhinav Kumar,
Vladimir Zapolskiy, Neil Armstrong, Kyungmin Park,
Krzysztof Kozlowski, Marek Vasut, Michael Turquette, Alim Akhtar,
Kuninori Morimoto, Rob Herring, Geert Uytterhoeven, Chanwoo Choi,
Krishna Manikandan, Dmitry Baryshkov, Yoshihiro Shimoda,
linux-crypto, linux-clk, Damien Le Moal, Tomasz Figa,
Jernej Skrabec, Robert Foss, Linus Walleij
On Thu, 25 Aug 2022 14:33:30 +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@linaro.org>
> ---
> .../bindings/arm/socionext/socionext,uniphier-system-cache.yaml | 1 -
> 1 file changed, 1 deletion(-)
>
Applied, thanks!
^ permalink raw reply [flat|nested] 20+ messages in thread