ipq806x: fix wrong QCA8K LED definition for NEC Aterm WG2600HP3
authorChristian Marangi <ansuelsmth@gmail.com>
Thu, 5 Oct 2023 10:40:34 +0000 (12:40 +0200)
committerChristian Marangi <ansuelsmth@gmail.com>
Thu, 5 Oct 2023 10:40:34 +0000 (12:40 +0200)
Fix copy-paste error in migrating NEC Aterm WG2600HP3 to new LED
implementation for the QCA8K switch. Correct define the missing
additional LED pin used for each port and fix wrong color for LED 2 for
each port. Also add the required function-enumerator as all 3 LED have
the same color and function.

Fixes: c707cff6c94b ("ipq806x: add LEDs definition for non-standard qca8k LEDs")
Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
target/linux/ipq806x/files/arch/arm/boot/dts/qcom-ipq8062-wg2600hp3.dts

index 2c181bd8830843eb9fddd831a6d1ca46c7ea93aa..04f4b7cd023834769244f1892b254268fad78360 100644 (file)
                                                reg = <0>;
                                                color = <LED_COLOR_ID_GREEN>;
                                                function = LED_FUNCTION_WAN;
+                                               function-enumerator = <1>;
                                                default-state = "keep";
                                        };
 
                                        led@1 {
                                                reg = <1>;
-                                               color = <LED_COLOR_ID_AMBER>;
+                                               color = <LED_COLOR_ID_GREEN>;
+                                               function = LED_FUNCTION_WAN;
+                                               function-enumerator = <2>;
+                                               default-state = "keep";
+                                       };
+
+                                       led@2 {
+                                               reg = <2>;
+                                               color = <LED_COLOR_ID_GREEN>;
                                                function = LED_FUNCTION_WAN;
+                                               function-enumerator = <3>;
                                                default-state = "keep";
                                        };
                                };
                                                reg = <0>;
                                                color = <LED_COLOR_ID_GREEN>;
                                                function = LED_FUNCTION_LAN;
+                                               function-enumerator = <1>;
                                                default-state = "keep";
                                        };
 
                                        led@1 {
                                                reg = <1>;
-                                               color = <LED_COLOR_ID_AMBER>;
+                                               color = <LED_COLOR_ID_GREEN>;
                                                function = LED_FUNCTION_LAN;
+                                               function-enumerator = <2>;
+                                               default-state = "keep";
+                                       };
+
+                                       led@2 {
+                                               reg = <2>;
+                                               color = <LED_COLOR_ID_GREEN>;
+                                               function = LED_FUNCTION_LAN;
+                                               function-enumerator = <3>;
                                                default-state = "keep";
                                        };
                                };
                                                reg = <0>;
                                                color = <LED_COLOR_ID_GREEN>;
                                                function = LED_FUNCTION_LAN;
+                                               function-enumerator = <1>;
                                                default-state = "keep";
                                        };
 
                                        led@1 {
                                                reg = <1>;
-                                               color = <LED_COLOR_ID_AMBER>;
+                                               color = <LED_COLOR_ID_GREEN>;
+                                               function = LED_FUNCTION_LAN;
+                                               function-enumerator = <2>;
+                                               default-state = "keep";
+                                       };
+
+                                       led@2 {
+                                               reg = <2>;
+                                               color = <LED_COLOR_ID_GREEN>;
                                                function = LED_FUNCTION_LAN;
+                                               function-enumerator = <3>;
                                                default-state = "keep";
                                        };
                                };
                                                reg = <0>;
                                                color = <LED_COLOR_ID_GREEN>;
                                                function = LED_FUNCTION_LAN;
+                                               function-enumerator = <1>;
                                                default-state = "keep";
                                        };
 
                                        led@1 {
                                                reg = <1>;
-                                               color = <LED_COLOR_ID_AMBER>;
+                                               color = <LED_COLOR_ID_GREEN>;
                                                function = LED_FUNCTION_LAN;
+                                               function-enumerator = <2>;
+                                               default-state = "keep";
+                                       };
+
+                                       led@2 {
+                                               reg = <2>;
+                                               color = <LED_COLOR_ID_GREEN>;
+                                               function = LED_FUNCTION_LAN;
+                                               function-enumerator = <3>;
                                                default-state = "keep";
                                        };
                                };
                                                reg = <0>;
                                                color = <LED_COLOR_ID_GREEN>;
                                                function = LED_FUNCTION_LAN;
+                                               function-enumerator = <1>;
                                                default-state = "keep";
                                        };
 
                                        led@1 {
                                                reg = <1>;
-                                               color = <LED_COLOR_ID_AMBER>;
+                                               color = <LED_COLOR_ID_GREEN>;
+                                               function = LED_FUNCTION_LAN;
+                                               function-enumerator = <2>;
+                                               default-state = "keep";
+                                       };
+
+                                       led@2 {
+                                               reg = <2>;
+                                               color = <LED_COLOR_ID_GREEN>;
                                                function = LED_FUNCTION_LAN;
+                                               function-enumerator = <3>;
                                                default-state = "keep";
                                        };
                                };