ipq40xx: rename RT-AC42U WLAN/LAN LEDs

Assign LED numbers properly by adding function-enumerator property in DTS.

While at it, remove default trigger of LAN LEDs as it will be handled in
01_leds anyway.

Fixes: 51b9aef553a8 ("ipq40xx: add support for ASUS RT-ACRH17/RT-AC42U")
Signed-off-by: Sungbo Eo <mans0n@gorani.run>

diff --git a/target/linux/ipq40xx/base-files/etc/board.d/01_leds b/target/linux/ipq40xx/base-files/etc/board.d/01_leds
index 26acfb60375cc6a7d478b48d9257eb08acff02bd..bc2ec58806e00547d09a72eedf51abd2b33c7c4f 100644 (file)
--- a/target/linux/ipq40xx/base-files/etc/board.d/01_leds
+++ b/target/linux/ipq40xx/base-files/etc/board.d/01_leds
@@ -13,10 +13,10 @@ alfa-network,ap120c-ac)
        ucidef_set_led_netdev "wan" "WAN" "amber:wan" "eth1"
        ;;
 asus,rt-ac42u)
-       ucidef_set_led_switch "lan1" "LAN1" "blue:lan" "switch0" "0x02"
-       ucidef_set_led_switch "lan2" "LAN2" "blue:lan_1" "switch0" "0x04"
-       ucidef_set_led_switch "lan3" "LAN3" "blue:lan_2" "switch0" "0x08"
-       ucidef_set_led_switch "lan4" "LAN4" "blue:lan_3" "switch0" "0x10"
+       ucidef_set_led_switch "lan1" "LAN1" "blue:lan-1" "switch0" "0x02"
+       ucidef_set_led_switch "lan2" "LAN2" "blue:lan-2" "switch0" "0x04"
+       ucidef_set_led_switch "lan3" "LAN3" "blue:lan-3" "switch0" "0x08"
+       ucidef_set_led_switch "lan4" "LAN4" "blue:lan-4" "switch0" "0x10"
        ;;
 asus,rt-ac58u)
        ucidef_set_led_netdev "wan" "WAN" "blue:wan" "eth1"

diff --git a/target/linux/ipq40xx/files/arch/arm/boot/dts/qcom-ipq4019-rt-ac42u.dts b/target/linux/ipq40xx/files/arch/arm/boot/dts/qcom-ipq4019-rt-ac42u.dts
index 31e2e40ac0328fb53e5af1e444caffc74be9919d..3d58eac09ed3b3b9bfb797ac51953f29453b7cc9 100644 (file)
--- a/target/linux/ipq40xx/files/arch/arm/boot/dts/qcom-ipq4019-rt-ac42u.dts
+++ b/target/linux/ipq40xx/files/arch/arm/boot/dts/qcom-ipq4019-rt-ac42u.dts
@@ -138,6 +138,7 @@
                led-3 {
                        color = <LED_COLOR_ID_BLUE>;
                        function = LED_FUNCTION_WLAN;
+                       function-enumerator = <0>;
                        gpios = <&tlmm 52 GPIO_ACTIVE_LOW>;
                        linux,default-trigger = "phy1tpt";
                };
@@ -145,6 +146,7 @@
                led-4 {
                        color = <LED_COLOR_ID_BLUE>;
                        function = LED_FUNCTION_WLAN;
+                       function-enumerator = <1>;
                        gpios = <&tlmm 54 GPIO_ACTIVE_LOW>;
                        linux,default-trigger = "phy0tpt";
                };
@@ -152,29 +154,29 @@
                led-5 {
                        color = <LED_COLOR_ID_BLUE>;
                        function = LED_FUNCTION_LAN;
+                       function-enumerator = <1>;
                        gpios = <&tlmm 45 GPIO_ACTIVE_LOW>;
-                       linux,default-trigger = "switch0";
                };
 
                led-6 {
                        color = <LED_COLOR_ID_BLUE>;
                        function = LED_FUNCTION_LAN;
+                       function-enumerator = <2>;
                        gpios = <&tlmm 43 GPIO_ACTIVE_LOW>;
-                       linux,default-trigger = "switch0";
                };
 
                led-7 {
                        color = <LED_COLOR_ID_BLUE>;
                        function = LED_FUNCTION_LAN;
+                       function-enumerator = <3>;
                        gpios = <&tlmm 42 GPIO_ACTIVE_LOW>;
-                       linux,default-trigger = "switch0";
                };
 
                led-8 {
                        color = <LED_COLOR_ID_BLUE>;
                        function = LED_FUNCTION_LAN;
+                       function-enumerator = <4>;
                        gpios = <&tlmm 49 GPIO_ACTIVE_LOW>;
-                       linux,default-trigger = "switch0";
                };
        };
 };