wifi: mt76: mt7925: add mt7925_mac_link_bss_add to create per-link BSS
authorSean Wang <sean.wang@mediatek.com>
Thu, 13 Jun 2024 03:02:39 +0000 (20:02 -0700)
committerFelix Fietkau <nbd@nbd.name>
Tue, 9 Jul 2024 21:01:59 +0000 (23:01 +0200)
The mt7925_mac_link_bss_add function currently creates the per-link BSS.
We will extend this function when we implement the MLO functionality.

This patch only includes structural changes and does not involve any
logic changes.

Signed-off-by: Sean Wang <sean.wang@mediatek.com>
Link: https://patch.msgid.link/20240613030241.5771-46-sean.wang@kernel.org
Signed-off-by: Felix Fietkau <nbd@nbd.name>
drivers/net/wireless/mediatek/mt76/mt7925/main.c

index 21a2d1bd8d186c0ba50e484806bb7b2695712a1f..8b10b63bc98dd43ea9a7605e48faacc9d5c5211e 100644 (file)
@@ -317,63 +317,81 @@ static int mt7925_start(struct ieee80211_hw *hw)
        return err;
 }
 
-static int
-mt7925_add_interface(struct ieee80211_hw *hw, struct ieee80211_vif *vif)
+static int mt7925_mac_link_bss_add(struct mt792x_dev *dev,
+                                  struct ieee80211_bss_conf *link_conf,
+                                  struct mt792x_link_sta *mlink)
 {
-       struct mt792x_vif *mvif = (struct mt792x_vif *)vif->drv_priv;
-       struct mt792x_dev *dev = mt792x_hw_dev(hw);
-       struct mt792x_phy *phy = mt792x_hw_phy(hw);
+       struct mt792x_bss_conf *mconf = mt792x_link_conf_to_mconf(link_conf);
+       struct ieee80211_vif *vif = link_conf->vif;
+       struct mt792x_vif *mvif = mconf->vif;
        struct mt76_txq *mtxq;
        int idx, ret = 0;
 
-       mt792x_mutex_acquire(dev);
-
-       mvif->bss_conf.mt76.idx = __ffs64(~dev->mt76.vif_mask);
-       if (mvif->bss_conf.mt76.idx >= MT792x_MAX_INTERFACES) {
+       mconf->mt76.idx = __ffs64(~dev->mt76.vif_mask);
+       if (mconf->mt76.idx >= MT792x_MAX_INTERFACES) {
                ret = -ENOSPC;
                goto out;
        }
 
-       mvif->bss_conf.mt76.omac_idx = mvif->bss_conf.mt76.idx;
-       mvif->phy = phy;
-       mvif->bss_conf.mt76.band_idx = 0;
-       mvif->bss_conf.mt76.wmm_idx = mvif->bss_conf.mt76.idx % MT76_CONNAC_MAX_WMM_SETS;
-       mvif->bss_conf.vif = mvif;
+       mconf->mt76.omac_idx = mconf->mt76.idx;
+       mconf->mt76.band_idx = 0;
+       mconf->mt76.wmm_idx = mconf->mt76.idx % MT76_CONNAC_MAX_WMM_SETS;
 
-       if (phy->mt76->chandef.chan->band != NL80211_BAND_2GHZ)
-               mvif->bss_conf.mt76.basic_rates_idx = MT792x_BASIC_RATES_TBL + 4;
+       if (mvif->phy->mt76->chandef.chan->band != NL80211_BAND_2GHZ)
+               mconf->mt76.basic_rates_idx = MT792x_BASIC_RATES_TBL + 4;
        else
-               mvif->bss_conf.mt76.basic_rates_idx = MT792x_BASIC_RATES_TBL;
+               mconf->mt76.basic_rates_idx = MT792x_BASIC_RATES_TBL;
 
-       ret = mt76_connac_mcu_uni_add_dev(&dev->mphy, &vif->bss_conf,
-                                         &mvif->sta.deflink.wcid, true);
+       ret = mt76_connac_mcu_uni_add_dev(&dev->mphy, link_conf,
+                                         &mlink->wcid, true);
        if (ret)
                goto out;
 
-       dev->mt76.vif_mask |= BIT_ULL(mvif->bss_conf.mt76.idx);
-       phy->omac_mask |= BIT_ULL(mvif->bss_conf.mt76.omac_idx);
+       dev->mt76.vif_mask |= BIT_ULL(mconf->mt76.idx);
+       mvif->phy->omac_mask |= BIT_ULL(mconf->mt76.omac_idx);
 
-       idx = MT792x_WTBL_RESERVED - mvif->bss_conf.mt76.idx;
+       idx = MT792x_WTBL_RESERVED - mconf->mt76.idx;
 
-       INIT_LIST_HEAD(&mvif->sta.deflink.wcid.poll_list);
-       mvif->sta.deflink.wcid.idx = idx;
-       mvif->sta.deflink.wcid.phy_idx = mvif->bss_conf.mt76.band_idx;
-       mvif->sta.deflink.wcid.hw_key_idx = -1;
-       mvif->sta.deflink.wcid.tx_info |= MT_WCID_TX_INFO_SET;
-       mvif->sta.vif = mvif;
-       mt76_wcid_init(&mvif->sta.deflink.wcid);
+       INIT_LIST_HEAD(&mlink->wcid.poll_list);
+       mlink->wcid.idx = idx;
+       mlink->wcid.phy_idx = mconf->mt76.band_idx;
+       mlink->wcid.hw_key_idx = -1;
+       mlink->wcid.tx_info |= MT_WCID_TX_INFO_SET;
+       mt76_wcid_init(&mlink->wcid);
 
        mt7925_mac_wtbl_update(dev, idx,
                               MT_WTBL_UPDATE_ADM_COUNT_CLEAR);
 
-       ewma_rssi_init(&mvif->bss_conf.rssi);
+       ewma_rssi_init(&mconf->rssi);
 
-       rcu_assign_pointer(dev->mt76.wcid[idx], &mvif->sta.deflink.wcid);
+       rcu_assign_pointer(dev->mt76.wcid[idx], &mlink->wcid);
        if (vif->txq) {
                mtxq = (struct mt76_txq *)vif->txq->drv_priv;
                mtxq->wcid = idx;
        }
 
+out:
+       return ret;
+}
+
+static int
+mt7925_add_interface(struct ieee80211_hw *hw, struct ieee80211_vif *vif)
+{
+       struct mt792x_vif *mvif = (struct mt792x_vif *)vif->drv_priv;
+       struct mt792x_dev *dev = mt792x_hw_dev(hw);
+       struct mt792x_phy *phy = mt792x_hw_phy(hw);
+       int ret = 0;
+
+       mt792x_mutex_acquire(dev);
+
+       mvif->phy = phy;
+       mvif->bss_conf.vif = mvif;
+       mvif->sta.vif = mvif;
+
+       ret = mt7925_mac_link_bss_add(dev, &vif->bss_conf, &mvif->sta.deflink);
+       if (ret < 0)
+               goto out;
+
        vif->driver_flags |= IEEE80211_VIF_BEACON_FILTER;
 out:
        mt792x_mutex_release(dev);