Re: [RFC v2 4/8] nl80211/cfg80211: support VHT channel configuration

[Mahesh Palivela <maheshp@posedge.com>]

On 11/10/2012 01:47 AM, Johannes Berg wrote:
> From: Johannes Berg <johannes.berg@intel.com>
>

> + *
> + * @NL80211_CHAN_WIDTH_20_NOHT: 20 MHz, non-HT channel

Should we have NL80211_CHAN_WIDTH_LEGACY or rename 
NL80211_CHAN_WIDTH_20_NOHT to NL80211_CHAN_WIDTH_NOHT to cover non HT 
associations in both 2.4 & 5 GHz bands.

> + * @NL80211_CHAN_WIDTH_20: 20 MHz HT channel
> + * @NL80211_CHAN_WIDTH_40: 40 MHz channel, the %NL80211_ATTR_CENTER_FREQ1
> + *	attribute must be provided as well
> + * @NL80211_CHAN_WIDTH_80: 80 MHz channel, the %NL80211_ATTR_CENTER_FREQ1
> + *	attribute must be provided as well
> + * @NL80211_CHAN_WIDTH_160: 160 MHz channel, the %NL80211_ATTR_CENTER_FREQ1
> + *	attribute must be provided as well
> + * @NL80211_CHAN_WIDTH_80P80: 80+80 MHz channel, the %NL80211_ATTR_CENTER_FREQ1
> + *	and %NL80211_ATTR_CENTER_FREQ2 attributes must be provided as well
> + */
> +enum nl80211_chan_width {
> +	NL80211_CHAN_WIDTH_20_NOHT,
> +	NL80211_CHAN_WIDTH_20,
> +	NL80211_CHAN_WIDTH_40,
> +	NL80211_CHAN_WIDTH_80,
> +	NL80211_CHAN_WIDTH_160,
> +	NL80211_CHAN_WIDTH_80P80,
> +};