diff mbox series

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

Message ID 1558962845-12903-1-git-send-email-srirrama@codeaurora.org (mailing list archive)
State Accepted
Commit 48177eb4fb0e82b3ee0fc8f115aaeac3d283ca1c
Delegated to: Kalle Valo
Headers show
Series ath11k: Avoid frequency offset based sanity check in radar detection event | expand

Commit Message

Sriram R May 27, 2019, 1:14 p.m. UTC
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(-)

Comments

Kalle Valo May 29, 2019, 3:08 p.m. UTC | #1
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
diff mbox series

Patch

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;