2017-01-18 10:47:28

by Jean-Pierre TOSONI

[permalink] [raw]
Subject: RE: ath9k/ath10k DFS testing / certification

Hi Adrian,

Also, about ath9k:

I don't know if it's relevant to you, but we had the same problem of
NO_IR/RADAR mismatch in ath9k.
We fixed this in mac80211 layer at that time. Here is the patch. We didn't
send it upstream since
We have no experience with other chipsets, and we didn't know whether it
would be generic enough.

--- a/net/mac80211/scan.c
+++ b/net/mac80211/scan.c
@@ -598,7 +598,7 @@ static int __ieee80211_start_scan(struct
ieee80211_hw_config(local, 0);

if ((req->channels[0]->flags &
- IEEE80211_CHAN_NO_IR) ||
+ (IEEE80211_CHAN_NO_IR | IEEE80211_CHAN_RADAR)) ||
!req->n_ssids) {
next_delay = IEEE80211_PASSIVE_CHANNEL_TIME;
} else {
@@ -646,7 +646,7 @@ ieee80211_scan_get_channel_time(struct i
* TODO: channel switching also consumes quite some time,
* add that delay as well to get a better estimation
*/
- if (chan->flags & IEEE80211_CHAN_NO_IR)
+ if (chan->flags & (IEEE80211_CHAN_NO_IR | IEEE80211_CHAN_RADAR))
return IEEE80211_PASSIVE_CHANNEL_TIME;
return local->scan_req->probe_delay_ticks +
local->scan_req->max_channel_time_ticks;
}
@@ -810,7 +810,7 @@ static void ieee80211_scan_state_set_cha
*
* In any case, it is not necessary for a passive scan.
*/
- if (chan->flags & IEEE80211_CHAN_NO_IR || !scan_req->n_ssids) {
+ if (chan->flags & (IEEE80211_CHAN_NO_IR | IEEE80211_CHAN_RADAR) ||
!scan_req->n_ssids) {
*next_delay = IEEE80211_PASSIVE_CHANNEL_TIME;
local->next_scan_state = SCAN_DECISION;
return;
--
Diff -u

> -----Message d'origine-----
> De?: ath10k [mailto:[email protected]] De la part de
> Adrian Chadd
> Envoy??: dimanche 15 janvier 2017 22:17
> ??: ath9k-devel; [email protected]
> Objet?: ath9k/ath10k DFS testing / certification
>
> hiya,
>
> I'm working on a set of things that will involve DFS certification for
> ath9k/ath10k. Initially it'll be for FCC but I'll branch out to the other
> regions shortly afterwards.
>
> I'd love to hear from anyone else who has done this and what their
> challenges were, including whether they have any local patches / tools
> that haven't yet been upstreamed.
>
> Thanks!
>
>
> -adrian
>
> _______________________________________________
> ath10k mailing list
> [email protected]
> http://lists.infradead.org/mailman/listinfo/ath10k