apm821xx: WNDAP660: fix ethernet port ordering
authorChristian Lamparter <chunkeey@gmail.com>
Fri, 24 Jun 2022 18:55:15 +0000 (20:55 +0200)
committerChristian Lamparter <chunkeey@gmail.com>
Fri, 24 Jun 2022 22:06:46 +0000 (00:06 +0200)
Tim Small reported:
| Viewing the 'Network' -> 'Switch' config page in LuCI:
|
| The LuCI LAN 1 port corresponds to the port physically
|     labelled 2 at the rear of the device.
| [...]
|
| When a patch cord is attached to the port labelled 1 [...],
| the LED labelled 2 illuminates. [...]

=> Ports, LuCI and LEDs are reversed/don't match.

Reported-by: Tim Small
Fixes: #10111
Signed-off-by: Christian Lamparter <chunkeey@gmail.com>
target/linux/apm821xx/base-files/etc/board.d/01_leds
target/linux/apm821xx/base-files/etc/board.d/02_network
target/linux/apm821xx/base-files/etc/uci-defaults/04_led_migration
target/linux/apm821xx/dts/netgear-wndap660.dts

index 0b203fd8a7b958245b05219bebcb0854e60ecb4f..c384d24a30e6c23755d2e6239244eea0a9661161 100644 (file)
@@ -30,10 +30,10 @@ netgear,wndap620)
 
 netgear,wndap660)
        ucidef_set_led_netdev "lan_act" "LAN (Activity)" "green:activity" "eth0"
-       ucidef_set_led_switch "lan1_100" "LAN 100Mbps" "amber:lan-0" "switch0" "0x04" "0x04" "link"
-       ucidef_set_led_switch "lan1_1000" "LAN 1000Mbps" "green:lan-0" "switch0" "0x04" "0x08" "link"
-       ucidef_set_led_switch "lan2_100" "LAN 100Mbps" "amber:lan-1" "switch0" "0x02" "0x04" "link"
-       ucidef_set_led_switch "lan2_1000" "LAN 1000Mbps" "green:lan-1" "switch0" "0x02" "0x08" "link"
+       ucidef_set_led_switch "lan1_100" "LAN 100Mbps" "amber:lan-1" "switch0" "0x02" "0x04" "link"
+       ucidef_set_led_switch "lan1_1000" "LAN 1000Mbps" "green:lan-1" "switch0" "0x02" "0x08" "link"
+       ucidef_set_led_switch "lan2_100" "LAN 100Mbps" "amber:lan-2" "switch0" "0x04" "0x04" "link"
+       ucidef_set_led_switch "lan2_1000" "LAN 1000Mbps" "green:lan-2" "switch0" "0x04" "0x08" "link"
        ucidef_set_led_wlan "wlan2g" "WLAN2G" "green:wlan-0" "phy0tpt"
        ucidef_set_led_wlan "wlan5g" "WLAN5G" "green:wlan-1" "phy1tpt"
        ;;
index 3a97c67f28c3379ae8f5703950e90f1efa3ddbd4..756ea348b45adbf99696f96d276665f487d685ed 100644 (file)
@@ -16,7 +16,7 @@ netgear,wndap620)
        ucidef_add_switch "switch0" "2:lan" "5@eth0"
        ;;
 netgear,wndap660)
-       ucidef_add_switch "switch0" "1:lan:2" "2:lan:1" "5@eth0"
+       ucidef_add_switch "switch0" "1:lan" "2:lan" "5@eth0"
        ;;
 meraki,mx60|\
 netgear,wndr4700)
index b15ffa34af93b8ed5846d9afcb0204a5e62e311c..0152f2f47bead3459d6e28df8ef21f2a10d89cbe 100644 (file)
@@ -32,9 +32,9 @@ netgear,wndap660)
                ":test=:fault" \
                ":wlan2g=:wlan-0" \
                ":wlan5g=:wlan-1" \
-               ":lan1-link100=:lan-0" \
+               ":lan1-link100=:lan-1" \
                ":lan1-link1000=:lan-1" \
-               ":lan2-link100=:lan-1" \
+               ":lan2-link100=:lan-2" \
                ":lan2-link1000=:lan-2"
        ;;
 netgear,wndr4700)
index f01d1b3be0ddf55f98a86d6b91c239901e37aa6e..7cef1f5a452b4e9c952ef86d1cbf532ce82e5300 100644 (file)
                color = <LED_COLOR_ID_GREEN>;
                function = LED_FUNCTION_LAN;
                function-enumerator = <1>;
-               gpios = <&GPIO0 22 GPIO_ACTIVE_HIGH>;
+               gpios = <&GPIO0 9 GPIO_ACTIVE_HIGH>;
        };
 
        led-6 {
                color = <LED_COLOR_ID_AMBER>;
                function = LED_FUNCTION_LAN;
-               function-enumerator = <0>;
-               gpios = <&GPIO0 23 GPIO_ACTIVE_HIGH>;
+               function-enumerator = <1>;
+               gpios = <&GPIO0 10 GPIO_ACTIVE_HIGH>;
        };
 
        led-7 {
                color = <LED_COLOR_ID_GREEN>;
                function = LED_FUNCTION_LAN;
                function-enumerator = <2>;
-               gpios = <&GPIO0 9 GPIO_ACTIVE_HIGH>;
+               gpios = <&GPIO0 22 GPIO_ACTIVE_HIGH>;
        };
 
        led-8 {
                color = <LED_COLOR_ID_AMBER>;
                function = LED_FUNCTION_LAN;
-               function-enumerator = <1>;
-               gpios = <&GPIO0 10 GPIO_ACTIVE_HIGH>;
+               function-enumerator = <2>;
+               gpios = <&GPIO0 23 GPIO_ACTIVE_HIGH>;
        };
 };