diff options
author | Michael Buesch <mb@bu3sch.de> | 2008-01-09 14:15:31 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-01-28 18:10:42 -0500 |
commit | bfe6a50156ab0c33729378fae7038bce3c6c5184 (patch) | |
tree | 4d29fdaa77eef7772eadab31033227e49e67c911 /drivers/net/wireless/b43legacy/b43legacy.h | |
parent | a8bdf29c6cd117644d27677962fe832b33036c77 (diff) |
b43legacy: 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/b43legacy/b43legacy.h')
-rw-r--r-- | drivers/net/wireless/b43legacy/b43legacy.h | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/drivers/net/wireless/b43legacy/b43legacy.h b/drivers/net/wireless/b43legacy/b43legacy.h index 8352a4e1cfc9..93419adb925e 100644 --- a/drivers/net/wireless/b43legacy/b43legacy.h +++ b/drivers/net/wireless/b43legacy/b43legacy.h | |||
@@ -415,7 +415,6 @@ struct b43legacy_phy { | |||
415 | u8 calibrated:1; | 415 | u8 calibrated:1; |
416 | u8 radio_rev; /* Radio revision */ | 416 | u8 radio_rev; /* Radio revision */ |
417 | 417 | ||
418 | bool locked; /* Only used in b43legacy_phy_{un}lock() */ | ||
419 | bool dyn_tssi_tbl; /* tssi2dbm is kmalloc()ed. */ | 418 | bool dyn_tssi_tbl; /* tssi2dbm is kmalloc()ed. */ |
420 | 419 | ||
421 | /* ACI (adjacent channel interference) flags. */ | 420 | /* ACI (adjacent channel interference) flags. */ |
@@ -458,11 +457,6 @@ struct b43legacy_phy { | |||
458 | s16 lna_gain; /* LNA */ | 457 | s16 lna_gain; /* LNA */ |
459 | s16 pga_gain; /* PGA */ | 458 | s16 pga_gain; /* PGA */ |
460 | 459 | ||
461 | /* PHY lock for core.rev < 3 | ||
462 | * This lock is only used by b43legacy_phy_{un}lock() | ||
463 | */ | ||
464 | spinlock_t lock; | ||
465 | |||
466 | /* Desired TX power level (in dBm). This is set by the user and | 460 | /* Desired TX power level (in dBm). This is set by the user and |
467 | * adjusted in b43legacy_phy_xmitpower(). */ | 461 | * adjusted in b43legacy_phy_xmitpower(). */ |
468 | u8 power_level; | 462 | u8 power_level; |
@@ -486,9 +480,6 @@ struct b43legacy_phy { | |||
486 | u16 txpwr_offset; | 480 | u16 txpwr_offset; |
487 | }; | 481 | }; |
488 | 482 | ||
489 | #ifdef CONFIG_B43LEGACY_DEBUG | ||
490 | bool manual_txpower_control; /* Manual TX-power control enabled? */ | ||
491 | #endif | ||
492 | /* Current Interference Mitigation mode */ | 483 | /* Current Interference Mitigation mode */ |
493 | int interfmode; | 484 | int interfmode; |
494 | /* Stack of saved values from the Interference Mitigation code. | 485 | /* Stack of saved values from the Interference Mitigation code. |
@@ -516,6 +507,13 @@ struct b43legacy_phy { | |||
516 | 507 | ||
517 | /* PHY TX errors counter. */ | 508 | /* PHY TX errors counter. */ |
518 | atomic_t txerr_cnt; | 509 | atomic_t txerr_cnt; |
510 | |||
511 | #if B43legacy_DEBUG | ||
512 | /* Manual TX-power control enabled? */ | ||
513 | bool manual_txpower_control; | ||
514 | /* PHY registers locked by b43legacy_phy_lock()? */ | ||
515 | bool phy_locked; | ||
516 | #endif /* B43legacy_DEBUG */ | ||
519 | }; | 517 | }; |
520 | 518 | ||
521 | /* Data structures for DMA transmission, per 80211 core. */ | 519 | /* Data structures for DMA transmission, per 80211 core. */ |