From 158a5af80102dc01979ad7c405867eea6efd37dc Mon Sep 17 00:00:00 2001 From: =?utf8?q?Thibaut=20VAR=C3=88NE?= Date: Thu, 30 Jun 2022 10:06:35 +0200 Subject: [PATCH] ramips: improve YunCore AX820 LEDs MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit At least two AX820 hardware variants are known to exist, but they cannot be distinguished (same hardware revision, no specific markings). They appear to have the same LED hardware, but wired differently: - One has a red system LED at GPIO 15, a green wlan2g LED at GPIO 14 and a blue wlan5g LED at GPIO 16; - The other only offers a green system LED at GPIO 15, with GPIO 14 and 16 being apparently not connected Finally, a Yuncore datasheet says the canonical wiring should be: - Blue wlan2g GPIO 14, green system GPIO 15, red wlan5g GPIO 16 All GPIOs are tied to a single RGB LED which is exposed via lightpipe on the device front casing. Considering the above, this patch exposes all three LEDs, preserves the common system LED (GPIO 15) as the openwrt status LED, and removes the color information from the LEDs names since it is not consistent across hardware. The LED naming is made consistent with other YunCore devices. A note is added in DTS to ensure this information is always available and prevent unwanted changes in the future. Fixes: #10131 "YunCore AX820: GPIO LED not correct" Reviewed-by: Sander Vanheule Signed-off-by: Thibaut VARÈNE --- .../linux/ramips/dts/mt7621_yuncore_ax820.dts | 32 +++++++++++++++---- 1 file changed, 26 insertions(+), 6 deletions(-) diff --git a/target/linux/ramips/dts/mt7621_yuncore_ax820.dts b/target/linux/ramips/dts/mt7621_yuncore_ax820.dts index fe0e9daf7f..9638f1f060 100644 --- a/target/linux/ramips/dts/mt7621_yuncore_ax820.dts +++ b/target/linux/ramips/dts/mt7621_yuncore_ax820.dts @@ -10,10 +10,10 @@ model = "YunCore AX820"; aliases { - led-boot = &led_status_green; - led-failsafe = &led_status_green; - led-running = &led_status_green; - led-upgrade = &led_status_green; + led-boot = &led_system; + led-failsafe = &led_system; + led-running = &led_system; + led-upgrade = &led_system; }; chosen { @@ -23,10 +23,30 @@ leds { compatible = "gpio-leds"; - led_status_green: status_green { - label = "green:status"; + /* At least 2 AX820 variants are known and cannot be distinguished (same hw rev): + - System LED green GPIO 15; red and blue apparently not connected + - System LED red GPIO 15, WLAN2G LED green GPIO 14, WLAN5G LED blue GPIO 16 + The canonical values taken from a Yuncore datasheet are: + - System LED green GPIO 15, WLAN2G LED blue GPIO 14, WLAN5G LED red GPIO 16 + All 3 colors relate to the same single RGB LED showing up on the front lightpipe. + As a result, all 3 LEDs are defined even if only one may be available, and + the color information is absent as it is not reliable */ + wlan2g { + label = "wlan2g"; + gpios = <&gpio 14 GPIO_ACTIVE_LOW>; + linux,default-trigger = "phy0radio"; + }; + + led_system: system { + label = "system"; gpios = <&gpio 15 GPIO_ACTIVE_LOW>; }; + + wlan5g { + label = "wlan5g"; + gpios = <&gpio 16 GPIO_ACTIVE_LOW>; + linux,default-trigger = "phy1radio"; + }; }; watchdog { -- 2.30.2