mirror of
git://projects.qi-hardware.com/openwrt-xburst.git
synced 2024-11-27 21:44:04 +02:00
ath9k: fix a few crash issues on hardware reset
git-svn-id: svn://svn.openwrt.org/openwrt/trunk@28122 3c298f89-4303-0410-b956-a3cf2f4a3e73
This commit is contained in:
parent
cd439b4056
commit
d8083fae30
253
package/mac80211/patches/581-ath9k_use_reset_work.patch
Normal file
253
package/mac80211/patches/581-ath9k_use_reset_work.patch
Normal file
@ -0,0 +1,253 @@
|
|||||||
|
--- a/drivers/net/wireless/ath/ath9k/ath9k.h
|
||||||
|
+++ b/drivers/net/wireless/ath/ath9k/ath9k.h
|
||||||
|
@@ -429,6 +429,7 @@ void ath9k_set_beaconing_status(struct a
|
||||||
|
|
||||||
|
#define ATH_PAPRD_TIMEOUT 100 /* msecs */
|
||||||
|
|
||||||
|
+void ath_reset_work(struct work_struct *work);
|
||||||
|
void ath_hw_check(struct work_struct *work);
|
||||||
|
void ath_hw_pll_work(struct work_struct *work);
|
||||||
|
void ath_paprd_calibrate(struct work_struct *work);
|
||||||
|
@@ -609,6 +610,7 @@ struct ath_softc {
|
||||||
|
struct mutex mutex;
|
||||||
|
struct work_struct paprd_work;
|
||||||
|
struct work_struct hw_check_work;
|
||||||
|
+ struct work_struct hw_reset_work;
|
||||||
|
struct completion paprd_complete;
|
||||||
|
|
||||||
|
unsigned int hw_busy_count;
|
||||||
|
@@ -655,7 +657,6 @@ struct ath_softc {
|
||||||
|
};
|
||||||
|
|
||||||
|
void ath9k_tasklet(unsigned long data);
|
||||||
|
-int ath_reset(struct ath_softc *sc, bool retry_tx);
|
||||||
|
int ath_cabq_update(struct ath_softc *);
|
||||||
|
|
||||||
|
static inline void ath_read_cachesize(struct ath_common *common, int *csz)
|
||||||
|
--- a/drivers/net/wireless/ath/ath9k/init.c
|
||||||
|
+++ b/drivers/net/wireless/ath/ath9k/init.c
|
||||||
|
@@ -776,6 +776,7 @@ int ath9k_init_device(u16 devid, struct
|
||||||
|
goto error_world;
|
||||||
|
}
|
||||||
|
|
||||||
|
+ INIT_WORK(&sc->hw_reset_work, ath_reset_work);
|
||||||
|
INIT_WORK(&sc->hw_check_work, ath_hw_check);
|
||||||
|
INIT_WORK(&sc->paprd_work, ath_paprd_calibrate);
|
||||||
|
INIT_DELAYED_WORK(&sc->hw_pll_work, ath_hw_pll_work);
|
||||||
|
--- a/drivers/net/wireless/ath/ath9k/main.c
|
||||||
|
+++ b/drivers/net/wireless/ath/ath9k/main.c
|
||||||
|
@@ -595,74 +595,6 @@ static void ath_node_detach(struct ath_s
|
||||||
|
ath_tx_node_cleanup(sc, an);
|
||||||
|
}
|
||||||
|
|
||||||
|
-void ath_hw_check(struct work_struct *work)
|
||||||
|
-{
|
||||||
|
- struct ath_softc *sc = container_of(work, struct ath_softc, hw_check_work);
|
||||||
|
- struct ath_common *common = ath9k_hw_common(sc->sc_ah);
|
||||||
|
- unsigned long flags;
|
||||||
|
- int busy;
|
||||||
|
-
|
||||||
|
- ath9k_ps_wakeup(sc);
|
||||||
|
- if (ath9k_hw_check_alive(sc->sc_ah))
|
||||||
|
- goto out;
|
||||||
|
-
|
||||||
|
- spin_lock_irqsave(&common->cc_lock, flags);
|
||||||
|
- busy = ath_update_survey_stats(sc);
|
||||||
|
- spin_unlock_irqrestore(&common->cc_lock, flags);
|
||||||
|
-
|
||||||
|
- ath_dbg(common, ATH_DBG_RESET, "Possible baseband hang, "
|
||||||
|
- "busy=%d (try %d)\n", busy, sc->hw_busy_count + 1);
|
||||||
|
- if (busy >= 99) {
|
||||||
|
- if (++sc->hw_busy_count >= 3) {
|
||||||
|
- spin_lock_bh(&sc->sc_pcu_lock);
|
||||||
|
- ath_reset(sc, true);
|
||||||
|
- spin_unlock_bh(&sc->sc_pcu_lock);
|
||||||
|
- }
|
||||||
|
- } else if (busy >= 0)
|
||||||
|
- sc->hw_busy_count = 0;
|
||||||
|
-
|
||||||
|
-out:
|
||||||
|
- ath9k_ps_restore(sc);
|
||||||
|
-}
|
||||||
|
-
|
||||||
|
-static void ath_hw_pll_rx_hang_check(struct ath_softc *sc, u32 pll_sqsum)
|
||||||
|
-{
|
||||||
|
- static int count;
|
||||||
|
- struct ath_common *common = ath9k_hw_common(sc->sc_ah);
|
||||||
|
-
|
||||||
|
- if (pll_sqsum >= 0x40000) {
|
||||||
|
- count++;
|
||||||
|
- if (count == 3) {
|
||||||
|
- /* Rx is hung for more than 500ms. Reset it */
|
||||||
|
- ath_dbg(common, ATH_DBG_RESET,
|
||||||
|
- "Possible RX hang, resetting");
|
||||||
|
- spin_lock_bh(&sc->sc_pcu_lock);
|
||||||
|
- ath_reset(sc, true);
|
||||||
|
- spin_unlock_bh(&sc->sc_pcu_lock);
|
||||||
|
- count = 0;
|
||||||
|
- }
|
||||||
|
- } else
|
||||||
|
- count = 0;
|
||||||
|
-}
|
||||||
|
-
|
||||||
|
-void ath_hw_pll_work(struct work_struct *work)
|
||||||
|
-{
|
||||||
|
- struct ath_softc *sc = container_of(work, struct ath_softc,
|
||||||
|
- hw_pll_work.work);
|
||||||
|
- u32 pll_sqsum;
|
||||||
|
-
|
||||||
|
- if (AR_SREV_9485(sc->sc_ah)) {
|
||||||
|
-
|
||||||
|
- ath9k_ps_wakeup(sc);
|
||||||
|
- pll_sqsum = ar9003_get_pll_sqsum_dvc(sc->sc_ah);
|
||||||
|
- ath9k_ps_restore(sc);
|
||||||
|
-
|
||||||
|
- ath_hw_pll_rx_hang_check(sc, pll_sqsum);
|
||||||
|
-
|
||||||
|
- ieee80211_queue_delayed_work(sc->hw, &sc->hw_pll_work, HZ/5);
|
||||||
|
- }
|
||||||
|
-}
|
||||||
|
-
|
||||||
|
|
||||||
|
void ath9k_tasklet(unsigned long data)
|
||||||
|
{
|
||||||
|
@@ -675,9 +607,7 @@ void ath9k_tasklet(unsigned long data)
|
||||||
|
|
||||||
|
if ((status & ATH9K_INT_FATAL) ||
|
||||||
|
(status & ATH9K_INT_BB_WATCHDOG)) {
|
||||||
|
- spin_lock(&sc->sc_pcu_lock);
|
||||||
|
- ath_reset(sc, true);
|
||||||
|
- spin_unlock(&sc->sc_pcu_lock);
|
||||||
|
+ ieee80211_queue_work(sc->hw, &sc->hw_reset_work);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
@@ -968,7 +898,7 @@ void ath_radio_disable(struct ath_softc
|
||||||
|
ath9k_ps_restore(sc);
|
||||||
|
}
|
||||||
|
|
||||||
|
-int ath_reset(struct ath_softc *sc, bool retry_tx)
|
||||||
|
+static int ath_reset(struct ath_softc *sc, bool retry_tx)
|
||||||
|
{
|
||||||
|
struct ath_hw *ah = sc->sc_ah;
|
||||||
|
struct ath_common *common = ath9k_hw_common(ah);
|
||||||
|
@@ -1035,6 +965,84 @@ int ath_reset(struct ath_softc *sc, bool
|
||||||
|
return r;
|
||||||
|
}
|
||||||
|
|
||||||
|
+void ath_reset_work(struct work_struct *work)
|
||||||
|
+{
|
||||||
|
+ struct ath_softc *sc = container_of(work, struct ath_softc, hw_check_work);
|
||||||
|
+
|
||||||
|
+ spin_lock_bh(&sc->sc_pcu_lock);
|
||||||
|
+ ath_reset(sc, true);
|
||||||
|
+ spin_unlock_bh(&sc->sc_pcu_lock);
|
||||||
|
+}
|
||||||
|
+
|
||||||
|
+void ath_hw_check(struct work_struct *work)
|
||||||
|
+{
|
||||||
|
+ struct ath_softc *sc = container_of(work, struct ath_softc, hw_check_work);
|
||||||
|
+ struct ath_common *common = ath9k_hw_common(sc->sc_ah);
|
||||||
|
+ unsigned long flags;
|
||||||
|
+ int busy;
|
||||||
|
+
|
||||||
|
+ ath9k_ps_wakeup(sc);
|
||||||
|
+ if (ath9k_hw_check_alive(sc->sc_ah))
|
||||||
|
+ goto out;
|
||||||
|
+
|
||||||
|
+ spin_lock_irqsave(&common->cc_lock, flags);
|
||||||
|
+ busy = ath_update_survey_stats(sc);
|
||||||
|
+ spin_unlock_irqrestore(&common->cc_lock, flags);
|
||||||
|
+
|
||||||
|
+ ath_dbg(common, ATH_DBG_RESET, "Possible baseband hang, "
|
||||||
|
+ "busy=%d (try %d)\n", busy, sc->hw_busy_count + 1);
|
||||||
|
+ if (busy >= 99) {
|
||||||
|
+ if (++sc->hw_busy_count >= 3) {
|
||||||
|
+ spin_lock_bh(&sc->sc_pcu_lock);
|
||||||
|
+ ath_reset(sc, true);
|
||||||
|
+ spin_unlock_bh(&sc->sc_pcu_lock);
|
||||||
|
+ }
|
||||||
|
+
|
||||||
|
+ } else if (busy >= 0)
|
||||||
|
+ sc->hw_busy_count = 0;
|
||||||
|
+
|
||||||
|
+out:
|
||||||
|
+ ath9k_ps_restore(sc);
|
||||||
|
+}
|
||||||
|
+
|
||||||
|
+static void ath_hw_pll_rx_hang_check(struct ath_softc *sc, u32 pll_sqsum)
|
||||||
|
+{
|
||||||
|
+ static int count;
|
||||||
|
+ struct ath_common *common = ath9k_hw_common(sc->sc_ah);
|
||||||
|
+
|
||||||
|
+ if (pll_sqsum >= 0x40000) {
|
||||||
|
+ count++;
|
||||||
|
+ if (count == 3) {
|
||||||
|
+ /* Rx is hung for more than 500ms. Reset it */
|
||||||
|
+ ath_dbg(common, ATH_DBG_RESET,
|
||||||
|
+ "Possible RX hang, resetting");
|
||||||
|
+ spin_lock_bh(&sc->sc_pcu_lock);
|
||||||
|
+ ath_reset(sc, true);
|
||||||
|
+ spin_unlock_bh(&sc->sc_pcu_lock);
|
||||||
|
+ count = 0;
|
||||||
|
+ }
|
||||||
|
+ } else
|
||||||
|
+ count = 0;
|
||||||
|
+}
|
||||||
|
+
|
||||||
|
+void ath_hw_pll_work(struct work_struct *work)
|
||||||
|
+{
|
||||||
|
+ struct ath_softc *sc = container_of(work, struct ath_softc,
|
||||||
|
+ hw_pll_work.work);
|
||||||
|
+ u32 pll_sqsum;
|
||||||
|
+
|
||||||
|
+ if (AR_SREV_9485(sc->sc_ah)) {
|
||||||
|
+
|
||||||
|
+ ath9k_ps_wakeup(sc);
|
||||||
|
+ pll_sqsum = ar9003_get_pll_sqsum_dvc(sc->sc_ah);
|
||||||
|
+ ath9k_ps_restore(sc);
|
||||||
|
+
|
||||||
|
+ ath_hw_pll_rx_hang_check(sc, pll_sqsum);
|
||||||
|
+
|
||||||
|
+ ieee80211_queue_delayed_work(sc->hw, &sc->hw_pll_work, HZ/5);
|
||||||
|
+ }
|
||||||
|
+}
|
||||||
|
+
|
||||||
|
/**********************/
|
||||||
|
/* mac80211 callbacks */
|
||||||
|
/**********************/
|
||||||
|
--- a/drivers/net/wireless/ath/ath9k/xmit.c
|
||||||
|
+++ b/drivers/net/wireless/ath/ath9k/xmit.c
|
||||||
|
@@ -601,7 +601,7 @@ static void ath_tx_complete_aggr(struct
|
||||||
|
rcu_read_unlock();
|
||||||
|
|
||||||
|
if (needreset)
|
||||||
|
- ath_reset(sc, false);
|
||||||
|
+ ieee80211_queue_work(sc->hw, &sc->hw_reset_work);
|
||||||
|
}
|
||||||
|
|
||||||
|
static bool ath_lookup_legacy(struct ath_buf *bf)
|
||||||
|
@@ -2268,9 +2268,7 @@ static void ath_tx_complete_poll_work(st
|
||||||
|
if (needreset) {
|
||||||
|
ath_dbg(ath9k_hw_common(sc->sc_ah), ATH_DBG_RESET,
|
||||||
|
"tx hung, resetting the chip\n");
|
||||||
|
- spin_lock_bh(&sc->sc_pcu_lock);
|
||||||
|
- ath_reset(sc, true);
|
||||||
|
- spin_unlock_bh(&sc->sc_pcu_lock);
|
||||||
|
+ ieee80211_queue_work(sc->hw, &sc->hw_reset_work);
|
||||||
|
}
|
||||||
|
|
||||||
|
ieee80211_queue_delayed_work(sc->hw, &sc->tx_complete_work,
|
||||||
|
--- a/drivers/net/wireless/ath/ath9k/beacon.c
|
||||||
|
+++ b/drivers/net/wireless/ath/ath9k/beacon.c
|
||||||
|
@@ -386,9 +386,7 @@ void ath_beacon_tasklet(unsigned long da
|
||||||
|
ath_dbg(common, ATH_DBG_BSTUCK,
|
||||||
|
"beacon is officially stuck\n");
|
||||||
|
sc->sc_flags |= SC_OP_TSF_RESET;
|
||||||
|
- spin_lock(&sc->sc_pcu_lock);
|
||||||
|
- ath_reset(sc, true);
|
||||||
|
- spin_unlock(&sc->sc_pcu_lock);
|
||||||
|
+ ieee80211_queue_work(sc->hw, &sc->hw_reset_work);
|
||||||
|
}
|
||||||
|
|
||||||
|
return;
|
@ -1,6 +1,6 @@
|
|||||||
--- a/drivers/net/wireless/ath/ath9k/main.c
|
--- a/drivers/net/wireless/ath/ath9k/main.c
|
||||||
+++ b/drivers/net/wireless/ath/ath9k/main.c
|
+++ b/drivers/net/wireless/ath/ath9k/main.c
|
||||||
@@ -212,83 +212,47 @@ static int ath_update_survey_stats(struc
|
@@ -212,83 +212,57 @@ static int ath_update_survey_stats(struc
|
||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -11,33 +11,42 @@
|
|||||||
-*/
|
-*/
|
||||||
-static int ath_set_channel(struct ath_softc *sc, struct ieee80211_hw *hw,
|
-static int ath_set_channel(struct ath_softc *sc, struct ieee80211_hw *hw,
|
||||||
- struct ath9k_channel *hchan)
|
- struct ath9k_channel *hchan)
|
||||||
+static bool ath_prepare_reset(struct ath_softc *sc, bool retry_tx, bool flush)
|
+static void __ath_cancel_work(struct ath_softc *sc)
|
||||||
{
|
{
|
||||||
struct ath_hw *ah = sc->sc_ah;
|
- struct ath_hw *ah = sc->sc_ah;
|
||||||
struct ath_common *common = ath9k_hw_common(ah);
|
- struct ath_common *common = ath9k_hw_common(ah);
|
||||||
- struct ieee80211_conf *conf = &common->hw->conf;
|
- struct ieee80211_conf *conf = &common->hw->conf;
|
||||||
- bool fastcc = true, stopped;
|
- bool fastcc = true, stopped;
|
||||||
- struct ieee80211_channel *channel = hw->conf.channel;
|
- struct ieee80211_channel *channel = hw->conf.channel;
|
||||||
- struct ath9k_hw_cal_data *caldata = NULL;
|
- struct ath9k_hw_cal_data *caldata = NULL;
|
||||||
- int r;
|
- int r;
|
||||||
+ bool ret;
|
-
|
||||||
|
|
||||||
- if (sc->sc_flags & SC_OP_INVALID)
|
- if (sc->sc_flags & SC_OP_INVALID)
|
||||||
- return -EIO;
|
- return -EIO;
|
||||||
+ ieee80211_stop_queues(sc->hw);
|
|
||||||
|
|
||||||
sc->hw_busy_count = 0;
|
|
||||||
-
|
-
|
||||||
del_timer_sync(&common->ani.timer);
|
- sc->hw_busy_count = 0;
|
||||||
|
-
|
||||||
|
- del_timer_sync(&common->ani.timer);
|
||||||
cancel_work_sync(&sc->paprd_work);
|
cancel_work_sync(&sc->paprd_work);
|
||||||
cancel_work_sync(&sc->hw_check_work);
|
cancel_work_sync(&sc->hw_check_work);
|
||||||
cancel_delayed_work_sync(&sc->tx_complete_work);
|
cancel_delayed_work_sync(&sc->tx_complete_work);
|
||||||
cancel_delayed_work_sync(&sc->hw_pll_work);
|
cancel_delayed_work_sync(&sc->hw_pll_work);
|
||||||
|
+}
|
||||||
|
|
||||||
- ath9k_ps_wakeup(sc);
|
- ath9k_ps_wakeup(sc);
|
||||||
-
|
+static void ath_cancel_work(struct ath_softc *sc)
|
||||||
|
+{
|
||||||
|
+ __ath_cancel_work(sc);
|
||||||
|
+ cancel_work_sync(&sc->hw_reset_work);
|
||||||
|
+}
|
||||||
|
|
||||||
- spin_lock_bh(&sc->sc_pcu_lock);
|
- spin_lock_bh(&sc->sc_pcu_lock);
|
||||||
-
|
+static bool ath_prepare_reset(struct ath_softc *sc, bool retry_tx, bool flush)
|
||||||
|
+{
|
||||||
|
+ struct ath_hw *ah = sc->sc_ah;
|
||||||
|
+ struct ath_common *common = ath9k_hw_common(ah);
|
||||||
|
+ bool ret;
|
||||||
|
|
||||||
- /*
|
- /*
|
||||||
- * This is only performed if the channel settings have
|
- * This is only performed if the channel settings have
|
||||||
- * actually changed.
|
- * actually changed.
|
||||||
@ -47,20 +56,24 @@
|
|||||||
- * hardware at the new frequency, and then re-enable
|
- * hardware at the new frequency, and then re-enable
|
||||||
- * the relevant bits of the h/w.
|
- * the relevant bits of the h/w.
|
||||||
- */
|
- */
|
||||||
ath9k_hw_disable_interrupts(ah);
|
- ath9k_hw_disable_interrupts(ah);
|
||||||
- stopped = ath_drain_all_txq(sc, false);
|
- stopped = ath_drain_all_txq(sc, false);
|
||||||
-
|
+ ieee80211_stop_queues(sc->hw);
|
||||||
|
|
||||||
- if (!ath_stoprecv(sc))
|
- if (!ath_stoprecv(sc))
|
||||||
- stopped = false;
|
- stopped = false;
|
||||||
|
+ sc->hw_busy_count = 0;
|
||||||
|
+ del_timer_sync(&common->ani.timer);
|
||||||
|
|
||||||
- if (!ath9k_hw_check_alive(ah))
|
- if (!ath9k_hw_check_alive(ah))
|
||||||
- stopped = false;
|
- stopped = false;
|
||||||
+ ret = ath_drain_all_txq(sc, retry_tx);
|
+ ath9k_hw_disable_interrupts(ah);
|
||||||
|
|
||||||
- /* XXX: do not flush receive queue here. We don't want
|
- /* XXX: do not flush receive queue here. We don't want
|
||||||
- * to flush data frames already in queue because of
|
- * to flush data frames already in queue because of
|
||||||
- * changing channel. */
|
- * changing channel. */
|
||||||
-
|
+ ret = ath_drain_all_txq(sc, retry_tx);
|
||||||
|
|
||||||
- if (!stopped || !(sc->sc_flags & SC_OP_OFFCHANNEL))
|
- if (!stopped || !(sc->sc_flags & SC_OP_OFFCHANNEL))
|
||||||
- fastcc = false;
|
- fastcc = false;
|
||||||
+ if (!ath_stoprecv(sc))
|
+ if (!ath_stoprecv(sc))
|
||||||
@ -104,7 +117,7 @@
|
|||||||
}
|
}
|
||||||
|
|
||||||
ath9k_cmn_update_txpow(ah, sc->curtxpow,
|
ath9k_cmn_update_txpow(ah, sc->curtxpow,
|
||||||
@@ -296,21 +260,89 @@ static int ath_set_channel(struct ath_so
|
@@ -296,21 +270,93 @@ static int ath_set_channel(struct ath_so
|
||||||
ath9k_hw_set_interrupts(ah, ah->imask);
|
ath9k_hw_set_interrupts(ah, ah->imask);
|
||||||
ath9k_hw_enable_interrupts(ah);
|
ath9k_hw_enable_interrupts(ah);
|
||||||
|
|
||||||
@ -136,6 +149,10 @@
|
|||||||
+ bool flush = false;
|
+ bool flush = false;
|
||||||
+ int r;
|
+ int r;
|
||||||
+
|
+
|
||||||
|
+ __ath_cancel_work(sc);
|
||||||
|
+
|
||||||
|
+ spin_lock_bh(&sc->sc_pcu_lock);
|
||||||
|
|
||||||
+ if (!(sc->sc_flags & SC_OP_OFFCHANNEL)) {
|
+ if (!(sc->sc_flags & SC_OP_OFFCHANNEL)) {
|
||||||
+ fastcc = false;
|
+ fastcc = false;
|
||||||
+ caldata = &sc->caldata;
|
+ caldata = &sc->caldata;
|
||||||
@ -163,12 +180,14 @@
|
|||||||
+ if (r) {
|
+ if (r) {
|
||||||
+ ath_err(common,
|
+ ath_err(common,
|
||||||
+ "Unable to reset channel, reset status %d\n", r);
|
+ "Unable to reset channel, reset status %d\n", r);
|
||||||
+ return r;
|
+ goto out;
|
||||||
+ }
|
+ }
|
||||||
+
|
+
|
||||||
+ if (!ath_complete_reset(sc, true))
|
+ if (!ath_complete_reset(sc, true))
|
||||||
+ return -EIO;
|
+ r = -EIO;
|
||||||
+
|
+
|
||||||
|
+out:
|
||||||
|
spin_unlock_bh(&sc->sc_pcu_lock);
|
||||||
+ return 0;
|
+ return 0;
|
||||||
+}
|
+}
|
||||||
+
|
+
|
||||||
@ -182,22 +201,20 @@
|
|||||||
+ struct ath9k_channel *hchan)
|
+ struct ath9k_channel *hchan)
|
||||||
+{
|
+{
|
||||||
+ int r;
|
+ int r;
|
||||||
|
+
|
||||||
+ if (sc->sc_flags & SC_OP_INVALID)
|
+ if (sc->sc_flags & SC_OP_INVALID)
|
||||||
+ return -EIO;
|
+ return -EIO;
|
||||||
+
|
+
|
||||||
+ ath9k_ps_wakeup(sc);
|
+ ath9k_ps_wakeup(sc);
|
||||||
+
|
+
|
||||||
+ spin_lock_bh(&sc->sc_pcu_lock);
|
|
||||||
+ r = ath_reset_internal(sc, hchan, false);
|
+ r = ath_reset_internal(sc, hchan, false);
|
||||||
spin_unlock_bh(&sc->sc_pcu_lock);
|
|
||||||
|
|
||||||
ath9k_ps_restore(sc);
|
ath9k_ps_restore(sc);
|
||||||
+
|
+
|
||||||
return r;
|
return r;
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -893,28 +925,13 @@ static void ath_radio_enable(struct ath_
|
@@ -823,28 +869,13 @@ static void ath_radio_enable(struct ath_
|
||||||
channel->center_freq, r);
|
channel->center_freq, r);
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -227,20 +244,21 @@
|
|||||||
spin_unlock_bh(&sc->sc_pcu_lock);
|
spin_unlock_bh(&sc->sc_pcu_lock);
|
||||||
|
|
||||||
ath9k_ps_restore(sc);
|
ath9k_ps_restore(sc);
|
||||||
@@ -927,12 +944,8 @@ void ath_radio_disable(struct ath_softc
|
@@ -857,11 +888,10 @@ void ath_radio_disable(struct ath_softc
|
||||||
int r;
|
int r;
|
||||||
|
|
||||||
ath9k_ps_wakeup(sc);
|
ath9k_ps_wakeup(sc);
|
||||||
- cancel_delayed_work_sync(&sc->hw_pll_work);
|
- cancel_delayed_work_sync(&sc->hw_pll_work);
|
||||||
-
|
|
||||||
spin_lock_bh(&sc->sc_pcu_lock);
|
- spin_lock_bh(&sc->sc_pcu_lock);
|
||||||
|
+ ath_cancel_work(sc);
|
||||||
|
|
||||||
- ieee80211_stop_queues(hw);
|
- ieee80211_stop_queues(hw);
|
||||||
-
|
+ spin_lock_bh(&sc->sc_pcu_lock);
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* Keep the LED on when the radio is disabled
|
* Keep the LED on when the radio is disabled
|
||||||
* during idle unassociated state.
|
@@ -872,13 +902,7 @@ void ath_radio_disable(struct ath_softc
|
||||||
@@ -942,13 +955,7 @@ void ath_radio_disable(struct ath_softc
|
|
||||||
ath9k_hw_cfg_gpio_input(ah, ah->led_pin);
|
ath9k_hw_cfg_gpio_input(ah, ah->led_pin);
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -255,9 +273,9 @@
|
|||||||
|
|
||||||
if (!ah->curchan)
|
if (!ah->curchan)
|
||||||
ah->curchan = ath9k_cmn_get_curchannel(hw, ah);
|
ah->curchan = ath9k_cmn_get_curchannel(hw, ah);
|
||||||
@@ -970,48 +977,11 @@ void ath_radio_disable(struct ath_softc
|
@@ -900,48 +924,11 @@ void ath_radio_disable(struct ath_softc
|
||||||
|
|
||||||
int ath_reset(struct ath_softc *sc, bool retry_tx)
|
static int ath_reset(struct ath_softc *sc, bool retry_tx)
|
||||||
{
|
{
|
||||||
- struct ath_hw *ah = sc->sc_ah;
|
- struct ath_hw *ah = sc->sc_ah;
|
||||||
- struct ath_common *common = ath9k_hw_common(ah);
|
- struct ath_common *common = ath9k_hw_common(ah);
|
||||||
@ -305,7 +323,7 @@
|
|||||||
|
|
||||||
if (retry_tx) {
|
if (retry_tx) {
|
||||||
int i;
|
int i;
|
||||||
@@ -1024,12 +994,6 @@ int ath_reset(struct ath_softc *sc, bool
|
@@ -954,12 +941,6 @@ static int ath_reset(struct ath_softc *s
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -318,7 +336,42 @@
|
|||||||
ath9k_ps_restore(sc);
|
ath9k_ps_restore(sc);
|
||||||
|
|
||||||
return r;
|
return r;
|
||||||
@@ -1081,28 +1045,6 @@ static int ath9k_start(struct ieee80211_
|
@@ -969,9 +950,7 @@ void ath_reset_work(struct work_struct *
|
||||||
|
{
|
||||||
|
struct ath_softc *sc = container_of(work, struct ath_softc, hw_check_work);
|
||||||
|
|
||||||
|
- spin_lock_bh(&sc->sc_pcu_lock);
|
||||||
|
ath_reset(sc, true);
|
||||||
|
- spin_unlock_bh(&sc->sc_pcu_lock);
|
||||||
|
}
|
||||||
|
|
||||||
|
void ath_hw_check(struct work_struct *work)
|
||||||
|
@@ -992,11 +971,8 @@ void ath_hw_check(struct work_struct *wo
|
||||||
|
ath_dbg(common, ATH_DBG_RESET, "Possible baseband hang, "
|
||||||
|
"busy=%d (try %d)\n", busy, sc->hw_busy_count + 1);
|
||||||
|
if (busy >= 99) {
|
||||||
|
- if (++sc->hw_busy_count >= 3) {
|
||||||
|
- spin_lock_bh(&sc->sc_pcu_lock);
|
||||||
|
- ath_reset(sc, true);
|
||||||
|
- spin_unlock_bh(&sc->sc_pcu_lock);
|
||||||
|
- }
|
||||||
|
+ if (++sc->hw_busy_count >= 3)
|
||||||
|
+ ieee80211_queue_work(sc->hw, &sc->hw_reset_work);
|
||||||
|
|
||||||
|
} else if (busy >= 0)
|
||||||
|
sc->hw_busy_count = 0;
|
||||||
|
@@ -1016,9 +992,7 @@ static void ath_hw_pll_rx_hang_check(str
|
||||||
|
/* Rx is hung for more than 500ms. Reset it */
|
||||||
|
ath_dbg(common, ATH_DBG_RESET,
|
||||||
|
"Possible RX hang, resetting");
|
||||||
|
- spin_lock_bh(&sc->sc_pcu_lock);
|
||||||
|
- ath_reset(sc, true);
|
||||||
|
- spin_unlock_bh(&sc->sc_pcu_lock);
|
||||||
|
+ ieee80211_queue_work(sc->hw, &sc->hw_reset_work);
|
||||||
|
count = 0;
|
||||||
|
}
|
||||||
|
} else
|
||||||
|
@@ -1089,28 +1063,6 @@ static int ath9k_start(struct ieee80211_
|
||||||
goto mutex_unlock;
|
goto mutex_unlock;
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -347,7 +400,7 @@
|
|||||||
/* Setup our intr mask. */
|
/* Setup our intr mask. */
|
||||||
ah->imask = ATH9K_INT_TX | ATH9K_INT_RXEOL |
|
ah->imask = ATH9K_INT_TX | ATH9K_INT_RXEOL |
|
||||||
ATH9K_INT_RXORN | ATH9K_INT_FATAL |
|
ATH9K_INT_RXORN | ATH9K_INT_FATAL |
|
||||||
@@ -1125,12 +1067,14 @@ static int ath9k_start(struct ieee80211_
|
@@ -1133,12 +1085,14 @@ static int ath9k_start(struct ieee80211_
|
||||||
|
|
||||||
/* Disable BMISS interrupt when we're not associated */
|
/* Disable BMISS interrupt when we're not associated */
|
||||||
ah->imask &= ~(ATH9K_INT_SWBA | ATH9K_INT_BMISS);
|
ah->imask &= ~(ATH9K_INT_SWBA | ATH9K_INT_BMISS);
|
||||||
@ -366,3 +419,28 @@
|
|||||||
|
|
||||||
if ((ah->btcoex_hw.scheme != ATH_BTCOEX_CFG_NONE) &&
|
if ((ah->btcoex_hw.scheme != ATH_BTCOEX_CFG_NONE) &&
|
||||||
!ah->btcoex_hw.enabled) {
|
!ah->btcoex_hw.enabled) {
|
||||||
|
@@ -1231,10 +1185,7 @@ static void ath9k_stop(struct ieee80211_
|
||||||
|
|
||||||
|
mutex_lock(&sc->mutex);
|
||||||
|
|
||||||
|
- cancel_delayed_work_sync(&sc->tx_complete_work);
|
||||||
|
- cancel_delayed_work_sync(&sc->hw_pll_work);
|
||||||
|
- cancel_work_sync(&sc->paprd_work);
|
||||||
|
- cancel_work_sync(&sc->hw_check_work);
|
||||||
|
+ ath_cancel_work(sc);
|
||||||
|
|
||||||
|
if (sc->sc_flags & SC_OP_INVALID) {
|
||||||
|
ath_dbg(common, ATH_DBG_ANY, "Device not present\n");
|
||||||
|
@@ -2351,9 +2302,11 @@ static void ath9k_flush(struct ieee80211
|
||||||
|
ath9k_ps_wakeup(sc);
|
||||||
|
spin_lock_bh(&sc->sc_pcu_lock);
|
||||||
|
drain_txq = ath_drain_all_txq(sc, false);
|
||||||
|
+ spin_unlock_bh(&sc->sc_pcu_lock);
|
||||||
|
+
|
||||||
|
if (!drain_txq)
|
||||||
|
ath_reset(sc, false);
|
||||||
|
- spin_unlock_bh(&sc->sc_pcu_lock);
|
||||||
|
+
|
||||||
|
ath9k_ps_restore(sc);
|
||||||
|
ieee80211_wake_queues(hw);
|
||||||
|
|
@ -57,7 +57,7 @@
|
|||||||
}
|
}
|
||||||
--- a/drivers/net/wireless/ath/ath9k/ath9k.h
|
--- a/drivers/net/wireless/ath/ath9k/ath9k.h
|
||||||
+++ b/drivers/net/wireless/ath/ath9k/ath9k.h
|
+++ b/drivers/net/wireless/ath/ath9k/ath9k.h
|
||||||
@@ -652,6 +652,7 @@ struct ath_softc {
|
@@ -654,6 +654,7 @@ struct ath_softc {
|
||||||
struct ath_descdma txsdma;
|
struct ath_descdma txsdma;
|
||||||
|
|
||||||
struct ath_ant_comb ant_comb;
|
struct ath_ant_comb ant_comb;
|
||||||
@ -65,7 +65,7 @@
|
|||||||
};
|
};
|
||||||
|
|
||||||
void ath9k_tasklet(unsigned long data);
|
void ath9k_tasklet(unsigned long data);
|
||||||
@@ -673,6 +674,7 @@ int ath9k_init_device(u16 devid, struct
|
@@ -674,6 +675,7 @@ int ath9k_init_device(u16 devid, struct
|
||||||
const struct ath_bus_ops *bus_ops);
|
const struct ath_bus_ops *bus_ops);
|
||||||
void ath9k_deinit_device(struct ath_softc *sc);
|
void ath9k_deinit_device(struct ath_softc *sc);
|
||||||
void ath9k_set_hw_capab(struct ath_softc *sc, struct ieee80211_hw *hw);
|
void ath9k_set_hw_capab(struct ath_softc *sc, struct ieee80211_hw *hw);
|
||||||
@ -75,7 +75,7 @@
|
|||||||
bool ath9k_uses_beacons(int type);
|
bool ath9k_uses_beacons(int type);
|
||||||
--- a/drivers/net/wireless/ath/ath9k/main.c
|
--- a/drivers/net/wireless/ath/ath9k/main.c
|
||||||
+++ b/drivers/net/wireless/ath/ath9k/main.c
|
+++ b/drivers/net/wireless/ath/ath9k/main.c
|
||||||
@@ -270,6 +270,22 @@ static bool ath_complete_reset(struct at
|
@@ -280,6 +280,22 @@ static bool ath_complete_reset(struct at
|
||||||
ath_start_ani(common);
|
ath_start_ani(common);
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -98,7 +98,7 @@
|
|||||||
ieee80211_wake_queues(sc->hw);
|
ieee80211_wake_queues(sc->hw);
|
||||||
|
|
||||||
return true;
|
return true;
|
||||||
@@ -2366,6 +2382,59 @@ static int ath9k_get_stats(struct ieee80
|
@@ -2383,6 +2399,59 @@ static int ath9k_get_stats(struct ieee80
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -158,7 +158,7 @@
|
|||||||
struct ieee80211_ops ath9k_ops = {
|
struct ieee80211_ops ath9k_ops = {
|
||||||
.tx = ath9k_tx,
|
.tx = ath9k_tx,
|
||||||
.start = ath9k_start,
|
.start = ath9k_start,
|
||||||
@@ -2392,4 +2461,6 @@ struct ieee80211_ops ath9k_ops = {
|
@@ -2409,4 +2478,6 @@ struct ieee80211_ops ath9k_ops = {
|
||||||
.tx_frames_pending = ath9k_tx_frames_pending,
|
.tx_frames_pending = ath9k_tx_frames_pending,
|
||||||
.tx_last_beacon = ath9k_tx_last_beacon,
|
.tx_last_beacon = ath9k_tx_last_beacon,
|
||||||
.get_stats = ath9k_get_stats,
|
.get_stats = ath9k_get_stats,
|
Loading…
Reference in New Issue
Block a user