Linux-Clk Archive on lore.kernel.org
 help / color / Atom feed
* [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	[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, back to index

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

Linux-Clk Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-clk/0 linux-clk/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-clk linux-clk/ https://lore.kernel.org/linux-clk \
		linux-clk@vger.kernel.org
	public-inbox-index linux-clk

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-clk


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git