Browse Source

ipq806x: fix wrong QCA8K LED definition for NEC Aterm WG2600HP3

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 <[email protected]>
Christian Marangi 2 years ago
parent
commit
7fc91be564

+ 55 - 5
target/linux/ipq806x/files/arch/arm/boot/dts/qcom-ipq8062-wg2600hp3.dts

@@ -451,13 +451,23 @@
 						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";
 					};
 				};
@@ -477,13 +487,23 @@
 						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";
 					};
 				};
@@ -503,13 +523,23 @@
 						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";
 					};
 				};
@@ -529,13 +559,23 @@
 						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";
 					};
 				};
@@ -555,13 +595,23 @@
 						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";
 					};
 				};