diff mbox series

[v4,2/3] dt-bindings: pwm: amlogic: Add new bindings for meson A1 PWM

Message ID 20240710234116.2370655-3-gnstark@salutedevices.com (mailing list archive)
State New, archived
Headers show
Series pwm: meson: add pwm support for A1 | expand

Commit Message

George Stark July 10, 2024, 11:41 p.m. UTC
The chip has 3 dual-channel PWM modules PWM_AB, PWM_CD, PWM_EF.

Signed-off-by: George Stark <gnstark@salutedevices.com>
Signed-off-by: Dmitry Rokosov <ddrokosov@salutedevices.com>
---
 .../devicetree/bindings/pwm/pwm-amlogic.yaml       | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

Comments

Rob Herring (Arm) July 12, 2024, 12:52 p.m. UTC | #1
On Thu, Jul 11, 2024 at 02:41:15AM +0300, George Stark wrote:
> The chip has 3 dual-channel PWM modules PWM_AB, PWM_CD, PWM_EF.
> 
> Signed-off-by: George Stark <gnstark@salutedevices.com>
> Signed-off-by: Dmitry Rokosov <ddrokosov@salutedevices.com>

Missing ack from Conor. When you submit new versions, it is your 
responsibility to add tags.

> ---
>  .../devicetree/bindings/pwm/pwm-amlogic.yaml       | 14 ++++++++++++++
>  1 file changed, 14 insertions(+)
George Stark July 12, 2024, 3:12 p.m. UTC | #2
Hello Rob, Conor

On 7/12/24 15:52, Rob Herring wrote:
> On Thu, Jul 11, 2024 at 02:41:15AM +0300, George Stark wrote:
>> The chip has 3 dual-channel PWM modules PWM_AB, PWM_CD, PWM_EF.
>>
>> Signed-off-by: George Stark <gnstark@salutedevices.com>
>> Signed-off-by: Dmitry Rokosov <ddrokosov@salutedevices.com>
> 
> Missing ack from Conor. When you submit new versions, it is your
> responsibility to add tags.

I had Conor's ack in my mind but his response was related to the
squashed patch (a1 compatible + power domains) and the current patch was
a bit different that's why I didn't dare to add the ack.

Conor, do you mind if I resend this patch (and may be [PATCH v4 1/3])
with your ack?

> 
>> ---
>>   .../devicetree/bindings/pwm/pwm-amlogic.yaml       | 14 ++++++++++++++
>>   1 file changed, 14 insertions(+)
Conor Dooley July 25, 2024, 2:08 p.m. UTC | #3
On Fri, Jul 12, 2024 at 06:12:26PM +0300, George Stark wrote:
> Hello Rob, Conor
> 
> On 7/12/24 15:52, Rob Herring wrote:
> > On Thu, Jul 11, 2024 at 02:41:15AM +0300, George Stark wrote:
> > > The chip has 3 dual-channel PWM modules PWM_AB, PWM_CD, PWM_EF.
> > > 
> > > Signed-off-by: George Stark <gnstark@salutedevices.com>
> > > Signed-off-by: Dmitry Rokosov <ddrokosov@salutedevices.com>
> > 
> > Missing ack from Conor. When you submit new versions, it is your
> > responsibility to add tags.
> 
> I had Conor's ack in my mind but his response was related to the
> squashed patch (a1 compatible + power domains) and the current patch was
> a bit different that's why I didn't dare to add the ack.
> 
> Conor, do you mind if I resend this patch (and may be [PATCH v4 1/3])
> with your ack?

