Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1432069 > unrolled thread

[PATCH] ARM: dts: da850: Add new ECAP and EPWM bindings

Started byFranklin S Cooper Jr <fcooper@ti.com>
First post2016-06-27 16:20 +0200
Last post2016-07-04 14:50 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] ARM: dts: da850: Add new ECAP and EPWM bindings Franklin S Cooper Jr <fcooper@ti.com> - 2016-06-27 16:20 +0200
    Re: [PATCH] ARM: dts: da850: Add new ECAP and EPWM bindings Sekhar Nori <nsekhar@ti.com> - 2016-07-04 14:50 +0200

#1432069 — [PATCH] ARM: dts: da850: Add new ECAP and EPWM bindings

FromFranklin S Cooper Jr <fcooper@ti.com>
Date2016-06-27 16:20 +0200
Subject[PATCH] ARM: dts: da850: Add new ECAP and EPWM bindings
Message-ID<rOFyh-1kx-5@gated-at.bofh.it>
From: "Cooper Jr., Franklin" <fcooper@ti.com>

Switch to a new ECAP and EPWM bindings that doesn't depend on hwmod to
provide the various required clocks.

Signed-off-by: Franklin S Cooper Jr <fcooper@ti.com>
Acked-by: Rob Herring <robh@kernel.org>
---

This patch is a da850 specific version of:
ARM: dts: am437x/am33xx/da850: Add new ECAP and EPWM bindings
https://patchwork.kernel.org/patch/9005661/

This patch was created since the da850 portion didn't apply cleanly in
the original patch.

 arch/arm/boot/dts/da850.dtsi | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/arch/arm/boot/dts/da850.dtsi b/arch/arm/boot/dts/da850.dtsi
index 25f0f8e..09cf663 100644
--- a/arch/arm/boot/dts/da850.dtsi
+++ b/arch/arm/boot/dts/da850.dtsi
@@ -274,31 +274,36 @@
 			status = "disabled";
 		};
 		ehrpwm0: pwm@300000 {
-			compatible = "ti,da850-ehrpwm", "ti,am33xx-ehrpwm";
+			compatible = "ti,da850-ehrpwm", "ti,am3352-ehrpwm",
+				     "ti,am33xx-ehrpwm";
 			#pwm-cells = <3>;
 			reg = <0x300000 0x2000>;
 			status = "disabled";
 		};
 		ehrpwm1: pwm@302000 {
-			compatible = "ti,da850-ehrpwm", "ti,am33xx-ehrpwm";
+			compatible = "ti,da850-ehrpwm", "ti,am3352-ehrpwm",
+				     "ti,am33xx-ehrpwm";
 			#pwm-cells = <3>;
 			reg = <0x302000 0x2000>;
 			status = "disabled";
 		};
 		ecap0: ecap@306000 {
-			compatible = "ti,da850-ecap", "ti,am33xx-ecap";
+			compatible = "ti,da850-ecap", "ti,am3352-ecap",
+				     "ti,am33xx-ecap";
 			#pwm-cells = <3>;
 			reg = <0x306000 0x80>;
 			status = "disabled";
 		};
 		ecap1: ecap@307000 {
-			compatible = "ti,da850-ecap", "ti,am33xx-ecap";
+			compatible = "ti,da850-ecap", "ti,am3352-ecap",
+				     "ti,am33xx-ecap";
 			#pwm-cells = <3>;
 			reg = <0x307000 0x80>;
 			status = "disabled";
 		};
 		ecap2: ecap@308000 {
-			compatible = "ti,da850-ecap", "ti,am33xx-ecap";
+			compatible = "ti,da850-ecap", "ti,am3352-ecap",
+				     "ti,am33xx-ecap";
 			#pwm-cells = <3>;
 			reg = <0x308000 0x80>;
 			status = "disabled";
-- 
2.7.0

[toc] | [next] | [standalone]


#1436468

FromSekhar Nori <nsekhar@ti.com>
Date2016-07-04 14:50 +0200
Message-ID<rRbu2-5EL-27@gated-at.bofh.it>
In reply to#1432069
On Monday 27 June 2016 07:44 PM, Franklin S Cooper Jr wrote:
> From: "Cooper Jr., Franklin" <fcooper@ti.com>
> 
> Switch to a new ECAP and EPWM bindings that doesn't depend on hwmod to
> provide the various required clocks.

there is nothing called hwmod on da850, so this need rewording to make
sense for that platform.

> 
> Signed-off-by: Franklin S Cooper Jr <fcooper@ti.com>
> Acked-by: Rob Herring <robh@kernel.org>

Also, documentation needs update as well since it says:

"
for da850  - compatible = "ti,da850-ehrpwm", "ti,am33xx-ehrpwm"
"

Thanks,
Sekhar

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web