[PATCH 10/13] nl80211: implement add_ts/del_ts ops

Eliad Peller eliad
Wed Oct 22 01:15:17 PDT 2014


On Wed, Oct 22, 2014 at 10:31 AM, Johannes Berg
<johannes at sipsolutions.net> wrote:
> On Wed, 2014-10-22 at 08:50 +0200, Johannes Berg wrote:
>> > +           case NL80211_CMD_ADD_TX_TS:
>> > +                   info->wmm_ac_supported = 1;
>> > +                   break;
>>
>> I believe this is wrong as I defined the command to cover both WMM and
>> 802.11 TSPEC, you need to check the WMM-specific feature flag instead.
>
> I'm an idiot, I made it a wiphy feature flag.
>
> It seems better if we were to make that flag visible, just in case
> somebody actually wants 802.11 tspec in the future and implements only
> that?
>
> I'll change the kernel code to advertise the feature flag so you can use
> it.
>
thanks, i'll use it.

Eliad.



More information about the Hostap mailing list