wifi: rtw89: refactor VIF related func ahead for MLO
authorZong-Zhe Yang <kevin_yang@realtek.com>
Mon, 16 Sep 2024 05:31:56 +0000 (13:31 +0800)
committerPing-Ke Shih <pkshih@realtek.com>
Tue, 24 Sep 2024 01:17:40 +0000 (09:17 +0800)
Refactor VIF related functions, e.g. add/remove/assoc/mapping
to separate most link stuffs into sub-functions for MLO reuse.

Signed-off-by: Zong-Zhe Yang <kevin_yang@realtek.com>
Signed-off-by: Ping-Ke Shih <pkshih@realtek.com>
Link: https://patch.msgid.link/20240916053158.47350-6-pkshih@realtek.com
drivers/net/wireless/realtek/rtw89/core.c
drivers/net/wireless/realtek/rtw89/core.h
drivers/net/wireless/realtek/rtw89/mac.c
drivers/net/wireless/realtek/rtw89/mac.h
drivers/net/wireless/realtek/rtw89/mac80211.c
drivers/net/wireless/realtek/rtw89/phy.c
drivers/net/wireless/realtek/rtw89/phy.h
drivers/net/wireless/realtek/rtw89/wow.c

index 84c1952fbea8866761cdb823552b188d5bc63f7d..c6d3b8fd98b65178c64d1bb24d616b01e406c6a3 100644 (file)
@@ -3466,9 +3466,9 @@ int rtw89_core_release_sta_ba_entry(struct rtw89_dev *rtwdev,
        case NL80211_IFTYPE_ ## _type:  \
                rtwvif_link->wifi_role = RTW89_WIFI_ROLE_ ## _type;     \
                break
-void rtw89_vif_type_mapping(struct ieee80211_vif *vif, bool assoc)
+void rtw89_vif_type_mapping(struct rtw89_vif_link *rtwvif_link, bool assoc)
 {
-       struct rtw89_vif_link *rtwvif_link = (struct rtw89_vif_link *)vif->drv_priv;
+       const struct ieee80211_vif *vif = rtwvif_to_vif(rtwvif_link);
        const struct ieee80211_bss_conf *bss_conf;
 
        switch (vif->type) {
@@ -3638,7 +3638,7 @@ int rtw89_core_sta_disconnect(struct rtw89_dev *rtwdev,
                rtw89_cam_deinit_bssid_cam(rtwdev, &rtwsta_link->bssid_cam);
 
        if (vif->type == NL80211_IFTYPE_STATION && !sta->tdls) {
-               rtw89_vif_type_mapping(vif, false);
+               rtw89_vif_type_mapping(rtwvif_link, false);
                rtw89_fw_release_general_pkt_list_vif(rtwdev, rtwvif_link, true);
        }
 
@@ -4785,9 +4785,9 @@ int rtw89_chip_info_setup(struct rtw89_dev *rtwdev)
 EXPORT_SYMBOL(rtw89_chip_info_setup);
 
 void rtw89_chip_cfg_txpwr_ul_tb_offset(struct rtw89_dev *rtwdev,
-                                      struct ieee80211_vif *vif)
+                                      struct rtw89_vif_link *rtwvif_link)
 {
-       struct rtw89_vif_link *rtwvif_link = (struct rtw89_vif_link *)vif->drv_priv;
+       struct ieee80211_vif *vif = rtwvif_to_vif(rtwvif_link);
        const struct rtw89_chip_info *chip = rtwdev->chip;
        struct ieee80211_bss_conf *bss_conf;
 
index 16ada1be9a0bfb50ee2fddc9ece322fd95d818fb..102fb6eba4275a0cdbbecfbe4b507adaf8f7c4d7 100644 (file)
@@ -6703,10 +6703,10 @@ int rtw89_core_acquire_sta_ba_entry(struct rtw89_dev *rtwdev,
 int rtw89_core_release_sta_ba_entry(struct rtw89_dev *rtwdev,
                                    struct rtw89_sta_link *rtwsta_link, u8 tid,
                                    u8 *cam_idx);
-void rtw89_vif_type_mapping(struct ieee80211_vif *vif, bool assoc);
+void rtw89_vif_type_mapping(struct rtw89_vif_link *rtwvif_link, bool assoc);
 int rtw89_chip_info_setup(struct rtw89_dev *rtwdev);
 void rtw89_chip_cfg_txpwr_ul_tb_offset(struct rtw89_dev *rtwdev,
-                                      struct ieee80211_vif *vif);
+                                      struct rtw89_vif_link *rtwvif_link);
 bool rtw89_ra_report_to_bitrate(struct rtw89_dev *rtwdev, u8 rpt_rate, u16 *bitrate);
 int rtw89_regd_setup(struct rtw89_dev *rtwdev);
 int rtw89_regd_init(struct rtw89_dev *rtwdev,
index 7ab2aac3c3d22d18809a0eb9e6c00a7ec645ae73..a7b89a25d7c46327f09eaa42ede6d0e2b43e1003 100644 (file)
@@ -4697,9 +4697,9 @@ static void rtw89_mac_check_he_obss_narrow_bw_ru_iter(struct wiphy *wiphy,
 }
 
 void rtw89_mac_set_he_obss_narrow_bw_ru(struct rtw89_dev *rtwdev,
-                                       struct ieee80211_vif *vif)
+                                       struct rtw89_vif_link *rtwvif_link)
 {
-       struct rtw89_vif_link *rtwvif_link = (struct rtw89_vif_link *)vif->drv_priv;
+       struct ieee80211_vif *vif = rtwvif_to_vif_safe(rtwvif_link);
        const struct rtw89_mac_gen_def *mac = rtwdev->chip->mac_def;
        struct ieee80211_hw *hw = rtwdev->hw;
        struct ieee80211_bss_conf *bss_conf;
@@ -4742,32 +4742,12 @@ void rtw89_mac_stop_ap(struct rtw89_dev *rtwdev, struct rtw89_vif_link *rtwvif_l
 
 int rtw89_mac_add_vif(struct rtw89_dev *rtwdev, struct rtw89_vif_link *rtwvif_link)
 {
-       int ret;
-
-       rtwvif_link->mac_id = rtw89_acquire_mac_id(rtwdev);
-       if (rtwvif_link->mac_id == RTW89_MAX_MAC_ID_NUM)
-               return -ENOSPC;
-
-       ret = rtw89_mac_vif_init(rtwdev, rtwvif_link);
-       if (ret)
-               goto release_mac_id;
-
-       return 0;
-
-release_mac_id:
-       rtw89_release_mac_id(rtwdev, rtwvif_link->mac_id);
-
-       return ret;
+       return rtw89_mac_vif_init(rtwdev, rtwvif_link);
 }
 
 int rtw89_mac_remove_vif(struct rtw89_dev *rtwdev, struct rtw89_vif_link *rtwvif_link)
 {
-       int ret;
-
-       ret = rtw89_mac_vif_deinit(rtwdev, rtwvif_link);
-       rtw89_release_mac_id(rtwdev, rtwvif_link->mac_id);
-
-       return ret;
+       return rtw89_mac_vif_deinit(rtwdev, rtwvif_link);
 }
 
 static void
index 3ea2dcbfa5b8cdbd4e0aab05c6e4a4fc9f748221..f2a31cba226e973b96fccff7faf9147c4bc1fc8a 100644 (file)
@@ -1158,7 +1158,7 @@ int rtw89_mac_port_get_tsf(struct rtw89_dev *rtwdev, struct rtw89_vif_link *rtwv
 void rtw89_mac_port_cfg_rx_sync(struct rtw89_dev *rtwdev,
                                struct rtw89_vif_link *rtwvif_link, bool en);
 void rtw89_mac_set_he_obss_narrow_bw_ru(struct rtw89_dev *rtwdev,
-                                       struct ieee80211_vif *vif);
+                                       struct rtw89_vif_link *rtwvif_link);
 void rtw89_mac_stop_ap(struct rtw89_dev *rtwdev, struct rtw89_vif_link *rtwvif_link);
 void rtw89_mac_enable_beacon_for_ap_vifs(struct rtw89_dev *rtwdev, bool en);
 int rtw89_mac_remove_vif(struct rtw89_dev *rtwdev, struct rtw89_vif_link *vif);
index f04032a8a8ec4cc99e225c61c5a9f8702997c3a1..d3fad66e56cd817be9575f26f38b8db94565e6c1 100644 (file)
@@ -106,12 +106,60 @@ static int rtw89_ops_config(struct ieee80211_hw *hw, u32 changed)
        return 0;
 }
 
+static int __rtw89_ops_add_iface_link(struct rtw89_dev *rtwdev,
+                                     struct rtw89_vif_link *rtwvif_link)
+{
+       struct ieee80211_bss_conf *bss_conf;
+       int ret;
+
+       rtw89_leave_ps_mode(rtwdev);
+
+       rtw89_vif_type_mapping(rtwvif_link, false);
+
+       INIT_WORK(&rtwvif_link->update_beacon_work, rtw89_core_update_beacon_work);
+       INIT_LIST_HEAD(&rtwvif_link->general_pkt_list);
+
+       rtwvif_link->hit_rule = 0;
+       rtwvif_link->bcn_hit_cond = 0;
+       rtwvif_link->chanctx_assigned = false;
+       rtwvif_link->chanctx_idx = RTW89_CHANCTX_0;
+       rtwvif_link->reg_6ghz_power = RTW89_REG_6GHZ_POWER_DFLT;
+
+       rcu_read_lock();
+
+       bss_conf = rtw89_vif_rcu_dereference_link(rtwvif_link, true);
+       ether_addr_copy(rtwvif_link->mac_addr, bss_conf->addr);
+
+       rcu_read_unlock();
+
+       ret = rtw89_mac_add_vif(rtwdev, rtwvif_link);
+       if (ret)
+               return ret;
+
+       rtw89_btc_ntfy_role_info(rtwdev, rtwvif_link, NULL, BTC_ROLE_START);
+       return 0;
+}
+
+static void __rtw89_ops_remove_iface_link(struct rtw89_dev *rtwdev,
+                                         struct rtw89_vif_link *rtwvif_link)
+{
+       mutex_unlock(&rtwdev->mutex);
+       cancel_work_sync(&rtwvif_link->update_beacon_work);
+       mutex_lock(&rtwdev->mutex);
+
+       rtw89_leave_ps_mode(rtwdev);
+
+       rtw89_btc_ntfy_role_info(rtwdev, rtwvif_link, NULL, BTC_ROLE_STOP);
+
+       rtw89_mac_remove_vif(rtwdev, rtwvif_link);
+}
+
 static int rtw89_ops_add_interface(struct ieee80211_hw *hw,
                                   struct ieee80211_vif *vif)
 {
        struct rtw89_dev *rtwdev = hw->priv;
        struct rtw89_vif_link *rtwvif_link = (struct rtw89_vif_link *)vif->drv_priv;
-       struct ieee80211_bss_conf *bss_conf;
+       u8 mac_id, port;
        int ret = 0;
 
        rtw89_debug(rtwdev, RTW89_DBG_STATE, "add vif %pM type %d, p2p %d\n",
@@ -128,53 +176,47 @@ static int rtw89_ops_add_interface(struct ieee80211_hw *hw,
        rtwvif_link->rtwdev = rtwdev;
        rtwvif_link->roc.state = RTW89_ROC_IDLE;
        rtwvif_link->offchan = false;
-       if (!rtw89_rtwvif_in_list(rtwdev, rtwvif_link))
-               list_add_tail(&rtwvif_link->list, &rtwdev->rtwvifs_list);
-
-       INIT_WORK(&rtwvif_link->update_beacon_work, rtw89_core_update_beacon_work);
        INIT_DELAYED_WORK(&rtwvif_link->roc.roc_work, rtw89_roc_work);
-       rtw89_leave_ps_mode(rtwdev);
 
        rtw89_traffic_stats_init(rtwdev, &rtwvif_link->stats);
-       rtw89_vif_type_mapping(vif, false);
-       rtwvif_link->port = rtw89_core_acquire_bit_map(rtwdev->hw_port,
-                                                      RTW89_PORT_NUM);
-       if (rtwvif_link->port == RTW89_PORT_NUM) {
+
+       mac_id = rtw89_acquire_mac_id(rtwdev);
+       if (mac_id == RTW89_MAX_MAC_ID_NUM) {
                ret = -ENOSPC;
-               list_del_init(&rtwvif_link->list);
-               goto out;
+               goto err;
+       }
+
+       port = rtw89_core_acquire_bit_map(rtwdev->hw_port, RTW89_PORT_NUM);
+       if (port == RTW89_PORT_NUM) {
+               ret = -ENOSPC;
+               goto release_macid;
        }
 
-       rtwvif_link->bcn_hit_cond = 0;
        rtwvif_link->mac_idx = RTW89_MAC_0;
        rtwvif_link->phy_idx = RTW89_PHY_0;
-       rtwvif_link->chanctx_idx = RTW89_CHANCTX_0;
-       rtwvif_link->chanctx_assigned = false;
-       rtwvif_link->hit_rule = 0;
-       rtwvif_link->reg_6ghz_power = RTW89_REG_6GHZ_POWER_DFLT;
+       rtwvif_link->mac_id = mac_id;
+       rtwvif_link->port = port;
 
-       rcu_read_lock();
-
-       bss_conf = rtw89_vif_rcu_dereference_link(rtwvif_link, true);
-       ether_addr_copy(rtwvif_link->mac_addr, bss_conf->addr);
-
-       rcu_read_unlock();
+       rtw89_core_txq_init(rtwdev, vif->txq);
 
-       INIT_LIST_HEAD(&rtwvif_link->general_pkt_list);
+       if (!rtw89_rtwvif_in_list(rtwdev, rtwvif_link))
+               list_add_tail(&rtwvif_link->list, &rtwdev->rtwvifs_list);
 
-       ret = rtw89_mac_add_vif(rtwdev, rtwvif_link);
-       if (ret) {
-               rtw89_core_release_bit_map(rtwdev->hw_port, rtwvif_link->port);
-               list_del_init(&rtwvif_link->list);
-               goto out;
-       }
+       ret = __rtw89_ops_add_iface_link(rtwdev, rtwvif_link);
+       if (ret)
+               goto release_port;
 
-       rtw89_core_txq_init(rtwdev, vif->txq);
+       rtw89_recalc_lps(rtwdev);
 
-       rtw89_btc_ntfy_role_info(rtwdev, rtwvif_link, NULL, BTC_ROLE_START);
+       mutex_unlock(&rtwdev->mutex);
+       return 0;
 
-       rtw89_recalc_lps(rtwdev);
-out:
+release_port:
+       list_del_init(&rtwvif_link->list);
+       rtw89_core_release_bit_map(rtwdev->hw_port, port);
+release_macid:
+       rtw89_release_mac_id(rtwdev, mac_id);
+err:
        mutex_unlock(&rtwdev->mutex);
 
        return ret;
@@ -189,14 +231,14 @@ static void rtw89_ops_remove_interface(struct ieee80211_hw *hw,
        rtw89_debug(rtwdev, RTW89_DBG_STATE, "remove vif %pM type %d p2p %d\n",
                    vif->addr, vif->type, vif->p2p);
 
-       cancel_work_sync(&rtwvif_link->update_beacon_work);
        cancel_delayed_work_sync(&rtwvif_link->roc.roc_work);
 
        mutex_lock(&rtwdev->mutex);
-       rtw89_leave_ps_mode(rtwdev);
-       rtw89_btc_ntfy_role_info(rtwdev, rtwvif_link, NULL, BTC_ROLE_STOP);
-       rtw89_mac_remove_vif(rtwdev, rtwvif_link);
+
+       __rtw89_ops_remove_iface_link(rtwdev, rtwvif_link);
+
        rtw89_core_release_bit_map(rtwdev->hw_port, rtwvif_link->port);
+       rtw89_release_mac_id(rtwdev, rtwvif_link->mac_id);
        list_del_init(&rtwvif_link->list);
        rtw89_recalc_lps(rtwdev);
        rtw89_enter_ips_by_hwflags(rtwdev);
@@ -417,6 +459,7 @@ static void rtw89_conf_tx(struct rtw89_dev *rtwdev,
 static void rtw89_station_mode_sta_assoc(struct rtw89_dev *rtwdev,
                                         struct ieee80211_vif *vif)
 {
+       struct rtw89_vif_link *rtwvif_link = (struct rtw89_vif_link *)vif->drv_priv;
        struct ieee80211_sta *sta;
 
        if (vif->type != NL80211_IFTYPE_STATION)
@@ -428,11 +471,20 @@ static void rtw89_station_mode_sta_assoc(struct rtw89_dev *rtwdev,
                return;
        }
 
-       rtw89_vif_type_mapping(vif, true);
+       rtw89_vif_type_mapping(rtwvif_link, true);
 
        rtw89_core_sta_assoc(rtwdev, vif, sta);
 }
 
+static void __rtw89_ops_bss_link_assoc(struct rtw89_dev *rtwdev,
+                                      struct rtw89_vif_link *rtwvif_link)
+{
+       rtw89_phy_set_bss_color(rtwdev, rtwvif_link);
+       rtw89_chip_cfg_txpwr_ul_tb_offset(rtwdev, rtwvif_link);
+       rtw89_mac_port_update(rtwdev, rtwvif_link);
+       rtw89_mac_set_he_obss_narrow_bw_ru(rtwdev, rtwvif_link);
+}
+
 static void rtw89_ops_vif_cfg_changed(struct ieee80211_hw *hw,
                                      struct ieee80211_vif *vif, u64 changed)
 {
@@ -445,10 +497,7 @@ static void rtw89_ops_vif_cfg_changed(struct ieee80211_hw *hw,
        if (changed & BSS_CHANGED_ASSOC) {
                if (vif->cfg.assoc) {
                        rtw89_station_mode_sta_assoc(rtwdev, vif);
-                       rtw89_phy_set_bss_color(rtwdev, vif);
-                       rtw89_chip_cfg_txpwr_ul_tb_offset(rtwdev, vif);
-                       rtw89_mac_port_update(rtwdev, rtwvif_link);
-                       rtw89_mac_set_he_obss_narrow_bw_ru(rtwdev, vif);
+                       __rtw89_ops_bss_link_assoc(rtwdev, rtwvif_link);
 
                        rtw89_queue_chanctx_work(rtwdev);
                } else {
@@ -494,7 +543,7 @@ static void rtw89_ops_link_info_changed(struct ieee80211_hw *hw,
                rtw89_conf_tx(rtwdev, rtwvif_link);
 
        if (changed & BSS_CHANGED_HE_BSS_COLOR)
-               rtw89_phy_set_bss_color(rtwdev, vif);
+               rtw89_phy_set_bss_color(rtwdev, rtwvif_link);
 
        if (changed & BSS_CHANGED_MU_GROUPS)
                rtw89_mac_bf_set_gid_table(rtwdev, vif, conf);
index 2da0f7bf286cf9b28f646f37423afd1a0867e50a..3cc5e94058b4ec2038881b2a64cb89022782fce4 100644 (file)
@@ -6177,9 +6177,10 @@ void rtw89_phy_dm_init(struct rtw89_dev *rtwdev)
        rtw89_chip_cfg_txrx_path(rtwdev);
 }
 
-void rtw89_phy_set_bss_color(struct rtw89_dev *rtwdev, struct ieee80211_vif *vif)
+void rtw89_phy_set_bss_color(struct rtw89_dev *rtwdev,
+                            struct rtw89_vif_link *rtwvif_link)
 {
-       struct rtw89_vif_link *rtwvif_link = (struct rtw89_vif_link *)vif->drv_priv;
+       struct ieee80211_vif *vif = rtwvif_to_vif(rtwvif_link);
        const struct rtw89_chip_info *chip = rtwdev->chip;
        const struct rtw89_reg_def *bss_clr_vld = &chip->bss_clr_vld;
        enum rtw89_phy_idx phy_idx = RTW89_PHY_0;
index 7b6ccf2f933ee168128b7fca541e3b0eb12dfd3e..310ef0570e7ad6991c504bff3357c694d7e04c20 100644 (file)
@@ -953,7 +953,8 @@ void rtw89_phy_antdiv_parse(struct rtw89_dev *rtwdev,
                            struct rtw89_rx_phy_ppdu *phy_ppdu);
 void rtw89_phy_antdiv_track(struct rtw89_dev *rtwdev);
 void rtw89_phy_antdiv_work(struct work_struct *work);
-void rtw89_phy_set_bss_color(struct rtw89_dev *rtwdev, struct ieee80211_vif *vif);
+void rtw89_phy_set_bss_color(struct rtw89_dev *rtwdev,
+                            struct rtw89_vif_link *rtwvif_link);
 void rtw89_phy_tssi_ctrl_set_bandedge_cfg(struct rtw89_dev *rtwdev,
                                          enum rtw89_mac_idx mac_idx,
                                          enum rtw89_tssi_bandedge_cfg bandedge_cfg);
index 17e2d5cbb1e6a771c0981d13394c605200265aba..20b776ae634eadbca2bd2005acab6dddeb7a8fcd 100644 (file)
@@ -1268,8 +1268,8 @@ static int rtw89_wow_swap_fw(struct rtw89_dev *rtwdev, bool wow)
                        return ret;
                }
                rtw89_phy_ra_assoc(rtwdev, wow_sta);
-               rtw89_phy_set_bss_color(rtwdev, wow_vif);
-               rtw89_chip_cfg_txpwr_ul_tb_offset(rtwdev, wow_vif);
+               rtw89_phy_set_bss_color(rtwdev, rtwvif_link);
+               rtw89_chip_cfg_txpwr_ul_tb_offset(rtwdev, rtwvif_link);
        }
 
        if (chip_gen == RTW89_CHIP_BE)