wifi: ath12k: Avoid multiple times configuring monitor filter

ath12k_mac_op_configure_filter() gets called multiple times during
interface bringup. Applying filter configuration from this function leads
to writing same filter configurations multiple times.

Resolve this issue by relocating the filter configuration to
ath12k_mac_config_mon_status_default(), which is invoked by both
ath12k_mac_op_start() and ath12k_mac_op_stop(). Additionally, set the
rxmon_disable flag to true when called from ath12k_mac_op_stop() to
disable the monitor destination ring.

Tested-on: QCN9274 hw2.0 PCI WLAN.WBE.1.3.1-00173-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: P Praneesh <quic_ppranees@quicinc.com>
Link: https://patch.msgid.link/20241223060132.3506372-4-quic_ppranees@quicinc.com
Signed-off-by: Jeff Johnson <jeff.johnson@oss.qualcomm.com>

diff --git a/drivers/net/wireless/ath/ath12k/mac.c b/drivers/net/wireless/ath/ath12k/mac.c
index a245eb6293fbc51a9375c98900b8d0de5d76a9e4..4fb7e235be66706931ec2ec2e1136b0f33f2fa3f 100644 (file)
--- a/drivers/net/wireless/ath/ath12k/mac.c
+++ b/drivers/net/wireless/ath/ath12k/mac.c
@@ -7336,8 +7336,36 @@ void ath12k_mac_drain_tx(struct ath12k *ar)
 
 static int ath12k_mac_config_mon_status_default(struct ath12k *ar, bool enable)
 {
-       return -EOPNOTSUPP;
-       /* TODO: Need to support new monitor mode */
+       struct htt_rx_ring_tlv_filter tlv_filter = {};
+       struct ath12k_base *ab = ar->ab;
+       u32 ring_id, i;
+       int ret = 0;
+
+       lockdep_assert_wiphy(ath12k_ar_to_hw(ar)->wiphy);
+
+       if (!ab->hw_params->rxdma1_enable)
+               return ret;
+
+       if (enable)
+               tlv_filter = ath12k_mac_mon_status_filter_default;
+       else
+               tlv_filter.rxmon_disable = true;
+
+       for (i = 0; i < ab->hw_params->num_rxdma_per_pdev; i++) {
+               ring_id = ar->dp.rxdma_mon_dst_ring[i].ring_id;
+               ret = ath12k_dp_tx_htt_rx_filter_setup(ab, ring_id,
+                                                      ar->dp.mac_id + i,
+                                                      HAL_RXDMA_MONITOR_DST,
+                                                      DP_RXDMA_REFILL_RING_SIZE,
+                                                      &tlv_filter);
+               if (ret) {
+                       ath12k_err(ab,
+                                  "failed to setup filter for monitor buf %d\n",
+                                  ret);
+               }
+       }
+
+       return ret;
 }
 
 static int ath12k_mac_start(struct ath12k *ar)
@@ -8468,29 +8496,6 @@ static void ath12k_mac_op_remove_interface(struct ieee80211_hw *hw,
        FIF_PROBE_REQ |                         \
        FIF_FCSFAIL)
 
-static void ath12k_mac_configure_filter(struct ath12k *ar,
-                                       unsigned int total_flags)
-{
-       bool reset_flag;
-       int ret;
-
-       lockdep_assert_wiphy(ath12k_ar_to_hw(ar)->wiphy);
-
-       ar->filter_flags = total_flags;
-
-       /* For monitor mode */
-       reset_flag = !(ar->filter_flags & FIF_BCN_PRBRESP_PROMISC);
-
-       ret = ath12k_dp_tx_htt_monitor_mode_ring_config(ar, reset_flag);
-       if (ret)
-               ath12k_warn(ar->ab,
-                           "fail to set monitor filter: %d\n", ret);
-
-       ath12k_dbg(ar->ab, ATH12K_DBG_MAC,
-                  "total_flags:0x%x, reset_flag:%d\n",
-                  total_flags, reset_flag);
-}
-
 static void ath12k_mac_op_configure_filter(struct ieee80211_hw *hw,
                                           unsigned int changed_flags,
                                           unsigned int *total_flags,
@@ -8504,7 +8509,7 @@ static void ath12k_mac_op_configure_filter(struct ieee80211_hw *hw,
        ar = ath12k_ah_to_ar(ah, 0);
 
        *total_flags &= SUPPORTED_FILTERS;
-       ath12k_mac_configure_filter(ar, *total_flags);
+       ar->filter_flags = *total_flags;
 }
 
 static int ath12k_mac_op_get_antenna(struct ieee80211_hw *hw, u32 *tx_ant, u32 *rx_ant)