rtw89: add addr_cam field to sta to support AP mode

In AP mode, each connected station needs an entry of address CAM. The
address CAM of vif is still needed to assit in AP itself.

For station mode, it still uses vif's address CAM.

Add a help macro rtw89_get_addr_cam_of() to get addr_cam from vif or sta
for all use cases.

Signed-off-by: Ping-Ke Shih <pkshih@realtek.com>
Signed-off-by: Kalle Valo <kvalo@kernel.org>
Link: https://lore.kernel.org/r/20220207063900.43643-3-pkshih@realtek.com

diff --git a/drivers/net/wireless/realtek/rtw89/cam.c b/drivers/net/wireless/realtek/rtw89/cam.c
index 2114d11..305dbbe 100644 (file)
--- a/drivers/net/wireless/realtek/rtw89/cam.c
+++ b/drivers/net/wireless/realtek/rtw89/cam.c
@@ -231,7 +231,7 @@ static int rtw89_cam_attach_sec_cam(struct rtw89_dev *rtwdev,
        }
 
        rtwvif = (struct rtw89_vif *)vif->drv_priv;
-       addr_cam = &rtwvif->addr_cam;
+       addr_cam = rtw89_get_addr_cam_of(rtwvif, rtwsta);
        ret = rtw89_cam_get_addr_cam_key_idx(addr_cam, sec_cam, key, &key_idx);
        if (ret) {
                rtw89_err(rtwdev, "failed to get addr cam key idx %d, %d\n",
@@ -387,7 +387,7 @@ int rtw89_cam_sec_key_del(struct rtw89_dev *rtwdev,
        }
 
        rtwvif = (struct rtw89_vif *)vif->drv_priv;
-       addr_cam = &rtwvif->addr_cam;
+       addr_cam = rtw89_get_addr_cam_of(rtwvif, rtwsta);
        sec_cam = addr_cam->sec_entries[key_idx];
        if (!sec_cam)
                return -EINVAL;
@@ -617,7 +617,7 @@ void rtw89_cam_fill_addr_cam_info(struct rtw89_dev *rtwdev,
                                  u8 *cmd)
 {
        struct ieee80211_vif *vif = rtwvif_to_vif(rtwvif);
-       struct rtw89_addr_cam_entry *addr_cam = &rtwvif->addr_cam;
+       struct rtw89_addr_cam_entry *addr_cam = rtw89_get_addr_cam_of(rtwvif, rtwsta);
        struct ieee80211_sta *sta = rtwsta_to_sta_safe(rtwsta);
        const u8 *sma = scan_mac_addr ? scan_mac_addr : rtwvif->mac_addr;
        u8 sma_hash, tma_hash, addr_msk_start;

diff --git a/drivers/net/wireless/realtek/rtw89/core.c b/drivers/net/wireless/realtek/rtw89/core.c
index 490f5d8..8157546 100644 (file)
--- a/drivers/net/wireless/realtek/rtw89/core.c
+++ b/drivers/net/wireless/realtek/rtw89/core.c
@@ -4,6 +4,7 @@
 #include <linux/ip.h>
 #include <linux/udp.h>
 
+#include "cam.h"
 #include "coex.h"
 #include "core.h"
 #include "efuse.h"
@@ -399,9 +400,11 @@ rtw89_core_tx_update_sec_key(struct rtw89_dev *rtwdev,
                             struct rtw89_core_tx_request *tx_req)
 {
        struct ieee80211_vif *vif = tx_req->vif;
+       struct ieee80211_sta *sta = tx_req->sta;
        struct ieee80211_tx_info *info;
        struct ieee80211_key_conf *key;
        struct rtw89_vif *rtwvif;
+       struct rtw89_sta *rtwsta = sta_to_rtwsta_safe(sta);
        struct rtw89_addr_cam_entry *addr_cam;
        struct rtw89_sec_cam_entry *sec_cam;
        struct rtw89_tx_desc_info *desc_info = &tx_req->desc_info;
@@ -414,7 +417,7 @@ rtw89_core_tx_update_sec_key(struct rtw89_dev *rtwdev,
        }
 
        rtwvif = (struct rtw89_vif *)vif->drv_priv;
-       addr_cam = &rtwvif->addr_cam;
+       addr_cam = rtw89_get_addr_cam_of(rtwvif, rtwsta);
 
        info = IEEE80211_SKB_CB(skb);
        key = info->control.hw_key;

diff --git a/drivers/net/wireless/realtek/rtw89/core.h b/drivers/net/wireless/realtek/rtw89/core.h
index cb71dfc..deb91f6 100644 (file)
--- a/drivers/net/wireless/realtek/rtw89/core.h
+++ b/drivers/net/wireless/realtek/rtw89/core.h
@@ -1842,32 +1842,6 @@ struct rtw89_ba_cam_entry {
        u8 tid;
 };
 
-struct rtw89_sta {
-       u8 mac_id;
-       bool disassoc;
-       struct rtw89_vif *rtwvif;
-       struct rtw89_ra_info ra;
-       struct rtw89_ra_report ra_report;
-       int max_agg_wait;
-       u8 prev_rssi;
-       struct ewma_rssi avg_rssi;
-       struct rtw89_ampdu_params ampdu_params[IEEE80211_NUM_TIDS];
-       struct ieee80211_rx_status rx_status;
-       u16 rx_hw_rate;
-       __le32 htc_template;
-
-       bool use_cfg_mask;
-       struct cfg80211_bitrate_mask mask;
-
-       bool cctl_tx_time;
-       u32 ampdu_max_time:4;
-       bool cctl_tx_retry_limit;
-       u32 data_tx_cnt_lmt:6;
-
-       DECLARE_BITMAP(ba_cam_map, RTW89_BA_CAM_NUM);
-       struct rtw89_ba_cam_entry ba_cam_entry[RTW89_BA_CAM_NUM];
-};
-
 #define RTW89_MAX_ADDR_CAM_NUM         128
 #define RTW89_MAX_BSSID_CAM_NUM                20
 #define RTW89_MAX_SEC_CAM_NUM          128
@@ -1911,6 +1885,33 @@ struct rtw89_sec_cam_entry {
        u8 key[32];
 };
 
+struct rtw89_sta {
+       u8 mac_id;
+       bool disassoc;
+       struct rtw89_vif *rtwvif;
+       struct rtw89_ra_info ra;
+       struct rtw89_ra_report ra_report;
+       int max_agg_wait;
+       u8 prev_rssi;
+       struct ewma_rssi avg_rssi;
+       struct rtw89_ampdu_params ampdu_params[IEEE80211_NUM_TIDS];
+       struct ieee80211_rx_status rx_status;
+       u16 rx_hw_rate;
+       __le32 htc_template;
+       struct rtw89_addr_cam_entry addr_cam; /* AP mode only */
+
+       bool use_cfg_mask;
+       struct cfg80211_bitrate_mask mask;
+
+       bool cctl_tx_time;
+       u32 ampdu_max_time:4;
+       bool cctl_tx_retry_limit;
+       u32 data_tx_cnt_lmt:6;
+
+       DECLARE_BITMAP(ba_cam_map, RTW89_BA_CAM_NUM);
+       struct rtw89_ba_cam_entry ba_cam_entry[RTW89_BA_CAM_NUM];
+};
+
 struct rtw89_efuse {
        bool valid;
        u8 xtal_cap;
@@ -3150,6 +3151,15 @@ static inline struct rtw89_sta *sta_to_rtwsta_safe(struct ieee80211_sta *sta)
 }
 
 static inline
+struct rtw89_addr_cam_entry *rtw89_get_addr_cam_of(struct rtw89_vif *rtwvif,
+                                                  struct rtw89_sta *rtwsta)
+{
+       if (rtwvif->net_type == RTW89_NET_TYPE_AP_MODE && rtwsta)
+               return &rtwsta->addr_cam;
+       return &rtwvif->addr_cam;
+}
+
+static inline
 void rtw89_chip_set_channel_prepare(struct rtw89_dev *rtwdev,
                                    struct rtw89_channel_help_params *p)
 {