[PATCH] ath11k: Avoid frequency offset based sanity check in radar detection event

[PATCH] ath11k: Avoid frequency offset based sanity check in radar detection event

[Sriram R]

Currently the detected radar is indicated to mac80211 only when the active
rx channel is indicated in the event through channel frequency and offset.
But in case there is a minor difference in this offset provided,say +/- 1,
the event will rejected.

Hence avoid having these checks to indicate radar to mac80211. Basically the
event from firmware is highly reliable and such sanity checks are not
necessary.

Signed-off-by: Sriram R <srirrama@codeaurora.org>
---
 drivers/net/wireless/ath/ath11k/wmi.c | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/drivers/net/wireless/ath/ath11k/wmi.c b/drivers/net/wireless/ath/ath11k/wmi.c
index 4f2ac21..90d9560 100644
--- a/drivers/net/wireless/ath/ath11k/wmi.c
+++ b/drivers/net/wireless/ath/ath11k/wmi.c
@@ -5809,12 +5809,6 @@ ath11k_wmi_pdev_dfs_radar_detected_event(struct ath11k_base *ab,
 		goto exit;
 	}
 
-	if (ar->rx_channel && ar->rx_channel->center_freq != (ev->chan_freq
-	    - ev->freq_offset)) {
-		ath11k_warn(ab, "Radar detected in non-operating channel");
-		goto exit;
-	}
-
 	ath11k_dbg(ar->ab, ATH11K_DBG_REG, "DFS Radar Detected in pdev %d\n",
 		   ev->pdev_id);
 
@@ -5826,6 +5820,7 @@ ath11k_wmi_pdev_dfs_radar_detected_event(struct ath11k_base *ab,
 exit:
 	kfree(tb);
 }
+
 static void ath11k_wmi_tlv_op_rx(struct ath11k_base *ab, struct sk_buff *skb)
 {
 	struct wmi_cmd_hdr *cmd_hdr;
-- 
2.7.4


_______________________________________________
ath11k mailing list
ath11k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath11k

Re: [PATCH] ath11k: Avoid frequency offset based sanity check in radar detection event

[Kalle Valo]

Sriram R <srirrama@codeaurora.org> wrote:

> Currently the detected radar is indicated to mac80211 only when the active
> rx channel is indicated in the event through channel frequency and offset.
> But in case there is a minor difference in this offset provided,say +/- 1,
> the event will rejected.
> 
> Hence avoid having these checks to indicate radar to mac80211. Basically the
> event from firmware is highly reliable and such sanity checks are not
> necessary.
> 
> Signed-off-by: Sriram R <srirrama@codeaurora.org>

Patch applied to ath.git, thanks.

48177eb4fb0e ath11k: Avoid frequency offset based sanity check in radar detection event

-- 
https://patchwork.kernel.org/patch/10963033/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches


_______________________________________________
ath11k mailing list
ath11k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath11k