Re: [PATCH 4/9] [cfg80211] add preliminary radar processing code

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

On Mon, Feb 28, 2011 at 08:48:49AM -0800, Bernhard Schmidt wrote:
> --- /dev/null
> +++ b/net/wireless/radar.c
> @@ -0,0 +1,86 @@
> +/*
> + * Radar handling
> + *
> + * Copyright 2011 Bernhard Schmidt <bernhard.schmidt@saxnet.de>
> + *
> + * This program is free software; you can redistribute it and/or modify
> + * it under the terms of the GNU General Public License version 2 as
> + * published by the Free Software Foundation.
> + */
> +
> +#include <linux/ieee80211.h>
> +#include <net/cfg80211.h>
> +#include "radar.h"
> +
> +static struct radar_parameters regdomain_params[] = {
> +	{ 60, 1800, 1000 },	/* FCC, correct? */
> +	{ 60, 1800, 1000 },	/* ETSI */
> +	{ 60, 1800, 1000 },	/* JP, correct? */
> +};

If FCC or JP are not supported, I rather code not be present for it, or some
sort of flag for now to allow the radar enablement thingy to fail all the
time. In fact if FCC / JP regions are not supported you should not even
be able to operate in the mode of operation desired if your target channel
is a DFS one. If you do enter into an allowed non-DFS channel for that region
and you later try to switch to a DFS chanel then we'd hit another failure.

  Luis