diff mbox

[OpenWrt-Devel,5/5] lantiq: Configure LED polarity for TDW8970.

Message ID owrt-patch-20151216-5@vittgam.net
State Accepted
Headers show

Commit Message

Vittorio Gambaletta Dec. 16, 2015, 4:42 a.m. UTC
This patch configures the correct ath9k WLAN LED polarity for the TDW8970.

Signed-off-by: Vittorio Gambaletta <openwrt@vittgam.net>

---

The TDW8970 LED is active high, but I'm not sure about the TDW8980 as
I don't have one. If the TDW8980 LED is active high as well, this can
be added to TDW89X0.dtsi instead of TDW8970.dts.

Comments

Mathias Kresin Dec. 16, 2015, 9:12 p.m. UTC | #1
Am 16.12.2015 um 05:42 schrieb Vittorio G (VittGam):
> This patch configures the correct ath9k WLAN LED polarity for the TDW8970.
>
> Signed-off-by: Vittorio Gambaletta <openwrt@vittgam.net>
>
> ---
>
> The TDW8970 LED is active high, but I'm not sure about the TDW8980 as
> I don't have one. If the TDW8980 LED is active high as well, this can
> be added to TDW89X0.dtsi instead of TDW8970.dts.

The TDW8980 2.4 GHz Wireless LED is active high as well and your patch 
fixed the inverse lighting.

Would you please move the definition to the dtsi!

>
> --- a/target/linux/lantiq/dts/TDW8970.dts
> +++ b/target/linux/lantiq/dts/TDW8970.dts
> @@ -4,4 +4,8 @@
>
>   / {
>   	model = "TDW8970 - TP-LINK TD-W8970";
> +
> +	ath9k_eep {
> +		ath,led-active-high;
> +	};
>   };
> _______________________________________________
> openwrt-devel mailing list
> openwrt-devel@lists.openwrt.org
> https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
>
Vittorio Gambaletta Dec. 16, 2015, 11:16 p.m. UTC | #2
On 16 December 2015 22:12:15 CET, Mathias Kresin <openwrt@kresin.me> wrote:
>The TDW8980 2.4 GHz Wireless LED is active high as well and your patch 
>fixed the inverse lighting.
>
>Would you please move the definition to the dtsi!

Sure, I will update the patches then. Thanks for testing!
diff mbox

Patch

--- a/target/linux/lantiq/dts/TDW8970.dts
+++ b/target/linux/lantiq/dts/TDW8970.dts
@@ -4,4 +4,8 @@ 
 
 / {
 	model = "TDW8970 - TP-LINK TD-W8970";
+
+	ath9k_eep {
+		ath,led-active-high;
+	};
 };