diff mbox series

dt-bindings: timer: Add power-domains for TI timer-dm on K3

Message ID 20220919112357.64997-1-tony@atomide.com (mailing list archive)
State New, archived
Headers show
Series dt-bindings: timer: Add power-domains for TI timer-dm on K3 | expand

Commit Message

Tony Lindgren Sept. 19, 2022, 11:23 a.m. UTC
On K3 SoCs, the power-domains property is needed. On the earlier SoCs,
the power-domains property is handled by the interconnect target module
parent device.

Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
Cc: Grygorii Strashko <grygorii.strashko@ti.com>
Cc: Keerthy <j-keerthy@ti.com>
Cc: Nishanth Menon <nm@ti.com>
Cc: Vignesh Raghavendra <vigneshr@ti.com>
Signed-off-by: Tony Lindgren <tony@atomide.com>
---
 .../devicetree/bindings/timer/ti,timer-dm.yaml  | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

Comments

Krzysztof Kozlowski Sept. 19, 2022, 11:35 a.m. UTC | #1
On 19/09/2022 13:23, Tony Lindgren wrote:
> On K3 SoCs, the power-domains property is needed. On the earlier SoCs,
> the power-domains property is handled by the interconnect target module
> parent device.

Rebase your patch on recent Linux kernel and use get_maintainers.pl.

> 
> Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
> Cc: Grygorii Strashko <grygorii.strashko@ti.com>
> Cc: Keerthy <j-keerthy@ti.com>
> Cc: Nishanth Menon <nm@ti.com>
> Cc: Vignesh Raghavendra <vigneshr@ti.com>
> Signed-off-by: Tony Lindgren <tony@atomide.com>


Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>


Best regards,
Krzysztof
Tony Lindgren Sept. 19, 2022, 11:48 a.m. UTC | #2
* Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> [220919 11:27]:
> On 19/09/2022 13:23, Tony Lindgren wrote:
> > On K3 SoCs, the power-domains property is needed. On the earlier SoCs,
> > the power-domains property is handled by the interconnect target module
> > parent device.
> 
> Rebase your patch on recent Linux kernel and use get_maintainers.pl.

OK sorry about your old email address, noted :)

> > Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
> > Cc: Grygorii Strashko <grygorii.strashko@ti.com>
> > Cc: Keerthy <j-keerthy@ti.com>
> > Cc: Nishanth Menon <nm@ti.com>
> > Cc: Vignesh Raghavendra <vigneshr@ti.com>
> > Signed-off-by: Tony Lindgren <tony@atomide.com>
> 
> 
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Thanks,

Tony
Rob Herring (Arm) Sept. 29, 2022, 7 p.m. UTC | #3
On Mon, 19 Sep 2022 14:23:57 +0300, Tony Lindgren wrote:
> On K3 SoCs, the power-domains property is needed. On the earlier SoCs,
> the power-domains property is handled by the interconnect target module
> parent device.
> 
> Cc: Daniel Lezcano <daniel.lezcano@linaro.org>
> Cc: Grygorii Strashko <grygorii.strashko@ti.com>
> Cc: Keerthy <j-keerthy@ti.com>
> Cc: Nishanth Menon <nm@ti.com>
> Cc: Vignesh Raghavendra <vigneshr@ti.com>
> Signed-off-by: Tony Lindgren <tony@atomide.com>
> ---
>  .../devicetree/bindings/timer/ti,timer-dm.yaml  | 17 ++++++++++++-----
>  1 file changed, 12 insertions(+), 5 deletions(-)
> 

Applied, thanks!
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/timer/ti,timer-dm.yaml b/Documentation/devicetree/bindings/timer/ti,timer-dm.yaml
--- a/Documentation/devicetree/bindings/timer/ti,timer-dm.yaml
+++ b/Documentation/devicetree/bindings/timer/ti,timer-dm.yaml
@@ -51,6 +51,11 @@  properties:
       - const: timer_sys_ck
     minItems: 1
 
+  power-domains:
+    description:
+      Power domain if available
+    maxItems: 1
+
   interrupts:
     description:
       Interrupt if available. The timer PWM features may be usable
@@ -94,12 +99,14 @@  additionalProperties: false
 
 allOf:
   - if:
-      not:
-        properties:
-          compatible:
-            contains:
-              const: ti,am654-timer
+      properties:
+        compatible:
+          contains:
+            const: ti,am654-timer
     then:
+      required:
+        - power-domains
+    else:
       required:
         - interrupts