From: Jonas Jelonek <jelonek.jonas@gmail.com>
Date: Mon, 18 May 2026 09:01:11 +0000 (+0000)
Subject: realtek: fix LED function for XS1930 switches
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=9114f49446131ddd75ee4ec3168d327e6eaee9ed;p=thirdparty%2Fopenwrt.git

realtek: fix LED function for XS1930 switches

The cloud and locator LEDs were declared with no function (cloud) or
with the generic LED_FUNCTION_INDICATOR (locator), which doesn't
match what the hardware actually exposes. Use the descriptive
function strings "cloud" and "locator" instead so the LEDs end up
with sensible names in sysfs.

Signed-off-by: Jonas Jelonek <jelonek.jonas@gmail.com>
Link: https://github.com/openwrt/openwrt/pull/23428
Signed-off-by: Robert Marko <robimarko@gmail.com>
---

diff --git a/target/linux/realtek/dts/rtl9313_zyxel_xs1930.dtsi b/target/linux/realtek/dts/rtl9313_zyxel_xs1930.dtsi
index 2523740f632..d5fa56bc06a 100644
--- a/target/linux/realtek/dts/rtl9313_zyxel_xs1930.dtsi
+++ b/target/linux/realtek/dts/rtl9313_zyxel_xs1930.dtsi
@@ -66,15 +66,17 @@
 		};
 		led_cloud_green: led-4 {
 			color = <LED_COLOR_ID_GREEN>;
+			function = "cloud";
 			gpios = <&gpio0 24 GPIO_ACTIVE_LOW>;
 		};
 		led_cloud_red: led-5 {
 			color = <LED_COLOR_ID_RED>;
+			function = "cloud";
 			gpios = <&gpio0 25 GPIO_ACTIVE_LOW>;
 		};
 		led_locator: led-6 {
 			color = <LED_COLOR_ID_BLUE>;
-			function = LED_FUNCTION_INDICATOR;
+			function = "locator";
 			gpios = <&gpio0 26 GPIO_ACTIVE_LOW>;
 		};
 	};