Re: [PATCHv8 3/3] nl80211: allow DFS in start_ap

[Johannes Berg <johannes@xxxxxxxxxxxxxxxx>]

On Mon, 2013-02-04 at 13:49 +0100, Simon Wunderlich wrote:

> +	err = cfg80211_chandef_dfs_required(wdev->wiphy, &params.chandef);
> +	if (err < 0)
> +		return -EINVAL;

return err; ?

> +	if (err) {
> +		radar_detect_width = BIT(params.chandef.width);
> +		params.radar_required = true;

It seems you should probably just squash this into patch 1 since you
already use it in mac80211 in patch 2 but only patch 3 makes it valid?
seems a bit odd

johannes

--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html