From b1604b744b716a592841757c43bc5156c7772aca Mon Sep 17 00:00:00 2001
From: =?utf8?q?=C3=81lvaro=20Fern=C3=A1ndez=20Rojas?= <noltari@gmail.com>
Date: Sun, 24 May 2020 20:19:21 +0200
Subject: [PATCH] bcm63xx: ar-5315u: expose LEDs through controller
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

Exposing LEDs through LED controller instead of gpio-leds takes
advantage of HW blinking.

Signed-off-by: Álvaro Fernández Rojas <noltari@gmail.com>
---
 .../bcm63xx/dts/bcm6318-comtrend-ar-5315u.dts | 122 ++++++++++++------
 1 file changed, 86 insertions(+), 36 deletions(-)

diff --git a/target/linux/bcm63xx/dts/bcm6318-comtrend-ar-5315u.dts b/target/linux/bcm63xx/dts/bcm6318-comtrend-ar-5315u.dts
index 000ff99767..be10274625 100644
--- a/target/linux/bcm63xx/dts/bcm6318-comtrend-ar-5315u.dts
+++ b/target/linux/bcm63xx/dts/bcm6318-comtrend-ar-5315u.dts
@@ -40,40 +40,6 @@
 			debounce-interval = <60>;
 		};
 	};
-
-	leds {
-		compatible = "gpio-leds";
-
-		wps_green {
-			label = "AR-5315u:green:wps";
-			gpios = <&pinctrl 0 1>;
-		};
-		led_power_green: power_green {
-			label = "AR-5315u:green:power";
-			gpios = <&pinctrl 1 1>;
-			default-state = "on";
-		};
-		usb_green {
-			label = "AR-5315u:green:usb";
-			gpios = <&pinctrl 2 1>;
-		};
-		inet_green {
-			label = "AR-5315u:green:inet";
-			gpios = <&pinctrl 8 1>;
-		};
-		inet_red {
-			label = "AR-5315u:red:inet";
-			gpios = <&pinctrl 9 1>;
-		};
-		dsl_green {
-			label = "AR-5315u:green:dsl";
-			gpios = <&pinctrl 10 1>;
-		};
-		power_red {
-			label = "AR-5315u:red:power";
-			gpios = <&pinctrl 11 1>;
-		};
-	};
 };
 
 &hsspi {
@@ -114,10 +80,94 @@
 	};
 };
 
-&pinctrl {
+&leds {
+	status = "okay";
+
 	pinctrl-names = "default";
-	pinctrl-0 = <&pinctrl_ephy0_act_led &pinctrl_ephy1_act_led
+	pinctrl-0 = <&pinctrl_leds
+		     &pinctrl_ephy0_act_led &pinctrl_ephy1_act_led
 		     &pinctrl_ephy2_act_led &pinctrl_ephy3_act_led>;
+
+	led@0 {
+		reg = <0>;
+		active-low;
+		label = "AR-5315u:green:wps";
+	};
+
+	led_power_green: led@1 {
+		reg = <1>;
+		active-low;
+		label = "AR-5315u:green:power";
+		default-state = "on";
+	};
+
+	led@2 {
+		reg = <2>;
+		active-low;
+		label = "AR-5315u:green:usb";
+	};
+
+	led@4 {
+		reg = <4>;
+		brcm,hardware-controlled;
+		brcm,link-signal-sources = <4>;
+		/* EPHY0 Act */
+	};
+
+	led@5 {
+		reg = <5>;
+		brcm,hardware-controlled;
+		brcm,link-signal-sources = <5>;
+		/* EPHY1 Act */
+	};
+
+	led@6 {
+		reg = <6>;
+		brcm,hardware-controlled;
+		brcm,link-signal-sources = <6>;
+		/* EPHY2 Act */
+	};
+
+	led@7 {
+		reg = <7>;
+		brcm,hardware-controlled;
+		brcm,link-signal-sources = <7>;
+		/* EPHY3 Act */
+	};
+
+	led@8 {
+		reg = <8>;
+		active-low;
+		label = "AR-5315u:green:inet";
+	};
+
+	led@9 {
+		reg = <9>;
+		active-low;
+		label = "AR-5315u:red:inet";
+	};
+
+	led@10 {
+		reg = <10>;
+		active-low;
+		label = "AR-5315u:green:dsl";
+	};
+
+	led@11 {
+		reg = <11>;
+		active-low;
+		label = "AR-5315u:red:power";
+	};
+};
+
+&pinctrl {
+	pinctrl_leds: leds {
+		function = "led";
+		pins = "gpio0", "gpio1",
+		       "gpio2", "gpio8",
+		       "gpio9", "gpio10",
+		       "gpio11";
+	};
 };
 
 &uart0 {
-- 
2.25.1