[rt2x00-users] [PATCH 7/7] rt2x00: Fix WMM Queue naming

Johannes Stezenbach js at sig21.net
Wed Dec 8 02:15:25 EST 2010


Hi,

On Tue, Nov 30, 2010 at 05:48:48PM +0100, Helmut Schaa wrote:
> Am Sonntag 28 November 2010 schrieb Ivo van Doorn:
> > The Queue names were incorrectly copied from the legacy drivers,
> > as a result the queue names were inversed to what was expected.
> 
> This comment is a bit misleading. The values were correctly copied
> from the legacy drivers but mac80211 uses a different mapping and since
> we just use mac80211 queue id's our queue names don't match their
> purpose.
> 
> legacy drivers:
> 
> QID_AC_BE               0
> QID_AC_BK               1
> QID_AC_VI               2
> QID_AC_VO               3
> 
> mac80211:
> 
> QID_AC_BE               3
> QID_AC_BK               2
> QID_AC_VI               1
> QID_AC_VO               0
> 
> Otherwise fine with me ...

It seems the patch which ended up in git has
QID_AC_BK and QID_AC_BE swapped compared to mac80211.

IEEE80211_AC_BE = 2, while QID_AC_BE = 3.


BTW, what is QID_MGMT?  There is only one usage in rt2800pci.c,
this does not look right.


Best Regards
Johannes



More information about the users mailing list