diff mbox

ARM: dts: da850-lcdk: enable pwm nodes

Message ID 1484586189-3118-1-git-send-email-fisaksen@baylibre.com
State Not Applicable
Headers show

Commit Message

Frode Isaksen Jan. 16, 2017, 5:03 p.m. UTC
Enable the ecap1 and ecap2 pwm nodes for the
lcdk board.
Note that the other pwm nodes are not enabled,
since the pins are shared with ethernet and spi.

Signed-off-by: Frode Isaksen <fisaksen@baylibre.com>
---
 arch/arm/boot/dts/da850-lcdk.dts | 12 ++++++++++++
 1 file changed, 12 insertions(+)

Comments

Sekhar Nori Jan. 20, 2017, 12:53 p.m. UTC | #1
Hi Frode,

On Monday 16 January 2017 10:33 PM, Frode Isaksen wrote:
> Enable the ecap1 and ecap2 pwm nodes for the
> lcdk board.
> Note that the other pwm nodes are not enabled,
> since the pins are shared with ethernet and spi.
> 
> Signed-off-by: Frode Isaksen <fisaksen@baylibre.com>
> ---

Looks like the eCAP is available on the headers at the bottom of the
board. While patch like this is fine to be able to test the PWM
functionality, I dont think we should apply it because this can
potentially clash with usage of the pin on daughter card.

Thanks,
Sekhar

--
To unsubscribe from this list: send the line "unsubscribe linux-pwm" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/arm/boot/dts/da850-lcdk.dts b/arch/arm/boot/dts/da850-lcdk.dts
index 60e027e..6f4c367 100644
--- a/arch/arm/boot/dts/da850-lcdk.dts
+++ b/arch/arm/boot/dts/da850-lcdk.dts
@@ -322,3 +322,15 @@ 
 		};
 	};
 };
+
+&ecap1 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&ecap1_pins>;
+	status = "okay";
+};
+
+&ecap2 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&ecap2_pins>;
+	status = "okay";
+};