Re: [PATCH v2 2/4] cfg80211: update missing fields in custom regulatory path

["Luis R. Rodriguez" <mcgrof@suse.com>]

On Thu, Nov 13, 2014 at 06:13:37PM +0200, Arik Nemtsov wrote:
> From: Jonathan Doron <jond@wizery.com>
> 
> Some channels fields were not being updated in the custom regulatory
> path. Update them according to the code in handle_channel().
> 
> Signed-off-by: Jonathan Doron <jonathanx.doron@intel.com>
> Signed-off-by: Arik Nemtsov <arikx.nemtsov@intel.com>
> ---
>  net/wireless/reg.c | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
> 
> diff --git a/net/wireless/reg.c b/net/wireless/reg.c
> index 6459ddd..174d8f82 100644
> --- a/net/wireless/reg.c
> +++ b/net/wireless/reg.c
> @@ -1693,10 +1693,23 @@ static void handle_channel_custom(struct wiphy *wiphy,
>  	if (max_bandwidth_khz < MHZ_TO_KHZ(160))
>  		bw_flags |= IEEE80211_CHAN_NO_160MHZ;
>  
> +	chan->dfs_state = NL80211_DFS_USABLE;

NL80211_DFS_USABLE is 0 so this is not needed. Being explicit about it
is OK I suppose though.

> +	chan->dfs_state_entered = jiffies;

OK.

> +
> +	chan->beacon_found = false;

false is 0 so this is not needed but being explicit about it is OK.

>  	chan->flags |= map_regdom_flags(reg_rule->flags) | bw_flags;
>  	chan->max_antenna_gain = (int) MBI_TO_DBI(power_rule->max_antenna_gain);
>  	chan->max_reg_power = chan->max_power =
>  		(int) MBM_TO_DBM(power_rule->max_eirp);
> +
> +	if (chan->flags & IEEE80211_CHAN_RADAR) {
> +		if (reg_rule->dfs_cac_ms)
> +			chan->dfs_cac_ms = reg_rule->dfs_cac_ms;
> +		else
> +			chan->dfs_cac_ms = IEEE80211_DFS_MIN_CAC_TIME_MS;
> +	}
> +
> +	chan->max_power = chan->max_reg_power;
>  }

I rather you split this up into the stuff not required (things which
are good to be explicit about) Vs possible fixes, these last are
good fixes. Also put them as part of your first set of patches
and add my Acked-by so Johannes can already merge them. If you want
them merged faster just send them separately now.

Acked-by: Luis R. Rodriguez <mcgrof@suse.com>

  Luis