From 7fc91be56431ae7ecd19f1f26302769264ffd4eb Mon Sep 17 00:00:00 2001
From: Christian Marangi <ansuelsmth@gmail.com>
Date: Thu, 5 Oct 2023 12:40:34 +0200
Subject: [PATCH] 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>
---
 .../arm/boot/dts/qcom-ipq8062-wg2600hp3.dts   | 60 +++++++++++++++++--
 1 file changed, 55 insertions(+), 5 deletions(-)

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";
 					};
 				};