Re: [PATCH 2/9] [{mac|nl}80211] Add 2 new radar channel flags

["Luis R. Rodriguez" <lrodriguez@atheros.com>]

On Mon, Feb 28, 2011 at 08:47:06AM -0800, Bernhard Schmidt wrote:
> DFS introduces 2 new flags a channel requiring radar detection
> might take. The interference flag indicates that interference was
> detected either during CAC or in-service monitoring and the
> channel is not to be used for the NOP period. The clear flag
> indicates that during a full CAC no interference was detected and
> it is allowed to open a BSS on that channel. The case were both
> flags are set is used to indicate that interference has been
> detected and we are in progress of closing on that channel.

What does closing mean?

> Signed-off-by: Bernhard Schmidt <bernhard.schmidt@saxnet.de>

> ---
>  include/linux/nl80211.h |    6 ++++++
>  include/net/cfg80211.h  |   18 ++++++++++++------
>  net/mac80211/cfg.c      |    6 ++++++
>  net/mac80211/tx.c       |    6 ++++--
>  net/wireless/nl80211.c  |    4 ++++
>  5 files changed, 32 insertions(+), 8 deletions(-)
> 
> diff --git a/include/linux/nl80211.h b/include/linux/nl80211.h
> index e3c9ec7..2282f56 100644
> --- a/include/linux/nl80211.h
> +++ b/include/linux/nl80211.h
> @@ -1371,6 +1371,10 @@ enum nl80211_band_attr {
>   *	(100 * dBm).
>   * @NL80211_FREQUENCY_ATTR_MAX: highest frequency attribute number
>   *	currently defined
> + * @NL80211_FREQUENCY_ATTR_RADAR_CLEAR: during a full CAC no interference has
> + *	been detected.

Why do you need a flag for clear, can you simply count on the
lack of the flag as the "clear" state?

  Luis