OSDN Git Service

mt76: remove mt76_wcid pointer from mt76_tx_status_check signature
authorLorenzo Bianconi <lorenzo@kernel.org>
Mon, 13 Sep 2021 09:25:03 +0000 (11:25 +0200)
committerFelix Fietkau <nbd@nbd.name>
Wed, 20 Oct 2021 08:36:47 +0000 (10:36 +0200)
Remove mt76_wcid pointer from mt76_tx_status_check signature since it is
always set to NULL

Tested-by: mrkiko.rs@gmail.com
Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org>
Signed-off-by: Felix Fietkau <nbd@nbd.name>
14 files changed:
drivers/net/wireless/mediatek/mt76/mac80211.c
drivers/net/wireless/mediatek/mt76/mt76.h
drivers/net/wireless/mediatek/mt76/mt7603/mac.c
drivers/net/wireless/mediatek/mt76/mt7615/mac.c
drivers/net/wireless/mediatek/mt76/mt7615/pci_mac.c
drivers/net/wireless/mediatek/mt76/mt7615/sdio.c
drivers/net/wireless/mediatek/mt76/mt76x02_mac.c
drivers/net/wireless/mediatek/mt76/mt76x02_mmio.c
drivers/net/wireless/mediatek/mt76/mt7915/mac.c
drivers/net/wireless/mediatek/mt76/mt7921/dma.c
drivers/net/wireless/mediatek/mt76/mt7921/mac.c
drivers/net/wireless/mediatek/mt76/sdio.c
drivers/net/wireless/mediatek/mt76/tx.c
drivers/net/wireless/mediatek/mt76/usb.c

index ddc692d..61fff98 100644 (file)
@@ -511,7 +511,7 @@ void mt76_unregister_phy(struct mt76_phy *phy)
 {
        struct mt76_dev *dev = phy->dev;
 
-       mt76_tx_status_check(dev, NULL, true);
+       mt76_tx_status_check(dev, true);
        ieee80211_unregister_hw(phy->hw);
        dev->phy2 = NULL;
 }
@@ -639,7 +639,7 @@ void mt76_unregister_device(struct mt76_dev *dev)
 
        if (IS_ENABLED(CONFIG_MT76_LEDS))
                mt76_led_cleanup(dev);
-       mt76_tx_status_check(dev, NULL, true);
+       mt76_tx_status_check(dev, true);
        ieee80211_unregister_hw(hw);
 }
 EXPORT_SYMBOL_GPL(mt76_unregister_device);
index 7bd82cf..1c18e75 100644 (file)
@@ -1116,8 +1116,7 @@ mt76_tx_complete_skb(struct mt76_dev *dev, u16 wcid, struct sk_buff *skb)
     __mt76_tx_complete_skb(dev, wcid, skb, NULL);
 }
 
-void mt76_tx_status_check(struct mt76_dev *dev, struct mt76_wcid *wcid,
-                         bool flush);
+void mt76_tx_status_check(struct mt76_dev *dev, bool flush);
 int mt76_sta_state(struct ieee80211_hw *hw, struct ieee80211_vif *vif,
                   struct ieee80211_sta *sta,
                   enum ieee80211_sta_state old_state,
index d1cbe06..fe03e31 100644 (file)
@@ -1458,7 +1458,7 @@ static void mt7603_mac_watchdog_reset(struct mt7603_dev *dev)
                mt76_queue_rx_reset(dev, i);
        }
 
-       mt76_tx_status_check(&dev->mt76, NULL, true);
+       mt76_tx_status_check(&dev->mt76, true);
 
        mt7603_dma_sched_reset(dev);
 
@@ -1817,7 +1817,7 @@ void mt7603_mac_work(struct work_struct *work)
        bool reset = false;
        int i, idx;
 
-       mt76_tx_status_check(&dev->mt76, NULL, false);
+       mt76_tx_status_check(&dev->mt76, false);
 
        mutex_lock(&dev->mt76.mutex);
 
index 5455231..423f690 100644 (file)
@@ -2026,7 +2026,7 @@ void mt7615_mac_work(struct work_struct *work)
 
        mt7615_mutex_release(phy->dev);
 
-       mt76_tx_status_check(mphy->dev, NULL, false);
+       mt76_tx_status_check(mphy->dev, false);
 
        timeout = mt7615_get_macwork_timeout(phy->dev);
        ieee80211_queue_delayed_work(mphy->hw, &mphy->mac_work, timeout);
index 0a4cadd..5ee52cd 100644 (file)
@@ -198,7 +198,7 @@ void mt7615_dma_reset(struct mt7615_dev *dev)
        mt76_for_each_q_rx(&dev->mt76, i)
                mt76_queue_rx_reset(dev, i);
 
-       mt76_tx_status_check(&dev->mt76, NULL, true);
+       mt76_tx_status_check(&dev->mt76, true);
 
        mt7615_dma_start(dev);
 }
