linux-watchdog.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: watchdog: sunxi: fix error in schema
@ 2021-10-29 14:24 David Heidelberg
  2021-11-01 18:52 ` Rob Herring
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: David Heidelberg @ 2021-10-29 14:24 UTC (permalink / raw)
  To: Wim Van Sebroeck, Guenter Roeck, Rob Herring, Maxime Ripard,
	Chen-Yu Tsai, Jernej Skrabec
  Cc: ~okias/devicetree, David Heidelberg, linux-watchdog, devicetree,
	linux-arm-kernel, linux-sunxi, linux-kernel

"maxItems" is not needed with an "items" list

Fixes:
$ DT_SCHEMA_FILES=Documentation/devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml make dtbs_check
Documentation/devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml: properties:clocks: {'required': ['maxItems']} is not allowed for {'minItems': 1, 'maxItems': 2, 'items': [{'description': 'High-frequency oscillator input, divided internally'}, {'description': 'Low-frequency oscillator input, only found on some variants'}]}
	hint: "maxItems" is not needed with an "items" list
	from schema $id: http://devicetree.org/meta-schemas/items.yaml#
...

Signed-off-by: David Heidelberg <david@ixit.cz>
---
 .../devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml   | 2 --
 1 file changed, 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml b/Documentation/devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml
index 44cad9427ae6..43afa24513b9 100644
--- a/Documentation/devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml
+++ b/Documentation/devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml
@@ -40,14 +40,12 @@ properties:
 
   clocks:
     minItems: 1
-    maxItems: 2
     items:
       - description: High-frequency oscillator input, divided internally
       - description: Low-frequency oscillator input, only found on some variants
 
   clock-names:
     minItems: 1
-    maxItems: 2
     items:
       - const: hosc
       - const: losc
-- 
2.33.0


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] dt-bindings: watchdog: sunxi: fix error in schema
  2021-10-29 14:24 [PATCH] dt-bindings: watchdog: sunxi: fix error in schema David Heidelberg
@ 2021-11-01 18:52 ` Rob Herring
  2021-11-09 15:19 ` Guenter Roeck
  2021-11-12  4:19 ` Rob Herring
  2 siblings, 0 replies; 4+ messages in thread
From: Rob Herring @ 2021-11-01 18:52 UTC (permalink / raw)
  To: David Heidelberg
  Cc: Jernej Skrabec, devicetree, Chen-Yu Tsai, Guenter Roeck,
	linux-arm-kernel, linux-sunxi, Maxime Ripard, linux-kernel,
	Rob Herring, Wim Van Sebroeck, linux-watchdog, ~okias/devicetree

On Fri, 29 Oct 2021 16:24:42 +0200, David Heidelberg wrote:
> "maxItems" is not needed with an "items" list
> 
> Fixes:
> $ DT_SCHEMA_FILES=Documentation/devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml make dtbs_check
> Documentation/devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml: properties:clocks: {'required': ['maxItems']} is not allowed for {'minItems': 1, 'maxItems': 2, 'items': [{'description': 'High-frequency oscillator input, divided internally'}, {'description': 'Low-frequency oscillator input, only found on some variants'}]}
> 	hint: "maxItems" is not needed with an "items" list
> 	from schema $id: http://devicetree.org/meta-schemas/items.yaml#
> ...
> 
> Signed-off-by: David Heidelberg <david@ixit.cz>
> ---
>  .../devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml   | 2 --
>  1 file changed, 2 deletions(-)
> 

Acked-by: Rob Herring <robh@kernel.org>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] dt-bindings: watchdog: sunxi: fix error in schema
  2021-10-29 14:24 [PATCH] dt-bindings: watchdog: sunxi: fix error in schema David Heidelberg
  2021-11-01 18:52 ` Rob Herring
@ 2021-11-09 15:19 ` Guenter Roeck
  2021-11-12  4:19 ` Rob Herring
  2 siblings, 0 replies; 4+ messages in thread
From: Guenter Roeck @ 2021-11-09 15:19 UTC (permalink / raw)
  To: David Heidelberg
  Cc: Wim Van Sebroeck, Rob Herring, Maxime Ripard, Chen-Yu Tsai,
	Jernej Skrabec, ~okias/devicetree, linux-watchdog, devicetree,
	linux-arm-kernel, linux-sunxi, linux-kernel

On Fri, Oct 29, 2021 at 04:24:42PM +0200, David Heidelberg wrote:
> "maxItems" is not needed with an "items" list
> 
> Fixes:
> $ DT_SCHEMA_FILES=Documentation/devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml make dtbs_check
> Documentation/devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml: properties:clocks: {'required': ['maxItems']} is not allowed for {'minItems': 1, 'maxItems': 2, 'items': [{'description': 'High-frequency oscillator input, divided internally'}, {'description': 'Low-frequency oscillator input, only found on some variants'}]}
> 	hint: "maxItems" is not needed with an "items" list
> 	from schema $id: http://devicetree.org/meta-schemas/items.yaml#
> ...
> 
> Signed-off-by: David Heidelberg <david@ixit.cz>
> Acked-by: Rob Herring <robh@kernel.org>

Reviewed-by: Guenter Roeck <linux@roeck-us.net>

> ---
>  .../devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml   | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml b/Documentation/devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml
> index 44cad9427ae6..43afa24513b9 100644
> --- a/Documentation/devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml
> +++ b/Documentation/devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml
> @@ -40,14 +40,12 @@ properties:
>  
>    clocks:
>      minItems: 1
> -    maxItems: 2
>      items:
>        - description: High-frequency oscillator input, divided internally
>        - description: Low-frequency oscillator input, only found on some variants
>  
>    clock-names:
>      minItems: 1
> -    maxItems: 2
>      items:
>        - const: hosc
>        - const: losc

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] dt-bindings: watchdog: sunxi: fix error in schema
  2021-10-29 14:24 [PATCH] dt-bindings: watchdog: sunxi: fix error in schema David Heidelberg
  2021-11-01 18:52 ` Rob Herring
  2021-11-09 15:19 ` Guenter Roeck
@ 2021-11-12  4:19 ` Rob Herring
  2 siblings, 0 replies; 4+ messages in thread
From: Rob Herring @ 2021-11-12  4:19 UTC (permalink / raw)
  To: David Heidelberg
  Cc: linux-sunxi, Wim Van Sebroeck, Chen-Yu Tsai, ~okias/devicetree,
	linux-kernel, Guenter Roeck, devicetree, Rob Herring,
	Maxime Ripard, linux-watchdog, Jernej Skrabec, linux-arm-kernel

On Fri, 29 Oct 2021 16:24:42 +0200, David Heidelberg wrote:
> "maxItems" is not needed with an "items" list
> 
> Fixes:
> $ DT_SCHEMA_FILES=Documentation/devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml make dtbs_check
> Documentation/devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml: properties:clocks: {'required': ['maxItems']} is not allowed for {'minItems': 1, 'maxItems': 2, 'items': [{'description': 'High-frequency oscillator input, divided internally'}, {'description': 'Low-frequency oscillator input, only found on some variants'}]}
> 	hint: "maxItems" is not needed with an "items" list
> 	from schema $id: http://devicetree.org/meta-schemas/items.yaml#
> ...
> 
> Signed-off-by: David Heidelberg <david@ixit.cz>
> ---
>  .../devicetree/bindings/watchdog/allwinner,sun4i-a10-wdt.yaml   | 2 --
>  1 file changed, 2 deletions(-)
> 

Applied, thanks!

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2021-11-12  4:19 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-29 14:24 [PATCH] dt-bindings: watchdog: sunxi: fix error in schema David Heidelberg
2021-11-01 18:52 ` Rob Herring
2021-11-09 15:19 ` Guenter Roeck
2021-11-12  4:19 ` Rob Herring

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).