wifi: mt76: mt7921: Reset ampdu_state state in case of failure in mt76_connac2_tx_che...

Reset ampdu_state if ieee80211_start_tx_ba_session() fails in
mt76_connac2_tx_check_aggr(), otherwise the driver may incorrectly
assume aggregation is active and skip future BA setup attempts.

Fixes: 163f4d22c118 ("mt76: mt7921: add MAC support")
Signed-off-by: Sean Wang <sean.wang@mediatek.com>
Link: https://patch.msgid.link/20251216005930.9412-1-sean.wang@kernel.org
Signed-off-by: Felix Fietkau <nbd@nbd.name>

diff --git a/drivers/net/wireless/mediatek/mt76/mt76_connac_mac.c b/drivers/net/wireless/mediatek/mt76/mt76_connac_mac.c
index f946ddc20a479b1c9637b66c3780df757825f395..c466912485135fbb4e838645d55d0686a6b64ddc 100644 (file)
--- a/drivers/net/wireless/mediatek/mt76/mt76_connac_mac.c
+++ b/drivers/net/wireless/mediatek/mt76/mt76_connac_mac.c
@@ -1153,8 +1153,10 @@ void mt76_connac2_tx_check_aggr(struct ieee80211_sta *sta, __le32 *txwi)
                return;
 
        wcid = (struct mt76_wcid *)sta->drv_priv;
-       if (!test_and_set_bit(tid, &wcid->ampdu_state))
-               ieee80211_start_tx_ba_session(sta, tid, 0);
+       if (!test_and_set_bit(tid, &wcid->ampdu_state)) {
+               if (ieee80211_start_tx_ba_session(sta, tid, 0))
+                       clear_bit(tid, &wcid->ampdu_state);
+       }
 }
 EXPORT_SYMBOL_GPL(mt76_connac2_tx_check_aggr);