index 305bb85..fb0f8f3 100644 (file)
@@ -432,7 +432,7 @@ static int mt7663s_suspend(struct device *dev)
        cancel_work_sync(&mdev->mt76.sdio.stat_work);
        clear_bit(MT76_READING_STATS, &mdev->mphy.state);
 
-       mt76_tx_status_check(&mdev->mt76, NULL, true);
+       mt76_tx_status_check(&mdev->mt76, true);
 
        return 0;
 }
index 07b21b2..a404fd7 100644 (file)
@@ -1186,7 +1186,7 @@ void mt76x02_mac_work(struct work_struct *work)
 
        mutex_unlock(&dev->mt76.mutex);
 
-       mt76_tx_status_check(&dev->mt76, NULL, false);
+       mt76_tx_status_check(&dev->mt76, false);
 
        ieee80211_queue_delayed_work(mt76_hw(dev), &dev->mphy.mac_work,
                                     MT_MAC_WORK_INTERVAL);
index 468be65..ec0de69 100644 (file)
@@ -472,7 +472,7 @@ static void mt76x02_watchdog_reset(struct mt76x02_dev *dev)
                mt76_queue_rx_reset(dev, i);
        }
 
-       mt76_tx_status_check(&dev->mt76, NULL, true);
+       mt76_tx_status_check(&dev->mt76, true);
 
        mt76x02_mac_start(dev);
 
index 3e69919..4877799 100644 (file)
@@ -1792,7 +1792,7 @@ mt7915_dma_reset(struct mt7915_dev *dev)
        mt76_for_each_q_rx(&dev->mt76, i)
                mt76_queue_rx_reset(dev, i);
 
-       mt76_tx_status_check(&dev->mt76, NULL, true);
+       mt76_tx_status_check(&dev->mt76, true);
 
        /* re-init prefetch settings after reset */
        mt7915_dma_prefetch(dev);
@@ -2105,7 +2105,7 @@ void mt7915_mac_work(struct work_struct *work)
 
        mutex_unlock(&mphy->dev->mutex);
 
-       mt76_tx_status_check(mphy->dev, NULL, false);
+       mt76_tx_status_check(mphy->dev, false);
 
        ieee80211_queue_delayed_work(mphy->hw, &mphy->mac_work,
                                     MT7915_WATCHDOG_TIME);
index 62cee66..802e40e 100644 (file)
@@ -272,7 +272,7 @@ static int mt7921_dma_reset(struct mt7921_dev *dev, bool force)
        mt76_for_each_q_rx(&dev->mt76, i)
                mt76_queue_reset(dev, &dev->mt76.q_rx[i]);
 
-       mt76_tx_status_check(&dev->mt76, NULL, true);
+       mt76_tx_status_check(&dev->mt76, true);
 
        return mt7921_dma_enable(dev);
 }
index 14024ae..0659ff2 100644 (file)
@@ -1694,7 +1694,7 @@ void mt7921_mac_work(struct work_struct *work)
 
        mt7921_mutex_release(phy->dev);
 
-       mt76_tx_status_check(mphy->dev, NULL, false);
+       mt76_tx_status_check(mphy->dev, false);
        ieee80211_queue_delayed_work(phy->mt76->hw, &mphy->mac_work,
                                     MT7921_WATCHDOG_TIME);
 }
index 783a156..4abf432 100644 (file)
@@ -328,7 +328,7 @@ void mt76s_deinit(struct mt76_dev *dev)
        cancel_work_sync(&sdio->stat_work);
        clear_bit(MT76_READING_STATS, &dev->phy.state);
 
-       mt76_tx_status_check(dev, NULL, true);
+       mt76_tx_status_check(dev, true);
 
        sdio_claim_host(sdio->func);
        sdio_release_irq(sdio->func);
index 15916f6..8b4c6ce 100644 (file)
@@ -193,10 +193,10 @@ out:
 EXPORT_SYMBOL_GPL(mt76_tx_status_skb_get);
 
 void
-mt76_tx_status_check(struct mt76_dev *dev, struct mt76_wcid *wcid, bool flush)
+mt76_tx_status_check(struct mt76_dev *dev, bool flush)
 {
+       struct mt76_wcid *wcid, *tmp;
        struct sk_buff_head list;
-       struct mt76_wcid *tmp;
 
        mt76_tx_status_lock(dev, &list);
        list_for_each_entry_safe(wcid, tmp, &dev->wcid_list, list)
index 1e9f60b..0a7006c 100644 (file)
@@ -1081,7 +1081,7 @@ void mt76u_stop_tx(struct mt76_dev *dev)
 
        mt76_worker_enable(&dev->usb.status_worker);
 
-       mt76_tx_status_check(dev, NULL, true);
+       mt76_tx_status_check(dev, true);
 }
 EXPORT_SYMBOL_GPL(mt76u_stop_tx);