mt76: connac: move mt76_connac_tx_free in shared code
authorLorenzo Bianconi <lorenzo@kernel.org>
Sun, 19 Jun 2022 20:42:38 +0000 (22:42 +0200)
committerFelix Fietkau <nbd@nbd.name>
Mon, 11 Jul 2022 11:40:02 +0000 (13:40 +0200)
Move mt76_connac_tx_free structure in mt76_connac module since it is
shared by mt7615 and mt7921 drivers.

Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org>
Signed-off-by: Felix Fietkau <nbd@nbd.name>
drivers/net/wireless/mediatek/mt76/mt7615/mac.c
drivers/net/wireless/mediatek/mt76/mt7615/mac.h
drivers/net/wireless/mediatek/mt76/mt76_connac.h
drivers/net/wireless/mediatek/mt76/mt7921/mac.h
drivers/net/wireless/mediatek/mt76/mt7921/pci_mac.c

index de809cbf07335a5c9288cbb9cda4a6fa7ff90fa2..06d17e07034dc9c66d6bac812c445977d9e8db6a 100644 (file)
@@ -1638,7 +1638,8 @@ mt7615_mac_tx_free_token(struct mt7615_dev *dev, u16 token)
 
 static void mt7615_mac_tx_free(struct mt7615_dev *dev, void *data, int len)
 {
-       struct mt7615_tx_free *free = (struct mt7615_tx_free *)data;
+       struct mt76_connac_tx_free *free = data;
+       void *tx_token = data + sizeof(*free);
        void *end = data + len;
        u8 i, count;
 
@@ -1652,7 +1653,7 @@ static void mt7615_mac_tx_free(struct mt7615_dev *dev, void *data, int len)
 
        count = le16_get_bits(free->ctrl, MT_TX_FREE_MSDU_ID_CNT);
        if (is_mt7615(&dev->mt76)) {
-               __le16 *token = &free->token[0];
+               __le16 *token = tx_token;
 
                if (WARN_ON_ONCE((void *)&token[count] > end))
                        return;
@@ -1660,7 +1661,7 @@ static void mt7615_mac_tx_free(struct mt7615_dev *dev, void *data, int len)
                for (i = 0; i < count; i++)
                        mt7615_mac_tx_free_token(dev, le16_to_cpu(token[i]));
        } else {
-               __le32 *token = (__le32 *)&free->token[0];
+               __le32 *token = tx_token;
 
                if (WARN_ON_ONCE((void *)&token[count] > end))
                        return;
index 310c187a1a2920c257f90ca633470d6edb80f875..6af15f2e0f5a5880fb199d7778716457ee37229e 100644 (file)
@@ -252,14 +252,6 @@ enum tx_phy_bandwidth {
 /* mt7663 */
 #define MT_TXD_LEN_LAST                        BIT(15)
 
-struct mt7615_tx_free {
-       __le16 rx_byte_cnt;
-       __le16 ctrl;
-       u8 txd_cnt;
-       u8 rsv[3];
-       __le16 token[];
-} __packed __aligned(4);
-
 #define MT_TX_FREE_MSDU_ID_CNT         GENMASK(6, 0)
 
 #define MT_TXS0_PID                    GENMASK(31, 24)
index 4357fea1d79b3f01771b35526129c5fc80f0d8ed..0ef10895fbf8e7c8b49abf159c22dcaa5ff773c7 100644 (file)
@@ -136,6 +136,13 @@ struct mt76_connac_txp_common {
        };
 };
 
+struct mt76_connac_tx_free {
+       __le16 rx_byte_cnt;
+       __le16 ctrl;
+       u8 txd_cnt;
+       u8 rsv[3];
+} __packed __aligned(4);
+
 extern const struct wiphy_wowlan_support mt76_connac_wowlan_support;
 
 static inline bool is_mt7922(struct mt76_dev *dev)
index dc98d2c82988900c1f43a22b43c68341b0dc26c1..26b1ec35cc2b735374bc53f89b2db8552e84e0f1 100644 (file)
@@ -42,14 +42,6 @@ enum tx_mcu_port_q_idx {
        MT_TX_MCU_PORT_RX_FWDL = 0x3e
 };
 
-struct mt7921_tx_free {
-       __le16 rx_byte_cnt;
-       __le16 ctrl;
-       u8 txd_cnt;
-       u8 rsv[3];
-       __le32 info[];
-} __packed __aligned(4);
-
 #define MT_TX_FREE_MSDU_CNT            GENMASK(9, 0)
 #define MT_TX_FREE_WLAN_ID             GENMASK(23, 14)
 #define MT_TX_FREE_LATENCY             GENMASK(12, 0)
index bf58dcf1b98015c3606114ff77a4e2ef677c2237..333390cb791be66777f9312e0a3d0b5f105516e4 100644 (file)
@@ -150,7 +150,8 @@ out:
 static void
 mt7921e_mac_tx_free(struct mt7921_dev *dev, void *data, int len)
 {
-       struct mt7921_tx_free *free = (struct mt7921_tx_free *)data;
+       struct mt76_connac_tx_free *free = data;
+       __le32 *tx_info = (__le32 *)(data + sizeof(*free));
        struct mt76_dev *mdev = &dev->mt76;
        struct mt76_txwi_cache *txwi;
        struct ieee80211_sta *sta = NULL;
@@ -165,11 +166,11 @@ mt7921e_mac_tx_free(struct mt7921_dev *dev, void *data, int len)
        mt76_queue_tx_cleanup(dev, dev->mphy.q_tx[MT_TXQ_BE], false);
 
        count = le16_get_bits(free->ctrl, MT_TX_FREE_MSDU_CNT);
-       if (WARN_ON_ONCE((void *)&free->info[count] > end))
+       if (WARN_ON_ONCE((void *)&tx_info[count] > end))
                return;
 
        for (i = 0; i < count; i++) {
-               u32 msdu, info = le32_to_cpu(free->info[i]);
+               u32 msdu, info = le32_to_cpu(tx_info[i]);
                u8 stat;
 
                /* 1'b1: new wcid pair.