dt-bindings: watchdog: Add optional power-domains property

[PATCH] dt-bindings: watchdog: Add optional power-domains property

Posted by Harini T 1 year ago

AMD/Xilinx Versal Window Watchdog Timer has its own power domain, so add
an optional property to describe it.

Signed-off-by: Harini T <harini.t@amd.com>
---
 .../devicetree/bindings/watchdog/xlnx,versal-wwdt.yaml         | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/Documentation/devicetree/bindings/watchdog/xlnx,versal-wwdt.yaml b/Documentation/devicetree/bindings/watchdog/xlnx,versal-wwdt.yaml
index 14b069599740..fccfc785a077 100644
--- a/Documentation/devicetree/bindings/watchdog/xlnx,versal-wwdt.yaml
+++ b/Documentation/devicetree/bindings/watchdog/xlnx,versal-wwdt.yaml
@@ -32,6 +32,9 @@ properties:
   clocks:
     maxItems: 1
 
+  power-domains:
+    maxItems: 1
+
 required:
   - compatible
   - reg
-- 
2.34.1

Re: [PATCH] dt-bindings: watchdog: Add optional power-domains property

Posted by Rob Herring 1 year ago

On Wed, Jan 29, 2025 at 09:40:14PM +0530, Harini T wrote:
> AMD/Xilinx Versal Window Watchdog Timer has its own power domain, so add
> an optional property to describe it.

Please fix your subject to indicate this is for xlnx,versal-wwdt, not 
*every* watchdog.

> 
> Signed-off-by: Harini T <harini.t@amd.com>
> ---
>  .../devicetree/bindings/watchdog/xlnx,versal-wwdt.yaml         | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/watchdog/xlnx,versal-wwdt.yaml b/Documentation/devicetree/bindings/watchdog/xlnx,versal-wwdt.yaml
> index 14b069599740..fccfc785a077 100644
> --- a/Documentation/devicetree/bindings/watchdog/xlnx,versal-wwdt.yaml
> +++ b/Documentation/devicetree/bindings/watchdog/xlnx,versal-wwdt.yaml
> @@ -32,6 +32,9 @@ properties:
>    clocks:
>      maxItems: 1
>  
> +  power-domains:
> +    maxItems: 1
> +
>  required:
>    - compatible
>    - reg
> -- 
> 2.34.1
>