wifi: ath12k: Modify rts threshold mac op for single wiphy

Since multiple radios are abstracted under a single wiphy,
apply the rts threshold value to all the vdevs of the radios
combined under single wiphy.

This also implies that vif specific rts threshold support
needs to be added in future from cfg80211/mac80211.

Tested-on: QCN9274 hw2.0 PCI WLAN.WBE.1.0.1-00029-QCAHKSWPL_SILICONZ-1
Tested-on: WCN7850 hw2.0 PCI WLAN.HMT.1.0.c5-00481-QCAHMTSWPL_V1.0_V2.0_SILICONZ-3

Signed-off-by: Sriram R <quic_srirrama@quicinc.com>
Signed-off-by: Rameshkumar Sundaram <quic_ramess@quicinc.com>
Acked-by: Jeff Johnson <quic_jjohnson@quicinc.com>
Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
Link: https://msgid.link/20240405194519.1337906-12-quic_ramess@quicinc.com

diff --git a/drivers/net/wireless/ath/ath12k/mac.c b/drivers/net/wireless/ath/ath12k/mac.c
index 77abd46a69d312ae14b9f5826ad8b36bfc4ebdce..72495136022bcece58411ed8a3e21bac5333c769 100644 (file)
--- a/drivers/net/wireless/ath/ath12k/mac.c
+++ b/drivers/net/wireless/ath/ath12k/mac.c
@@ -7322,11 +7322,21 @@ static int ath12k_mac_op_set_rts_threshold(struct ieee80211_hw *hw, u32 value)
 {
        struct ath12k_hw *ah = ath12k_hw_to_ah(hw);
        struct ath12k *ar;
-       int param_id = WMI_VDEV_PARAM_RTS_THRESHOLD, ret;
+       int param_id = WMI_VDEV_PARAM_RTS_THRESHOLD, ret = 0, i;
 
-       ar = ath12k_ah_to_ar(ah, 0);
-
-       ret = ath12k_set_vdev_param_to_all_vifs(ar, param_id, value);
+       /* Currently we set the rts threshold value to all the vifs across
+        * all radios of the single wiphy.
+        * TODO Once support for vif specific RTS threshold in mac80211 is
+        * available, ath12k can make use of it.
+        */
+       for_each_ar(ah, ar, i) {
+               ret = ath12k_set_vdev_param_to_all_vifs(ar, param_id, value);
+               if (ret) {
+                       ath12k_warn(ar->ab, "failed to set RTS config for all vdevs of pdev %d",
+                                   ar->pdev->pdev_id);
+                       break;
+               }
+       }
 
        return ret;
 }