* [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.