wifi: mt76: mt7603: rely on shared poll_list field
authorLorenzo Bianconi <lorenzo@kernel.org>
Thu, 22 Jun 2023 16:50:24 +0000 (18:50 +0200)
committerFelix Fietkau <nbd@nbd.name>
Tue, 25 Jul 2023 19:59:42 +0000 (21:59 +0200)
Rely on poll_list field in mt76_wcid structure and get rid of
private copy.

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

index ac8a67e10a4d4d7edf1996424cf4ff0e2fb7044c..de11557eb04c3e9ca821db7d9e1dd62e056614e9 100644 (file)
@@ -419,8 +419,8 @@ void mt7603_mac_sta_poll(struct mt7603_dev *dev)
                }
 
                msta = list_first_entry(&dev->mt76.sta_poll_list,
-                                       struct mt7603_sta, poll_list);
-               list_del_init(&msta->poll_list);
+                                       struct mt7603_sta, wcid.poll_list);
+               list_del_init(&msta->wcid.poll_list);
                spin_unlock_bh(&dev->mt76.sta_poll_lock);
 
                addr = mt7603_wtbl4_addr(msta->wcid.idx);
@@ -1267,9 +1267,9 @@ void mt7603_mac_add_txs(struct mt7603_dev *dev, void *data)
        msta = container_of(wcid, struct mt7603_sta, wcid);
        sta = wcid_to_sta(wcid);
 
-       if (list_empty(&msta->poll_list)) {
+       if (list_empty(&msta->wcid.poll_list)) {
                spin_lock_bh(&dev->mt76.sta_poll_lock);
-               list_add_tail(&msta->poll_list, &dev->mt76.sta_poll_list);
+               list_add_tail(&msta->wcid.poll_list, &dev->mt76.sta_poll_list);
                spin_unlock_bh(&dev->mt76.sta_poll_lock);
        }
 
index 6d2295cf5753bf3f119453ddddcd6d4893c91b91..1d4893410ca56a090688b4e4597dcbfbf2ae3d92 100644 (file)
@@ -66,7 +66,7 @@ mt7603_add_interface(struct ieee80211_hw *hw, struct ieee80211_vif *vif)
 
        idx = MT7603_WTBL_RESERVED - 1 - mvif->idx;
        dev->mt76.vif_mask |= BIT_ULL(mvif->idx);
-       INIT_LIST_HEAD(&mvif->sta.poll_list);
+       INIT_LIST_HEAD(&mvif->sta.wcid.poll_list);
        mvif->sta.wcid.idx = idx;
        mvif->sta.wcid.hw_key_idx = -1;
        mt76_packet_id_init(&mvif->sta.wcid);
@@ -101,8 +101,8 @@ mt7603_remove_interface(struct ieee80211_hw *hw, struct ieee80211_vif *vif)
        rcu_assign_pointer(dev->mt76.wcid[idx], NULL);
 
        spin_lock_bh(&dev->mt76.sta_poll_lock);
-       if (!list_empty(&msta->poll_list))
-               list_del_init(&msta->poll_list);
+       if (!list_empty(&msta->wcid.poll_list))
+               list_del_init(&msta->wcid.poll_list);
        spin_unlock_bh(&dev->mt76.sta_poll_lock);
 
        mutex_lock(&dev->mt76.mutex);
@@ -351,7 +351,7 @@ mt7603_sta_add(struct mt76_dev *mdev, struct ieee80211_vif *vif,
        if (idx < 0)
                return -ENOSPC;
 
-       INIT_LIST_HEAD(&msta->poll_list);
+       INIT_LIST_HEAD(&msta->wcid.poll_list);
        __skb_queue_head_init(&msta->psq);
        msta->ps = ~0;
        msta->smps = ~0;
@@ -389,8 +389,8 @@ mt7603_sta_remove(struct mt76_dev *mdev, struct ieee80211_vif *vif,
        spin_unlock_bh(&dev->ps_lock);
 
        spin_lock_bh(&mdev->sta_poll_lock);
-       if (!list_empty(&msta->poll_list))
-               list_del_init(&msta->poll_list);
+       if (!list_empty(&msta->wcid.poll_list))
+               list_del_init(&msta->wcid.poll_list);
        spin_unlock_bh(&mdev->sta_poll_lock);
 
        mt7603_wtbl_clear(dev, wcid->idx);
index 689922d133e7870bacc5ce3d8369614b17fd86d0..354b189862f797eae27ea99ab8aba087721d6662 100644 (file)
@@ -64,7 +64,6 @@ struct mt7603_sta {
 
        struct mt7603_vif *vif;
 
-       struct list_head poll_list;
        u32 tx_airtime_ac[4];
 
        struct sk_buff_head psq;