aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/net/wireless/b43/debugfs.c
diff options
context:
space:
mode:
authorMichael Buesch <mb@bu3sch.de>2008-01-09 13:08:49 -0500
committerDavid S. Miller <davem@davemloft.net>2008-01-28 18:09:32 -0500
commitf31800d8b79bc42e495070aa6e6425841b7bdcbf (patch)
tree65f26267ae5b6d3876079913330634392654af00 /drivers/net/wireless/b43/debugfs.c
parent5250703e3144e50fbeceb4d1fc01ea2fd159fd4a (diff)
b43: Remove the PHY spinlock
This fixes a sparse warning about weird locking. The spinlock is not needed, so simply remove it. This also adds some sanity checks to the PHY and radio locking to protect against recursive locking. Signed-off-by: Michael Buesch <mb@bu3sch.de> Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net/wireless/b43/debugfs.c')
-rw-r--r--drivers/net/wireless/b43/debugfs.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/net/wireless/b43/debugfs.c b/drivers/net/wireless/b43/debugfs.c
index 93e5030aa6c3..e38ed0fe72e9 100644
--- a/drivers/net/wireless/b43/debugfs.c
+++ b/drivers/net/wireless/b43/debugfs.c
@@ -222,8 +222,6 @@ out:
222static int txpower_g_write_file(struct b43_wldev *dev, 222static int txpower_g_write_file(struct b43_wldev *dev,
223 const char *buf, size_t count) 223 const char *buf, size_t count)
224{ 224{
225 unsigned long phy_flags;
226
227 if (dev->phy.type != B43_PHYTYPE_G) 225 if (dev->phy.type != B43_PHYTYPE_G)
228 return -ENODEV; 226 return -ENODEV;
229 if ((count >= 4) && (memcmp(buf, "auto", 4) == 0)) { 227 if ((count >= 4) && (memcmp(buf, "auto", 4) == 0)) {
@@ -247,12 +245,12 @@ static int txpower_g_write_file(struct b43_wldev *dev,
247 dev->phy.tx_control |= B43_TXCTL_PA2DB; 245 dev->phy.tx_control |= B43_TXCTL_PA2DB;
248 if (pa3db) 246 if (pa3db)
249 dev->phy.tx_control |= B43_TXCTL_PA3DB; 247 dev->phy.tx_control |= B43_TXCTL_PA3DB;
250 b43_phy_lock(dev, phy_flags); 248 b43_phy_lock(dev);
251 b43_radio_lock(dev); 249 b43_radio_lock(dev);
252 b43_set_txpower_g(dev, &dev->phy.bbatt, 250 b43_set_txpower_g(dev, &dev->phy.bbatt,
253 &dev->phy.rfatt, dev->phy.tx_control); 251 &dev->phy.rfatt, dev->phy.tx_control);
254 b43_radio_unlock(dev); 252 b43_radio_unlock(dev);
255 b43_phy_unlock(dev, phy_flags); 253 b43_phy_unlock(dev);
256 } 254 }
257 255
258 return 0; 256 return 0;