ipq806x: rt4230w-rev6: fix status reporting via the LEDs

There is a custom LED controller between the 3 SoC GPIO outputs and
the red and blue LEDs of the device. It implements a strange mapping
that includes fixed, flashing, and breathing modes.

The current DTS configuration causes OpenWrt to flash the LEDs over
the controller's own flashing, resulting in chaotic output in boot,
failsafe, and upgrade modes.

This change fixes the LEDs in the best way possible as long as each
OpenWrt running state is limited to be signaled by a single led.

Signed-off-by: Rodrigo Balerdi <lanchon@gmail.com>
Link: https://github.com/openwrt/openwrt/pull/15440
Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>

diff --git a/target/linux/ipq806x/files-6.6/arch/arm/boot/dts/qcom/qcom-ipq8065-rt4230w-rev6.dts b/target/linux/ipq806x/files-6.6/arch/arm/boot/dts/qcom/qcom-ipq8065-rt4230w-rev6.dts
index 12f15bd1470d3922751cbb9e02b1bafb78ad4adf..f10fa367f1eb31b7b2eb83f0a393c705a44e89c8 100644 (file)
--- a/target/linux/ipq806x/files-6.6/arch/arm/boot/dts/qcom/qcom-ipq8065-rt4230w-rev6.dts
+++ b/target/linux/ipq806x/files-6.6/arch/arm/boot/dts/qcom/qcom-ipq8065-rt4230w-rev6.dts
@@ -14,10 +14,10 @@
        };
 
        aliases {
-               led-boot = &ledctrl3;
+               led-boot = &ledctrl1;
                led-failsafe = &ledctrl1;
-               led-running = &ledctrl2;
-               led-upgrade = &ledctrl3;
+               led-running = &ledctrl3;
+               led-upgrade = &ledctrl1;
        };
 
        chosen {
@@ -55,6 +55,7 @@
                ledctrl2: ledctrl2 {
                        label = "ledctrl2";
                        gpios = <&qcom_pinmux 23 GPIO_ACTIVE_HIGH>;
+                       linux,default-trigger = "default-on";
                };
 
                ledctrl3: ledctrl3 {