Commit 4286df60 authored by Felix Fietkau's avatar Felix Fietkau Committed by John W. Linville

ath9k: do not call ath9k_hw_txprocdesc on AR9003 outside of the tx tasklet

Since AR9003 uses a global tx status queue, processing tx status outside of
the regular tx tasklet is dangerous and messes up hardware/software
synchronization of tx status events.
Signed-off-by: default avatarFelix Fietkau <nbd@openwrt.org>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent d6157bf7
...@@ -2300,6 +2300,7 @@ static int ath9k_tx_last_beacon(struct ieee80211_hw *hw) ...@@ -2300,6 +2300,7 @@ static int ath9k_tx_last_beacon(struct ieee80211_hw *hw)
struct ath_vif *avp; struct ath_vif *avp;
struct ath_buf *bf; struct ath_buf *bf;
struct ath_tx_status ts; struct ath_tx_status ts;
bool edma = !!(ah->caps.hw_caps & ATH9K_HW_CAP_EDMA);
int status; int status;
vif = sc->beacon.bslot[0]; vif = sc->beacon.bslot[0];
...@@ -2310,7 +2311,7 @@ static int ath9k_tx_last_beacon(struct ieee80211_hw *hw) ...@@ -2310,7 +2311,7 @@ static int ath9k_tx_last_beacon(struct ieee80211_hw *hw)
if (!avp->is_bslot_active) if (!avp->is_bslot_active)
return 0; return 0;
if (!sc->beacon.tx_processed) { if (!sc->beacon.tx_processed && !edma) {
tasklet_disable(&sc->bcon_tasklet); tasklet_disable(&sc->bcon_tasklet);
bf = avp->av_bcbuf; bf = avp->av_bcbuf;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment