diff mbox series

dt-bindings: watchdog: ti,rti-wdt: Fix assigned-clock-parents

Message ID 20211206174028.2294330-1-robh@kernel.org (mailing list archive)
State Handled Elsewhere
Headers show
Series dt-bindings: watchdog: ti,rti-wdt: Fix assigned-clock-parents | expand

Commit Message

Rob Herring Dec. 6, 2021, 5:40 p.m. UTC
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(-)

Comments

Guenter Roeck Dec. 6, 2021, 7:47 p.m. UTC | #1
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
>
Thierry Reding Dec. 7, 2021, 8:23 a.m. UTC | #2
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>
diff mbox series

Patch

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