ath9k: avoid usage of ath9k_hw_setpower() on hw.c
ath9k_hw_setpower() is a core driver helper with locking
protection. Locking protection should be left to the driver
core, not the hw code. Hardware code no longer contends for
locking when it needs to wake up the chip or put it to sleep.
Signed-off-by: Luis R. Rodriguez <lrodriguez@atheros.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
diff --git a/drivers/net/wireless/ath/ath9k/hw.c b/drivers/net/wireless/ath/ath9k/hw.c
index 82c91e5..9107262 100644
--- a/drivers/net/wireless/ath/ath9k/hw.c
+++ b/drivers/net/wireless/ath/ath9k/hw.c
@@ -919,7 +919,7 @@
return -EIO;
}
- if (!ath9k_hw_setpower(ah, ATH9K_PM_AWAKE)) {
+ if (!ath9k_hw_setpower_nolock(ah, ATH9K_PM_AWAKE)) {
DPRINTF(ah, ATH_DBG_FATAL, "Couldn't wakeup chip\n");
return -EIO;
}
@@ -1234,7 +1234,7 @@
ath9k_hw_ani_disable(ah);
ath9k_hw_rf_free(ah);
- ath9k_hw_setpower(ah, ATH9K_PM_FULL_SLEEP);
+ ath9k_hw_setpower_nolock(ah, ATH9K_PM_FULL_SLEEP);
kfree(ah);
ah = NULL;
}
@@ -1800,7 +1800,7 @@
} else if (!ath9k_hw_set_reset_reg(ah, ATH9K_RESET_WARM))
return false;
- if (!ath9k_hw_setpower(ah, ATH9K_PM_AWAKE))
+ if (!ath9k_hw_setpower_nolock(ah, ATH9K_PM_AWAKE))
return false;
ah->chip_fullsleep = false;
@@ -2355,7 +2355,7 @@
ah->txchainmask = sc->tx_chainmask;
ah->rxchainmask = sc->rx_chainmask;
- if (!ath9k_hw_setpower(ah, ATH9K_PM_AWAKE))
+ if (!ath9k_hw_setpower_nolock(ah, ATH9K_PM_AWAKE))
return -EIO;
if (curchan && !ah->chip_fullsleep)
@@ -3998,7 +3998,7 @@
bool ath9k_hw_disable(struct ath_hw *ah)
{
- if (!ath9k_hw_setpower(ah, ATH9K_PM_AWAKE))
+ if (!ath9k_hw_setpower_nolock(ah, ATH9K_PM_AWAKE))
return false;
return ath9k_hw_set_reset_reg(ah, ATH9K_RESET_COLD);