diff options
author | Michael Buesch <mb@bu3sch.de> | 2008-01-09 13:08:49 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-01-28 18:09:32 -0500 |
commit | f31800d8b79bc42e495070aa6e6425841b7bdcbf (patch) | |
tree | 65f26267ae5b6d3876079913330634392654af00 /drivers/net/wireless/b43/debugfs.c | |
parent | 5250703e3144e50fbeceb4d1fc01ea2fd159fd4a (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.c | 6 |
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: | |||
222 | static int txpower_g_write_file(struct b43_wldev *dev, | 222 | static 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; |