Ye, both are fine :+1:
Uwe Kleine-König Sept. 10, 2024, 8:01 a.m. UTC | #4
On Thu, Jul 25, 2024 at 03:08:53PM +0100, Conor Dooley wrote:
> On Fri, Jul 12, 2024 at 06:12:26PM +0300, George Stark wrote:
> > Hello Rob, Conor
> > 
> > On 7/12/24 15:52, Rob Herring wrote:
> > > On Thu, Jul 11, 2024 at 02:41:15AM +0300, George Stark wrote:
> > > > The chip has 3 dual-channel PWM modules PWM_AB, PWM_CD, PWM_EF.
> > > > 
> > > > Signed-off-by: George Stark <gnstark@salutedevices.com>
> > > > Signed-off-by: Dmitry Rokosov <ddrokosov@salutedevices.com>
> > > 
> > > Missing ack from Conor. When you submit new versions, it is your
> > > responsibility to add tags.
> > 
> > I had Conor's ack in my mind but his response was related to the
> > squashed patch (a1 compatible + power domains) and the current patch was
> > a bit different that's why I didn't dare to add the ack.
> > 
> > Conor, do you mind if I resend this patch (and may be [PATCH v4 1/3])
> > with your ack?
> 
> Ye, both are fine :+1:

I interpreted that as an ack for patches 1 and 2 and applied these to
https://git.kernel.org/pub/scm/linux/kernel/git/ukleinek/linux.git pwm/for-next
. I guess that interpretation is fine, please tell me if not.

Best regards
Uwe
Conor Dooley Sept. 10, 2024, 4:08 p.m. UTC | #5
On Tue, Sep 10, 2024 at 10:01:39AM +0200, Uwe Kleine-König wrote:
> On Thu, Jul 25, 2024 at 03:08:53PM +0100, Conor Dooley wrote:
> > On Fri, Jul 12, 2024 at 06:12:26PM +0300, George Stark wrote:
> > > Hello Rob, Conor
> > > 
> > > On 7/12/24 15:52, Rob Herring wrote:
> > > > On Thu, Jul 11, 2024 at 02:41:15AM +0300, George Stark wrote:
> > > > > The chip has 3 dual-channel PWM modules PWM_AB, PWM_CD, PWM_EF.
> > > > > 
> > > > > Signed-off-by: George Stark <gnstark@salutedevices.com>
> > > > > Signed-off-by: Dmitry Rokosov <ddrokosov@salutedevices.com>
> > > > 
> > > > Missing ack from Conor. When you submit new versions, it is your
> > > > responsibility to add tags.
> > > 
> > > I had Conor's ack in my mind but his response was related to the
> > > squashed patch (a1 compatible + power domains) and the current patch was
> > > a bit different that's why I didn't dare to add the ack.
> > > 
> > > Conor, do you mind if I resend this patch (and may be [PATCH v4 1/3])
> > > with your ack?
> > 
> > Ye, both are fine :+1:
> 
> I interpreted that as an ack for patches 1 and 2 and applied these to
> https://git.kernel.org/pub/scm/linux/kernel/git/ukleinek/linux.git pwm/for-next
> . I guess that interpretation is fine, please tell me if not.

Yah, I was okaying my ack being added to both patches in a resend.
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/pwm/pwm-amlogic.yaml b/Documentation/devicetree/bindings/pwm/pwm-amlogic.yaml
index 267908c2bf7b..e021cf59421a 100644
--- a/Documentation/devicetree/bindings/pwm/pwm-amlogic.yaml
+++ b/Documentation/devicetree/bindings/pwm/pwm-amlogic.yaml
@@ -37,6 +37,10 @@  properties:
       - enum:
           - amlogic,meson8-pwm-v2
           - amlogic,meson-s4-pwm
+      - items:
+          - enum:
+              - amlogic,meson-a1-pwm
+          - const: amlogic,meson-s4-pwm
       - items:
           - enum:
               - amlogic,meson8b-pwm-v2
@@ -139,6 +143,16 @@  allOf:
       required:
         - clocks
 
+  - if:
+      properties:
+        compatible:
+          contains:
+            enum:
+              - amlogic,meson-a1-pwm
+    then:
+      required:
+        - power-domains
+
 additionalProperties: false
 
 examples: