All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] dt-bindings: watchdog: ti,rti-wdt: Fix assigned-clock-parents
@ 2021-12-06 17:40 Rob Herring
  2021-12-06 19:47 ` Guenter Roeck
  2021-12-07  8:23 ` Thierry Reding
  0 siblings, 2 replies; 3+ messages in thread
From: Rob Herring @ 2021-12-06 17:40 UTC (permalink / raw)
  To: Wim Van Sebroeck, Guenter Roeck, Tero Kristo
  Cc: devicetree, Thierry Reding, linux-watchdog, linux-kernel

With 'unevaluatedProperties' support implemented, the ti,rti-wdt example
has the following warning:

/home/rob/proj/git/linux-dt/.build-arm64/Documentation/devicetree/bindings/watchdog/ti,rti-wdt.example.dt.yaml: watchdog@2200000: Unevaluated properties are not allowed ('assigned-clock-parents' was unexpected)

The problem is the schema has a typo in 'assigned-clocks-parents'. As
it is not required to list assigned clocks in bindings, just drop the
property definitions to fix this.

Cc: Wim Van Sebroeck <wim@linux-watchdog.org>
Cc: Guenter Roeck <linux@roeck-us.net>
Cc: Tero Kristo <t-kristo@ti.com>
Cc: linux-watchdog@vger.kernel.org
Signed-off-by: Rob Herring <robh@kernel.org>
---
 Documentation/devicetree/bindings/watchdog/ti,rti-wdt.yaml | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/Documentation/devicetree/bindings/watchdog/ti,rti-wdt.yaml b/Documentation/devicetree/bindings/watchdog/ti,rti-wdt.yaml
index 054584d7543a..2f33635876ff 100644
--- a/Documentation/devicetree/bindings/watchdog/ti,rti-wdt.yaml
+++ b/Documentation/devicetree/bindings/watchdog/ti,rti-wdt.yaml
@@ -34,12 +34,6 @@ properties:
   power-domains:
     maxItems: 1
 
-  assigned-clocks:
-    maxItems: 1
-
-  assigned-clocks-parents:
-    maxItems: 1
-
 required:
   - compatible
   - reg
-- 
2.32.0


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

* Re: [PATCH] dt-bindings: watchdog: ti,rti-wdt: Fix assigned-clock-parents
  2021-12-06 17:40 [PATCH] dt-bindings: watchdog: ti,rti-wdt: Fix assigned-clock-parents Rob Herring
@ 2021-12-06 19:47 ` Guenter Roeck
  2021-12-07  8:23 ` Thierry Reding
  1 sibling, 0 replies; 3+ messages in thread
From: Guenter Roeck @ 2021-12-06 19:47 UTC (permalink / raw)
  To: Rob Herring
  Cc: Wim Van Sebroeck, Tero Kristo, devicetree, Thierry Reding,
	linux-watchdog, linux-kernel

On Mon, Dec 06, 2021 at 11:40:28AM -0600, Rob Herring wrote:
> With 'unevaluatedProperties' support implemented, the ti,rti-wdt example
> has the following warning:
> 
> /home/rob/proj/git/linux-dt/.build-arm64/Documentation/devicetree/bindings/watchdog/ti,rti-wdt.example.dt.yaml: watchdog@2200000: Unevaluated properties are not allowed ('assigned-clock-parents' was unexpected)
> 
> The problem is the schema has a typo in 'assigned-clocks-parents'. As
> it is not required to list assigned clocks in bindings, just drop the
> property definitions to fix this.
> 
> Cc: Wim Van Sebroeck <wim@linux-watchdog.org>
> Cc: Guenter Roeck <linux@roeck-us.net>
> Cc: Tero Kristo <t-kristo@ti.com>
> Cc: linux-watchdog@vger.kernel.org
> Signed-off-by: Rob Herring <robh@kernel.org>

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

... assuming you want to carry this and the following patch through
your tree. If not, please let me know.

Thanks,
Guenter

> ---
>  Documentation/devicetree/bindings/watchdog/ti,rti-wdt.yaml | 6 ------
>  1 file changed, 6 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/watchdog/ti,rti-wdt.yaml b/Documentation/devicetree/bindings/watchdog/ti,rti-wdt.yaml
> index 054584d7543a..2f33635876ff 100644
> --- a/Documentation/devicetree/bindings/watchdog/ti,rti-wdt.yaml
> +++ b/Documentation/devicetree/bindings/watchdog/ti,rti-wdt.yaml
> @@ -34,12 +34,6 @@ properties:
>    power-domains:
>      maxItems: 1
>  
> -  assigned-clocks:
> -    maxItems: 1
> -
> -  assigned-clocks-parents:
> -    maxItems: 1
> -
>  required:
>    - compatible
>    - reg
> -- 
> 2.32.0
> 

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

* Re: [PATCH] dt-bindings: watchdog: ti,rti-wdt: Fix assigned-clock-parents
  2021-12-06 17:40 [PATCH] dt-bindings: watchdog: ti,rti-wdt: Fix assigned-clock-parents Rob Herring
  2021-12-06 19:47 ` Guenter Roeck
@ 2021-12-07  8:23 ` Thierry Reding
  1 sibling, 0 replies; 3+ messages in thread
From: Thierry Reding @ 2021-12-07  8:23 UTC (permalink / raw)
  To: Rob Herring
  Cc: Wim Van Sebroeck, Guenter Roeck, Tero Kristo, devicetree,
	linux-watchdog, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 959 bytes --]

On Mon, Dec 06, 2021 at 11:40:28AM -0600, Rob Herring wrote:
> With 'unevaluatedProperties' support implemented, the ti,rti-wdt example
> has the following warning:
> 
> /home/rob/proj/git/linux-dt/.build-arm64/Documentation/devicetree/bindings/watchdog/ti,rti-wdt.example.dt.yaml: watchdog@2200000: Unevaluated properties are not allowed ('assigned-clock-parents' was unexpected)
> 
> The problem is the schema has a typo in 'assigned-clocks-parents'. As
> it is not required to list assigned clocks in bindings, just drop the
> property definitions to fix this.
> 
> Cc: Wim Van Sebroeck <wim@linux-watchdog.org>
> Cc: Guenter Roeck <linux@roeck-us.net>
> Cc: Tero Kristo <t-kristo@ti.com>
> Cc: linux-watchdog@vger.kernel.org
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
>  Documentation/devicetree/bindings/watchdog/ti,rti-wdt.yaml | 6 ------
>  1 file changed, 6 deletions(-)

Reviewed-by: Thierry Reding <treding@nvidia.com>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

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

end of thread, other threads:[~2021-12-07  8:23 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-06 17:40 [PATCH] dt-bindings: watchdog: ti,rti-wdt: Fix assigned-clock-parents Rob Herring
2021-12-06 19:47 ` Guenter Roeck
2021-12-07  8:23 ` Thierry Reding

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.