mt76: move beacon_mask in mt76_dev
authorLorenzo Bianconi <lorenzo@kernel.org>
Mon, 29 Apr 2019 08:13:00 +0000 (10:13 +0200)
committerFelix Fietkau <nbd@nbd.name>
Wed, 1 May 2019 11:04:00 +0000 (13:04 +0200)
Move beacon_mask in mt76_dev data structure since it is used by
all drivers

Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org>
Signed-off-by: Felix Fietkau <nbd@nbd.name>
drivers/net/wireless/mediatek/mt76/mt76.h
drivers/net/wireless/mediatek/mt76/mt7603/beacon.c
drivers/net/wireless/mediatek/mt76/mt7603/mt7603.h
drivers/net/wireless/mediatek/mt76/mt76x02.h
drivers/net/wireless/mediatek/mt76/mt76x02_beacon.c
drivers/net/wireless/mediatek/mt76/mt76x02_mac.c
drivers/net/wireless/mediatek/mt76/mt76x02_mmio.c
drivers/net/wireless/mediatek/mt76/mt76x02_usb_core.c

index 7a3172321c759c558c1431748006fdcfffe93ca4..f1f56d24e8fc3c51da5e65bf26f7a82a87f9ba10 100644 (file)
@@ -470,6 +470,7 @@ struct mt76_dev {
        u16 chainmask;
 
        int beacon_int;
+       u8 beacon_mask;
 
        struct mt76_sband sband_2g;
        struct mt76_sband sband_5g;
index 64e15d5662834d67013567db3986a1ab50911142..f3e7406e731f3e349320ca552de74ad7cc00f328 100644 (file)
@@ -16,7 +16,7 @@ mt7603_update_beacon_iter(void *priv, u8 *mac, struct ieee80211_vif *vif)
        struct mt7603_vif *mvif = (struct mt7603_vif *)vif->drv_priv;
        struct sk_buff *skb = NULL;
 
-       if (!(dev->beacon_mask & BIT(mvif->idx)))
+       if (!(dev->mt76.beacon_mask & BIT(mvif->idx)))
                return;
 
        skb = ieee80211_beacon_get(mt76_hw(dev), vif);
@@ -48,7 +48,7 @@ mt7603_add_buffered_bc(void *priv, u8 *mac, struct ieee80211_vif *vif)
        struct ieee80211_tx_info *info;
        struct sk_buff *skb;
 
-       if (!(dev->beacon_mask & BIT(mvif->idx)))
+       if (!(dev->mt76.beacon_mask & BIT(mvif->idx)))
                return;
 
        skb = ieee80211_get_buffered_bc(mt76_hw(dev), vif);
@@ -134,7 +134,7 @@ void mt7603_pre_tbtt_tasklet(unsigned long arg)
 out:
        mt76_queue_tx_cleanup(dev, MT_TXQ_BEACON, false);
        if (dev->mt76.q_tx[MT_TXQ_BEACON].q->queued >
-           hweight8(dev->beacon_mask))
+           hweight8(dev->mt76.beacon_mask))
                dev->beacon_check++;
 }
 
@@ -144,12 +144,12 @@ void mt7603_beacon_set_timer(struct mt7603_dev *dev, int idx, int intval)
 
        if (idx >= 0) {
                if (intval)
-                       dev->beacon_mask |= BIT(idx);
+                       dev->mt76.beacon_mask |= BIT(idx);
                else
-                       dev->beacon_mask &= ~BIT(idx);
+                       dev->mt76.beacon_mask &= ~BIT(idx);
        }
 
