Re: [PATCH] cfg80211: 80MHz (11ac) regulatory change

[Kalle Valo <kvalo@xxxxxxxxxx>]

Mahesh Palivela <maheshp@xxxxxxxxxxx> writes:

> 80MHz Regulatory changes for 11ac.
>
> Signed-off-by: Mahesh Palivela <maheshp@xxxxxxxxxxx>

[...]

> +static bool is_vht80_not_allowed(struct ieee80211_channel *chan)
> +{
> +	if (!chan)
> +		return true;
> +	if (chan->flags & IEEE80211_CHAN_DISABLED)
> +		return true;
> +	/* This would happen when regulatory rules disallow VHT80 completely */
> +	if (IEEE80211_CHAN_NO_VHT80 == (chan->flags & (IEEE80211_CHAN_NO_VHT80)))
> +		return true;
> +	return false;
> +}

I think it's better to change this to is_vht80_allowed(). The
not_allowed might trick someone.

-- 
Kalle Valo
--
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