From: Christian Marangi <ansuelsmth@gmail.com>
Date: Thu, 5 Oct 2023 10:40:34 +0000 (+0200)
Subject: ipq806x: fix wrong QCA8K LED definition for NEC Aterm WG2600HP3
X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=7fc91be56431ae7ecd19f1f26302769264ffd4eb;p=openwrt%2Fopenwrt.git

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 <ansuelsmth@gmail.com>
---

diff --git a/target/linux/ipq806x/files/arch/arm/boot/dts/qcom-ipq8062-wg2600hp3.dts b/target/linux/ipq806x/files/arch/arm/boot/dts/qcom-ipq8062-wg2600hp3.dts
index 2c181bd883..04f4b7cd02 100644
--- a/target/linux/ipq806x/files/arch/arm/boot/dts/qcom-ipq8062-wg2600hp3.dts
+++ b/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";
 					};
 				};