ath11k: support TXOP duration based RTS threshold

HE operation IE in beacons is constructed based on userspace params,
which firmware might not be aware of. This causes firmware not to
configure TXOP duration based RTS threshold which could cause mismatch
in behaviour with respect to what is being advertised in beacons. This
patch sends HE operation IE fetched from beacon to firmware using
WMI interface for configuration.

Tested-on: IPQ8074 hw2.0 AHB WLAN.HK.2.1.0.1-01228-QCAHKSWPL_SILICONZ-1

Signed-off-by: Pradeep Kumar Chitrapu <pradeepc@codeaurora.org>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/20201020183111.25458-4-pradeepc@codeaurora.org

diff --git a/drivers/net/wireless/ath/ath11k/mac.c b/drivers/net/wireless/ath/ath11k/mac.c
index 12cc16003e30d75c62036c33805d6beefd4362e9..e8b0b66d34e49c8b7d3b8a872a8a304ee2dba74a 100644 (file)
--- a/drivers/net/wireless/ath/ath11k/mac.c
+++ b/drivers/net/wireless/ath/ath11k/mac.c
@@ -1969,9 +1969,25 @@ static void ath11k_mac_op_bss_info_changed(struct ieee80211_hw *hw,
        if (changed & BSS_CHANGED_BSSID && !is_zero_ether_addr(info->bssid))
                ether_addr_copy(arvif->bssid, info->bssid);
 
-       if (changed & BSS_CHANGED_BEACON_ENABLED)
+       if (changed & BSS_CHANGED_BEACON_ENABLED) {
                ath11k_control_beaconing(arvif, info);
 
+               if (arvif->is_up && vif->bss_conf.he_support &&
+                   vif->bss_conf.he_oper.params) {
+                       param_id = WMI_VDEV_PARAM_HEOPS_0_31;
+                       param_value = vif->bss_conf.he_oper.params;
+                       ret = ath11k_wmi_vdev_set_param_cmd(ar, arvif->vdev_id,
+                                                           param_id, param_value);
+                       ath11k_dbg(ar->ab, ATH11K_DBG_MAC,
+                                  "he oper param: %x set for VDEV: %d\n",
+                                  param_value, arvif->vdev_id);
+
+                       if (ret)
+                               ath11k_warn(ar->ab, "Failed to set he oper params %x for VDEV %d: %i\n",
+                                           param_value, arvif->vdev_id, ret);
+               }
+       }
+
        if (changed & BSS_CHANGED_ERP_CTS_PROT) {
                u32 cts_prot;