2022-05-02 01:23:01

by Pali Rohár

Subject: [PATCH v2 1/2] dt-bindings: watchdog: max63xx: Add GPIO binding

GPIO is optional and used for WDI logic.

Signed-off-by: Pali Rohár <[email protected]>
---
Documentation/devicetree/bindings/watchdog/maxim,max63xx.yaml | 4 ++++
1 file changed, 4 insertions(+)

diff --git a/Documentation/devicetree/bindings/watchdog/maxim,max63xx.yaml b/Documentation/devicetree/bindings/watchdog/maxim,max63xx.yaml
index ab9641e845db..a97aa0135ef9 100644
--- a/Documentation/devicetree/bindings/watchdog/maxim,max63xx.yaml
+++ b/Documentation/devicetree/bindings/watchdog/maxim,max63xx.yaml
@@ -27,6 +27,10 @@ properties:
description: This is a 1-byte memory-mapped address
maxItems: 1

+ gpios:
+ description: Optional GPIO used for controlling WDI when WDI bit is not mapped to memory
+ maxItems: 1
+
required:
- compatible
- reg
--
2.20.1

2022-05-04 05:09:49

by Rob Herring (Arm)

Subject: Re: [PATCH v2 1/2] dt-bindings: watchdog: max63xx: Add GPIO binding

On Fri, Apr 29, 2022 at 03:13:48PM +0200, Pali Roh?r wrote:
> GPIO is optional and used for WDI logic.

Nowhere is WDI defined.

>
> Signed-off-by: Pali Roh?r <[email protected]>
> ---
> Documentation/devicetree/bindings/watchdog/maxim,max63xx.yaml | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/watchdog/maxim,max63xx.yaml b/Documentation/devicetree/bindings/watchdog/maxim,max63xx.yaml
> index ab9641e845db..a97aa0135ef9 100644
> --- a/Documentation/devicetree/bindings/watchdog/maxim,max63xx.yaml
> +++ b/Documentation/devicetree/bindings/watchdog/maxim,max63xx.yaml
> @@ -27,6 +27,10 @@ properties:
> description: This is a 1-byte memory-mapped address
> maxItems: 1
>
> + gpios:

Usually, we want a name here. Maybe wdi-gpios, but I don't know what WDI
is nor have I read the pin name in the datasheet for inspiration.

> + description: Optional GPIO used for controlling WDI when WDI bit is not mapped to memory
> + maxItems: 1
> +
> required:
> - compatible
> - reg
> --
> 2.20.1
>
>

2022-05-04 08:11:56

by Pali Rohár

Subject: Re: [PATCH v2 1/2] dt-bindings: watchdog: max63xx: Add GPIO binding

On Tuesday 03 May 2022 16:51:37 Rob Herring wrote:
> On Fri, Apr 29, 2022 at 03:13:48PM +0200, Pali Rohár wrote:
> > GPIO is optional and used for WDI logic.
>
> Nowhere is WDI defined.
>
> >
> > Signed-off-by: Pali Rohár <[email protected]>
> > ---
> > Documentation/devicetree/bindings/watchdog/maxim,max63xx.yaml | 4 ++++
> > 1 file changed, 4 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/watchdog/maxim,max63xx.yaml b/Documentation/devicetree/bindings/watchdog/maxim,max63xx.yaml
> > index ab9641e845db..a97aa0135ef9 100644
> > --- a/Documentation/devicetree/bindings/watchdog/maxim,max63xx.yaml
> > +++ b/Documentation/devicetree/bindings/watchdog/maxim,max63xx.yaml
> > @@ -27,6 +27,10 @@ properties:
> > description: This is a 1-byte memory-mapped address
> > maxItems: 1
> >
> > + gpios:
>
> Usually, we want a name here. Maybe wdi-gpios, but I don't know what WDI
> is nor have I read the pin name in the datasheet for inspiration.

WDI is name of logic used in the datasheet, it is abbreviation of
WatchDog Input (meaning that from watchdog chip this GPIO has input
direction).

I'm not sure if we need to put gpio direction into the property name or
also word watchdog (or its some abbrev) into name. As node is already
named "watchdog" and direction depends on point of view (chip vs CPU),
which can be in DTS misleading (because DTS describe direction from CPU
point of view).

What for sure makes sense is extending description by explaining WDI
abbreviation.

> > + description: Optional GPIO used for controlling WDI when WDI bit is not mapped to memory
> > + maxItems: 1
> > +
> > required:
> > - compatible
> > - reg
> > --
> > 2.20.1
> >
> >