-       if (!dev->beacon_mask || (!intval && idx < 0)) {
+       if (!dev->mt76.beacon_mask || (!intval && idx < 0)) {
                mt7603_irq_disable(dev, MT_INT_MAC_IRQ3);
                mt76_clear(dev, MT_ARB_SCR, MT_ARB_SCR_BCNQ_OPMODE_MASK);
                mt76_wr(dev, MT_HW_INT_MASK(3), 0);
@@ -174,10 +174,11 @@ void mt7603_beacon_set_timer(struct mt7603_dev *dev, int idx, int intval)
 
        mt76_set(dev, MT_WF_ARB_BCN_START,
                 MT_WF_ARB_BCN_START_BSSn(0) |
-                ((dev->beacon_mask >> 1) * MT_WF_ARB_BCN_START_BSS0n(1)));
+                ((dev->mt76.beacon_mask >> 1) *
+                 MT_WF_ARB_BCN_START_BSS0n(1)));
        mt7603_irq_enable(dev, MT_INT_MAC_IRQ3);
 
-       if (dev->beacon_mask & ~BIT(0))
+       if (dev->mt76.beacon_mask & ~BIT(0))
                mt76_set(dev, MT_LPON_SBTOR(0), MT_LPON_SBTOR_SUB_BSS_EN);
        else
                mt76_clear(dev, MT_LPON_SBTOR(0), MT_LPON_SBTOR_SUB_BSS_EN);
index 380bd8dced9d6d2c50a556ecfcf0cde72ea45256..cc20a0cbed8dc8c351884be37975634bb39c5b6d 100644 (file)
@@ -125,8 +125,6 @@ struct mt7603_dev {
 
        s8 sensitivity;
 
-       u8 beacon_mask;
-
        u8 beacon_check;
        u8 tx_hang_check;
        u8 tx_dma_check;
index 8462d3a7cc3f753a1624d97dce2559fb067b49de..a679914e45510948de11d3b25adb4514ef7178aa 100644 (file)
@@ -103,7 +103,6 @@ struct mt76x02_dev {
        u32 aggr_stats[32];
 
        struct sk_buff *beacons[8];
-       u8 beacon_mask;
        u8 beacon_data_mask;
 
        u8 tbtt_count;
index 985a9b5d0e45a1e0729cb63598e9db1764d18408..e196b9c0a6861545facb602d744171662edb48f8 100644 (file)
@@ -119,23 +119,23 @@ static void
 __mt76x02_mac_set_beacon_enable(struct mt76x02_dev *dev, u8 vif_idx,
                                bool val, struct sk_buff *skb)
 {
-       u8 old_mask = dev->beacon_mask;
+       u8 old_mask = dev->mt76.beacon_mask;
        bool en;
        u32 reg;
 
        if (val) {
-               dev->beacon_mask |= BIT(vif_idx);
+               dev->mt76.beacon_mask |= BIT(vif_idx);
                if (skb)
                        mt76x02_mac_set_beacon(dev, vif_idx, skb);
        } else {
-               dev->beacon_mask &= ~BIT(vif_idx);
+               dev->mt76.beacon_mask &= ~BIT(vif_idx);
                mt76x02_mac_set_beacon(dev, vif_idx, NULL);
        }
 
-       if (!!old_mask == !!dev->beacon_mask)
+       if (!!old_mask == !!dev->mt76.beacon_mask)
                return;
 
-       en = dev->beacon_mask;
+       en = dev->mt76.beacon_mask;
 
        reg = MT_BEACON_TIME_CFG_BEACON_TX |
              MT_BEACON_TIME_CFG_TBTT_EN |
@@ -156,7 +156,7 @@ void mt76x02_mac_set_beacon_enable(struct mt76x02_dev *dev,
        if (mt76_is_usb(dev))
                skb = ieee80211_beacon_get(mt76_hw(dev), vif);
 
-       if (!dev->beacon_mask)
+       if (!dev->mt76.beacon_mask)
                dev->tbtt_count = 0;
 
        __mt76x02_mac_set_beacon_enable(dev, vif_idx, val, skb);
@@ -203,7 +203,7 @@ mt76x02_update_beacon_iter(void *priv, u8 *mac, struct ieee80211_vif *vif)
        struct mt76x02_vif *mvif = (struct mt76x02_vif *)vif->drv_priv;
        struct sk_buff *skb = NULL;
 
-       if (!(dev->beacon_mask & BIT(mvif->idx)))
+       if (!(dev->mt76.beacon_mask & BIT(mvif->idx)))
                return;
 
        skb = ieee80211_beacon_get(mt76_hw(dev), vif);
@@ -223,7 +223,7 @@ mt76x02_add_buffered_bc(void *priv, u8 *mac, struct ieee80211_vif *vif)
        struct ieee80211_tx_info *info;
        struct sk_buff *skb;
 
-       if (!(dev->beacon_mask & BIT(mvif->idx)))
+       if (!(dev->mt76.beacon_mask & BIT(mvif->idx)))
                return;
 
        skb = ieee80211_get_buffered_bc(mt76_hw(dev), vif);
index c1f041e1a27995b759751a45d28474b1630e508d..56510a1a843a8aaf787ad0deb4c63f6f15c59a77 100644 (file)
@@ -1045,7 +1045,7 @@ void mt76x02_mac_work(struct work_struct *work)
                dev->aggr_stats[idx++] += val >> 16;
        }
 
-       if (!dev->beacon_mask)
+       if (!dev->mt76.beacon_mask)
                mt76x02_check_mac_err(dev);
 
        if (dev->ed_monitor)
index c834acce2af47a7b51377b43daec7843c21f291a..4e0f8aed46038884768f84ecf54ba3c1a6cf2099 100644 (file)
@@ -437,7 +437,7 @@ static void mt76x02_reset_state(struct mt76x02_dev *dev)
        }
 
        dev->vif_mask = 0;
-       dev->beacon_mask = 0;
+       dev->mt76.beacon_mask = 0;
 }
 
 static void mt76x02_watchdog_reset(struct mt76x02_dev *dev)
@@ -461,7 +461,7 @@ static void mt76x02_watchdog_reset(struct mt76x02_dev *dev)
        if (restart)
                mt76x02_reset_state(dev);
 
-       if (dev->beacon_mask)
+       if (dev->mt76.beacon_mask)
                mt76_clear(dev, MT_BEACON_TIME_CFG,
                           MT_BEACON_TIME_CFG_BEACON_TX |
                           MT_BEACON_TIME_CFG_TBTT_EN);
@@ -493,7 +493,7 @@ static void mt76x02_watchdog_reset(struct mt76x02_dev *dev)
        if (dev->ed_monitor)
                mt76_set(dev, MT_TXOP_CTRL_CFG, MT_TXOP_ED_CCA_EN);
 
-       if (dev->beacon_mask && !restart)
+       if (dev->mt76.beacon_mask && !restart)
                mt76_set(dev, MT_BEACON_TIME_CFG,
                         MT_BEACON_TIME_CFG_BEACON_TX |
                         MT_BEACON_TIME_CFG_TBTT_EN);
index 81cebd92a4e83fa99128e1a3affb9d9bcc5109df..5b6ac1b364e13ed7cfe370245075a183ce26f02c 100644 (file)
@@ -175,7 +175,7 @@ static void mt76x02u_pre_tbtt_work(struct work_struct *work)
        struct sk_buff *skb;
        int i, nbeacons;
 
-       if (!dev->beacon_mask)
+       if (!dev->mt76.beacon_mask)
                return;
 
        mt76x02_resync_beacon_timer(dev);
@@ -184,7 +184,7 @@ static void mt76x02u_pre_tbtt_work(struct work_struct *work)
                IEEE80211_IFACE_ITER_RESUME_ALL,
                mt76x02_update_beacon_iter, dev);
 
-       nbeacons = hweight8(dev->beacon_mask);
+       nbeacons = hweight8(dev->mt76.beacon_mask);
        mt76x02_enqueue_buffered_bc(dev, &data, N_BCN_SLOTS - nbeacons);
 
        for (i = nbeacons; i < N_BCN_SLOTS; i++) {
@@ -207,7 +207,8 @@ static enum hrtimer_restart mt76x02u_pre_tbtt_interrupt(struct hrtimer *timer)
 
 static void mt76x02u_pre_tbtt_enable(struct mt76x02_dev *dev, bool en)
 {
-       if (en && dev->beacon_mask && !hrtimer_active(&dev->pre_tbtt_timer))
+       if (en && dev->mt76.beacon_mask &&
+           !hrtimer_active(&dev->pre_tbtt_timer))
                mt76x02u_start_pre_tbtt_timer(dev);
        if (!en)
                mt76x02u_stop_pre_tbtt_timer(dev);