* [PATCH] dt-bindings: Remove redundant 'maxItems'
@ 2020-06-11 19:47 Rob Herring
2020-06-11 22:35 ` Stephen Boyd
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Rob Herring @ 2020-06-11 19:47 UTC (permalink / raw)
To: devicetree
Cc: linux-kernel, Stephen Boyd, Shawn Guo, Sascha Hauer,
Thierry Reding, Anson Huang, linux-clk, linux-pwm, linux-usb
There's no need to specify 'maxItems' with the same value as the number
of entries in 'items'. A meta-schema update will catch future cases.
Cc: Stephen Boyd <sboyd@kernel.org>
Cc: Shawn Guo <shawnguo@kernel.org>
Cc: Sascha Hauer <s.hauer@pengutronix.de>
Cc: Thierry Reding <thierry.reding@gmail.com>
Cc: Anson Huang <Anson.Huang@nxp.com>
Cc: linux-clk@vger.kernel.org
Cc: linux-pwm@vger.kernel.org
Cc: linux-usb@vger.kernel.org
Signed-off-by: Rob Herring <robh@kernel.org>
---
Documentation/devicetree/bindings/clock/imx6q-clock.yaml | 1 -
Documentation/devicetree/bindings/clock/imx6sl-clock.yaml | 1 -
Documentation/devicetree/bindings/clock/imx6sll-clock.yaml | 1 -
Documentation/devicetree/bindings/clock/imx6sx-clock.yaml | 1 -
Documentation/devicetree/bindings/clock/imx6ul-clock.yaml | 1 -
Documentation/devicetree/bindings/pwm/imx-pwm.yaml | 2 --
Documentation/devicetree/bindings/usb/nvidia,tegra-xudc.yaml | 2 --
7 files changed, 9 deletions(-)
diff --git a/Documentation/devicetree/bindings/clock/imx6q-clock.yaml b/Documentation/devicetree/bindings/clock/imx6q-clock.yaml
index 429e3b62b965..92a8e545e212 100644
--- a/Documentation/devicetree/bindings/clock/imx6q-clock.yaml
+++ b/Documentation/devicetree/bindings/clock/imx6q-clock.yaml
@@ -23,7 +23,6 @@ properties:
items:
- description: CCM interrupt request 1
- description: CCM interrupt request 2
- maxItems: 2
'#clock-cells':
const: 1
diff --git a/Documentation/devicetree/bindings/clock/imx6sl-clock.yaml b/Documentation/devicetree/bindings/clock/imx6sl-clock.yaml
index 135568c46350..c97bf95b4150 100644
--- a/Documentation/devicetree/bindings/clock/imx6sl-clock.yaml
+++ b/Documentation/devicetree/bindings/clock/imx6sl-clock.yaml
@@ -23,7 +23,6 @@ properties:
items:
- description: CCM interrupt request 1
- description: CCM interrupt request 2
- maxItems: 2
'#clock-cells':
const: 1
diff --git a/Documentation/devicetree/bindings/clock/imx6sll-clock.yaml b/Documentation/devicetree/bindings/clock/imx6sll-clock.yaml
index fa55f1ce3e57..de48924be191 100644
--- a/Documentation/devicetree/bindings/clock/imx6sll-clock.yaml
+++ b/Documentation/devicetree/bindings/clock/imx6sll-clock.yaml
@@ -23,7 +23,6 @@ properties:
items:
- description: CCM interrupt request 1
- description: CCM interrupt request 2
- maxItems: 2
'#clock-cells':
const: 1
diff --git a/Documentation/devicetree/bindings/clock/imx6sx-clock.yaml b/Documentation/devicetree/bindings/clock/imx6sx-clock.yaml
index 982d698e8c54..e50cddee43c3 100644
--- a/Documentation/devicetree/bindings/clock/imx6sx-clock.yaml
+++ b/Documentation/devicetree/bindings/clock/imx6sx-clock.yaml
@@ -23,7 +23,6 @@ properties:
items:
- description: CCM interrupt request 1
- description: CCM interrupt request 2
- maxItems: 2
'#clock-cells':
const: 1
diff --git a/Documentation/devicetree/bindings/clock/imx6ul-clock.yaml b/Documentation/devicetree/bindings/clock/imx6ul-clock.yaml
index 3c779eea6394..36ce7667c972 100644
--- a/Documentation/devicetree/bindings/clock/imx6ul-clock.yaml
+++ b/Documentation/devicetree/bindings/clock/imx6ul-clock.yaml
@@ -23,7 +23,6 @@ properties:
items:
- description: CCM interrupt request 1
- description: CCM interrupt request 2
- maxItems: 2
'#clock-cells':
const: 1
diff --git a/Documentation/devicetree/bindings/pwm/imx-pwm.yaml b/Documentation/devicetree/bindings/pwm/imx-pwm.yaml
index 4b62af27d4b3..01df06777cba 100644
--- a/Documentation/devicetree/bindings/pwm/imx-pwm.yaml
+++ b/Documentation/devicetree/bindings/pwm/imx-pwm.yaml
@@ -30,13 +30,11 @@ properties:
items:
- description: SoC PWM ipg clock
- description: SoC PWM per clock
- maxItems: 2
clock-names:
items:
- const: ipg
- const: per
- maxItems: 2
interrupts:
maxItems: 1
diff --git a/Documentation/devicetree/bindings/usb/nvidia,tegra-xudc.yaml b/Documentation/devicetree/bindings/usb/nvidia,tegra-xudc.yaml
index c4ddc0adf101..0073763a30d8 100644
--- a/Documentation/devicetree/bindings/usb/nvidia,tegra-xudc.yaml
+++ b/Documentation/devicetree/bindings/usb/nvidia,tegra-xudc.yaml
@@ -64,13 +64,11 @@ properties:
- const: hs_src
power-domains:
- maxItems: 2
items:
- description: XUSBB(device) power-domain
- description: XUSBA(superspeed) power-domain
power-domain-names:
- maxItems: 2
items:
- const: dev
- const: ss
--
2.25.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] dt-bindings: Remove redundant 'maxItems'
2020-06-11 19:47 [PATCH] dt-bindings: Remove redundant 'maxItems' Rob Herring
@ 2020-06-11 22:35 ` Stephen Boyd
2020-06-12 6:47 ` Thierry Reding
2020-06-26 16:23 ` Uwe Kleine-König
2 siblings, 0 replies; 4+ messages in thread
From: Stephen Boyd @ 2020-06-11 22:35 UTC (permalink / raw)
To: Rob Herring, devicetree
Cc: linux-kernel, Shawn Guo, Sascha Hauer, Thierry Reding,
Anson Huang, linux-clk, linux-pwm, linux-usb
Quoting Rob Herring (2020-06-11 12:47:38)
> There's no need to specify 'maxItems' with the same value as the number
> of entries in 'items'. A meta-schema update will catch future cases.
>
> Cc: Stephen Boyd <sboyd@kernel.org>
> Cc: Shawn Guo <shawnguo@kernel.org>
> Cc: Sascha Hauer <s.hauer@pengutronix.de>
> Cc: Thierry Reding <thierry.reding@gmail.com>
> Cc: Anson Huang <Anson.Huang@nxp.com>
> Cc: linux-clk@vger.kernel.org
> Cc: linux-pwm@vger.kernel.org
> Cc: linux-usb@vger.kernel.org
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
Reviewed-by: Stephen Boyd <sboyd@kernel.org> # clk
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] dt-bindings: Remove redundant 'maxItems'
2020-06-11 19:47 [PATCH] dt-bindings: Remove redundant 'maxItems' Rob Herring
2020-06-11 22:35 ` Stephen Boyd
@ 2020-06-12 6:47 ` Thierry Reding
2020-06-26 16:23 ` Uwe Kleine-König
2 siblings, 0 replies; 4+ messages in thread
From: Thierry Reding @ 2020-06-12 6:47 UTC (permalink / raw)
To: Rob Herring
Cc: devicetree, linux-kernel, Stephen Boyd, Shawn Guo, Sascha Hauer,
Anson Huang, linux-clk, linux-pwm, linux-usb
[-- Attachment #1: Type: text/plain, Size: 1170 bytes --]
On Thu, Jun 11, 2020 at 01:47:38PM -0600, Rob Herring wrote:
> There's no need to specify 'maxItems' with the same value as the number
> of entries in 'items'. A meta-schema update will catch future cases.
>
> Cc: Stephen Boyd <sboyd@kernel.org>
> Cc: Shawn Guo <shawnguo@kernel.org>
> Cc: Sascha Hauer <s.hauer@pengutronix.de>
> Cc: Thierry Reding <thierry.reding@gmail.com>
> Cc: Anson Huang <Anson.Huang@nxp.com>
> Cc: linux-clk@vger.kernel.org
> Cc: linux-pwm@vger.kernel.org
> Cc: linux-usb@vger.kernel.org
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
> Documentation/devicetree/bindings/clock/imx6q-clock.yaml | 1 -
> Documentation/devicetree/bindings/clock/imx6sl-clock.yaml | 1 -
> Documentation/devicetree/bindings/clock/imx6sll-clock.yaml | 1 -
> Documentation/devicetree/bindings/clock/imx6sx-clock.yaml | 1 -
> Documentation/devicetree/bindings/clock/imx6ul-clock.yaml | 1 -
> Documentation/devicetree/bindings/pwm/imx-pwm.yaml | 2 --
> Documentation/devicetree/bindings/usb/nvidia,tegra-xudc.yaml | 2 --
> 7 files changed, 9 deletions(-)
Acked-by: Thierry Reding <thierry.reding@gmail.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] dt-bindings: Remove redundant 'maxItems'
2020-06-11 19:47 [PATCH] dt-bindings: Remove redundant 'maxItems' Rob Herring
2020-06-11 22:35 ` Stephen Boyd
2020-06-12 6:47 ` Thierry Reding
@ 2020-06-26 16:23 ` Uwe Kleine-König
2 siblings, 0 replies; 4+ messages in thread
From: Uwe Kleine-König @ 2020-06-26 16:23 UTC (permalink / raw)
To: Rob Herring
Cc: devicetree, linux-kernel, Stephen Boyd, Shawn Guo, Sascha Hauer,
Thierry Reding, Anson Huang, linux-clk, linux-pwm, linux-usb
[-- Attachment #1: Type: text/plain, Size: 453 bytes --]
On Thu, Jun 11, 2020 at 01:47:38PM -0600, Rob Herring wrote:
> There's no need to specify 'maxItems' with the same value as the number
> of entries in 'items'. A meta-schema update will catch future cases.
Sounds reasonable,
Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | https://www.pengutronix.de/ |
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-06-26 16:23 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-11 19:47 [PATCH] dt-bindings: Remove redundant 'maxItems' Rob Herring
2020-06-11 22:35 ` Stephen Boyd
2020-06-12 6:47 ` Thierry Reding
2020-06-26 16:23 ` Uwe Kleine-König
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).