mt76: mt7915: simplify mt7915_lmac_mapping
authorFelix Fietkau <nbd@nbd.name>
Tue, 1 Sep 2020 19:26:33 +0000 (21:26 +0200)
committerFelix Fietkau <nbd@nbd.name>
Thu, 24 Sep 2020 16:10:18 +0000 (18:10 +0200)
Compared to mac80211 ACs, MT7915 queue numbers are in reverse order
There is no need for the defensive WARN_ON_ONCE, so we can simplify
the function to avoid the array lookup

Signed-off-by: Felix Fietkau <nbd@nbd.name>
drivers/net/wireless/mediatek/mt76/mt7915/mt7915.h

index ab3f209..4b8908f 100644 (file)
@@ -258,17 +258,8 @@ mt7915_ext_phy(struct mt7915_dev *dev)
 
 static inline u8 mt7915_lmac_mapping(struct mt7915_dev *dev, u8 ac)
 {
-       static const u8 lmac_queue_map[] = {
-               [IEEE80211_AC_BK] = MT_LMAC_AC00,
-               [IEEE80211_AC_BE] = MT_LMAC_AC01,
-               [IEEE80211_AC_VI] = MT_LMAC_AC02,
-               [IEEE80211_AC_VO] = MT_LMAC_AC03,
-       };
-
-       if (WARN_ON_ONCE(ac >= ARRAY_SIZE(lmac_queue_map)))
-               return MT_LMAC_AC01; /* BE */
-
-       return lmac_queue_map[ac];
+       /* LMAC uses the reverse order of mac80211 AC indexes */
+       return 3 - ac;
 }
 
 extern const struct ieee80211_ops mt7915_ops;