On Mon, Jan 26, 2015 at 10:26 AM, Johannes Berg
<johannes@xxxxxxxxxxxxxxxx> wrote:
> On Sun, 2015-01-25 at 12:30 +0200, Arik Nemtsov wrote:
>
>> > + switch (chandef->width) {
>> > + case NL80211_CHAN_WIDTH_80:
>> > + vht_oper->chan_width = IEEE80211_VHT_CHANWIDTH_80MHZ;
>> > + break;
>> > + case NL80211_CHAN_WIDTH_80P80:
>> > + vht_oper->chan_width = IEEE80211_VHT_CHANWIDTH_80P80MHZ;
>> > + vht_oper->center_freq_seg2_idx =
>> > + ieee80211_frequency_to_channel(chandef->center_freq2);
>> > + break;
>> > + case NL80211_CHAN_WIDTH_160:
>> > + vht_oper->chan_width = IEEE80211_VHT_CHANWIDTH_160MHZ;
>> > + break;
>> > + default:
>> > + return pos;
>> > + }
>>
>> Shouldn't you also set vht_oper->basic_mcs_set here? Of course if you
>> have no use for it in IBSS I can add it later.
>
> Why would you want to require VHT rates?
Are you sure it's required and no the other way around in this case?
As in specifying which rates are not supported.
Not sure it means the same thing as in the HT instance.
Arik
--
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
