@@ -159,12 +159,10 @@ enum {
#define MT_TXS4_TIMESTAMP GENMASK(31, 0)
/* PPDU based TXS */
-#define MT_TXS5_MPDU_TX_BYTE GENMASK(22, 0)
-#define MT_TXS5_MPDU_TX_CNT GENMASK(31, 23)
-
-#define MT_TXS6_MPDU_FAIL_CNT GENMASK(31, 23)
-
-#define MT_TXS7_MPDU_RETRY_CNT GENMASK(31, 23)
+#define MT_TXS5_MPDU_TX_BYTE GENMASK(23, 0)
+#define MT_TXS5_MPDU_TX_CNT GENMASK(31, 24)
+#define MT_TXS6_MPDU_FAIL_CNT GENMASK(31, 24)
+#define MT_TXS7_MPDU_RETRY_CNT GENMASK(31, 24)
/* RXD DW1 */
#define MT_RXD1_NORMAL_WLAN_IDX GENMASK(9, 0)
@@ -1061,9 +1061,6 @@ static void mt7915_mac_add_txs(struct mt7915_dev *dev, void *data)
u16 wcidx;
u8 pid;
- if (le32_get_bits(txs_data[0], MT_TXS0_TXS_FORMAT) > 1)
- return;
-
wcidx = le32_get_bits(txs_data[2], MT_TXS2_WCID);
pid = le32_get_bits(txs_data[3], MT_TXS3_PID);
The previous commit forgot to remove a leftover check and set wrong bitmask that led to unexpected tx_stat reporting. Fixes: 43eaa3689507 ("wifi: mt76: add PPDU based TxS support for WED device") Reported-By: Sujuan Chen <sujuan.chen@mediatek.com> Signed-off-by: Ryder Lee <ryder.lee@mediatek.com> --- changes since v2 - correct bitmask --- drivers/net/wireless/mediatek/mt76/mt76_connac2_mac.h | 10 ++++------ drivers/net/wireless/mediatek/mt76/mt7915/mac.c | 3 --- 2 files changed, 4 insertions(+), 9 deletions(-)