diff mbox

[1/2] ARM: versatile: add the syscon LEDs to the DT

Message ID 1452002529-3335-2-git-send-email-linus.walleij@linaro.org
State Accepted
Commit 07ebfa59b9db40ae29fbf2ef24ba7dc5fa8a5dad
Headers show

Commit Message

Linus Walleij Jan. 5, 2016, 2:02 p.m. UTC
The device tree version of Versatile AP/PB never had LED support
so we are missing LEDs from our hardware boards. Add this as
syscon LEDs like we did for Integrator and Juno. We need to
spawn devices in the syscon with "simple-mfd" for this to work.

Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

---
 arch/arm/boot/dts/versatile-ab.dts | 62 +++++++++++++++++++++++++++++++++++++-
 1 file changed, 61 insertions(+), 1 deletion(-)

-- 
2.4.3


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Comments

Rob Herring Jan. 5, 2016, 3:37 p.m. UTC | #1
On Tue, Jan 5, 2016 at 8:02 AM, Linus Walleij <linus.walleij@linaro.org> wrote:
> The device tree version of Versatile AP/PB never had LED support

> so we are missing LEDs from our hardware boards. Add this as

> syscon LEDs like we did for Integrator and Juno. We need to

> spawn devices in the syscon with "simple-mfd" for this to work.

>

> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

> ---

>  arch/arm/boot/dts/versatile-ab.dts | 62 +++++++++++++++++++++++++++++++++++++-

>  1 file changed, 61 insertions(+), 1 deletion(-)


Acked-by: Rob Herring <robh@kernel.org>


I guess this means you have tested my series on actual h/w now?

Rob

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Linus Walleij Jan. 5, 2016, 11:24 p.m. UTC | #2
On Tue, Jan 5, 2016 at 4:37 PM, Rob Herring <robh@kernel.org> wrote:
> On Tue, Jan 5, 2016 at 8:02 AM, Linus Walleij <linus.walleij@linaro.org> wrote:

>> The device tree version of Versatile AP/PB never had LED support

>> so we are missing LEDs from our hardware boards. Add this as

>> syscon LEDs like we did for Integrator and Juno. We need to

>> spawn devices in the syscon with "simple-mfd" for this to work.

>>

>> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

>> ---

>>  arch/arm/boot/dts/versatile-ab.dts | 62 +++++++++++++++++++++++++++++++++++++-

>>  1 file changed, 61 insertions(+), 1 deletion(-)

>

> Acked-by: Rob Herring <robh@kernel.org>

>

> I guess this means you have tested my series on actual h/w now?


Yups tested today on Versatile AB. I sent some 4 different
patches for various issues (that were there all the time with
the device tree boot) and after that it works like a charm.

Yours,
Linus Walleij

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Arnd Bergmann Jan. 8, 2016, 10:47 a.m. UTC | #3
On Tuesday 05 January 2016 15:02:08 Linus Walleij wrote:
> The device tree version of Versatile AP/PB never had LED support

> so we are missing LEDs from our hardware boards. Add this as

> syscon LEDs like we did for Integrator and Juno. We need to

> spawn devices in the syscon with "simple-mfd" for this to work.

> 

> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

> 


Applied to next/dt, thanks!

	Arnd

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
diff mbox

Patch

diff --git a/arch/arm/boot/dts/versatile-ab.dts b/arch/arm/boot/dts/versatile-ab.dts
index 3279bf1a17a1..6fd7efbead34 100644
--- a/arch/arm/boot/dts/versatile-ab.dts
+++ b/arch/arm/boot/dts/versatile-ab.dts
@@ -30,9 +30,69 @@ 
 	};
 
 	core-module@10000000 {
-		compatible = "arm,core-module-versatile", "syscon";
+		compatible = "arm,core-module-versatile", "syscon", "simple-mfd";
 		reg = <0x10000000 0x200>;
 
+		led@08.0 {
+			compatible = "register-bit-led";
+			offset = <0x08>;
+			mask = <0x01>;
+			label = "versatile:0";
+			linux,default-trigger = "heartbeat";
+			default-state = "on";
+		};
+		led@08.1 {
+			compatible = "register-bit-led";
+			offset = <0x08>;
+			mask = <0x02>;
+			label = "versatile:1";
+			linux,default-trigger = "mmc0";
+			default-state = "off";
+		};
+		led@08.2 {
+			compatible = "register-bit-led";
+			offset = <0x08>;
+			mask = <0x04>;
+			label = "versatile:2";
+			linux,default-trigger = "cpu0";
+			default-state = "off";
+		};
+		led@08.3 {
+			compatible = "register-bit-led";
+			offset = <0x08>;
+			mask = <0x08>;
+			label = "versatile:3";
+			default-state = "off";
+		};
+		led@08.4 {
+			compatible = "register-bit-led";
+			offset = <0x08>;
+			mask = <0x10>;
+			label = "versatile:4";
+			default-state = "off";
+		};
+		led@08.5 {
+			compatible = "register-bit-led";
+			offset = <0x08>;
+			mask = <0x20>;
+			label = "versatile:5";
+			default-state = "off";
+		};
+		led@08.6 {
+			compatible = "register-bit-led";
+			offset = <0x08>;
+			mask = <0x40>;
+			label = "versatile:6";
+			default-state = "off";
+		};
+		led@08.7 {
+			compatible = "register-bit-led";
+			offset = <0x08>;
+			mask = <0x80>;
+			label = "versatile:7";
+			default-state = "off";
+		};
+
 		/* OSC1 on AB, OSC4 on PB */
 		osc1: cm_aux_osc@24M {
 			#clock-cells = <0>;