linux-kernel - [PATCH] staging: r8187se: Fix for LED not following radio switch

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <4b746b74.8SETcYq7pcTkXOg9%Larry.Finger@lwfinger.net>
Date:	Thu, 11 Feb 2010 14:41:24 -0600
From:	Larry Finger <Larry.Finger@...inger.net>
To:	Greg Kroah-Hartman <gregkh@...e.de>
Cc:	Bernhard Schiffner <bernhard@...iffner-limbach.de>,
	linux-kernel@...r.kernel.org, linux-wireless@...r.kernel.org
Subject: [PATCH] staging: r8187se: Fix for LED not following radio switch

The current driver does not follow the state of the RF switch.

Signed-off-by: Larry Finger <Larry.Finger@...inger.net>
---

Index: wireless-testing/drivers/staging/rtl8187se/r8180_core.c
===================================================================
--- wireless-testing.orig/drivers/staging/rtl8187se/r8180_core.c
+++ wireless-testing/drivers/staging/rtl8187se/r8180_core.c
@@ -4293,11 +4293,12 @@ void GPIOChangeRFWorkItemCallBack(struct
 			//HW radio On/Off according to the value of FF51[4](config0)
 			btConfig0 = btPSR = read_nic_byte(dev, CONFIG0);
 
-			//Turn on LED.
-			write_nic_byte(dev, PSR, btPSR| BIT3);
-
 			eRfPowerStateToSet = (btConfig0 & BIT4) ?  eRfOn : eRfOff;
 
+			/* Turn LED back on when radio enabled */
+			if (eRfPowerStateToSet == eRfOn)
+				write_nic_byte(dev, PSR, btPSR | BIT3);
+
 			if((priv->ieee80211->bHwRadioOff == true) && (eRfPowerStateToSet == eRfOn))
 			{
 				priv->ieee80211->bHwRadioOff = false;
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/