2022-04-22 15:57:42

by Fabio Baltieri

[permalink] [raw]
Subject: [PATCH v5 3/4] dt-bindings: update google,cros-ec-pwm documentation

Update google,cros-ec-pwm node documentation to mention the
google,cros-ec-pwm-type compatible.

Signed-off-by: Fabio Baltieri <[email protected]>
---
.../devicetree/bindings/pwm/google,cros-ec-pwm.yaml | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/pwm/google,cros-ec-pwm.yaml b/Documentation/devicetree/bindings/pwm/google,cros-ec-pwm.yaml
index 7ab6912a845f..c8577bdf6c94 100644
--- a/Documentation/devicetree/bindings/pwm/google,cros-ec-pwm.yaml
+++ b/Documentation/devicetree/bindings/pwm/google,cros-ec-pwm.yaml
@@ -21,7 +21,14 @@ allOf:

properties:
compatible:
- const: google,cros-ec-pwm
+ oneOf:
+ - description: PWM controlled using EC_PWM_TYPE_GENERIC channels.
+ items:
+ - const: google,cros-ec-pwm
+ - description: PWM controlled using CROS_EC_PWM_DT_<...> types.
+ items:
+ - const: google,cros-ec-pwm-type
+
"#pwm-cells":
description: The cell specifies the PWM index.
const: 1
--
2.36.0.rc0.470.gd361397f0d-goog


2022-04-26 22:21:43

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH v5 3/4] dt-bindings: update google,cros-ec-pwm documentation

On Wed, Apr 20, 2022 at 02:15:55PM +0000, Fabio Baltieri wrote:
> Update google,cros-ec-pwm node documentation to mention the
> google,cros-ec-pwm-type compatible.

It would be good if the subject provided some clue what the update is.
Every change is an update. And all bindings are 'documentation' so
that can be dropped.

>
> Signed-off-by: Fabio Baltieri <[email protected]>
> ---
> .../devicetree/bindings/pwm/google,cros-ec-pwm.yaml | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)

In any case,

Reviewed-by: Rob Herring <[email protected]>

2022-04-27 11:30:56

by Fabio Baltieri

[permalink] [raw]
Subject: Re: [PATCH v5 3/4] dt-bindings: update google,cros-ec-pwm documentation

On Mon, Apr 25, 2022 at 04:58:55PM -0500, Rob Herring wrote:
> On Wed, Apr 20, 2022 at 02:15:55PM +0000, Fabio Baltieri wrote:
> > Update google,cros-ec-pwm node documentation to mention the
> > google,cros-ec-pwm-type compatible.
>
> It would be good if the subject provided some clue what the update is.
> Every change is an update. And all bindings are 'documentation' so
> that can be dropped.

Fair enough, I'll reword the subject and send a v6.

>
> >
> > Signed-off-by: Fabio Baltieri <[email protected]>
> > ---
> > .../devicetree/bindings/pwm/google,cros-ec-pwm.yaml | 9 ++++++++-
> > 1 file changed, 8 insertions(+), 1 deletion(-)
>
> In any case,
>
> Reviewed-by: Rob Herring <[email protected]>
>

Thanks!

--
Fabio Baltieri