Skip to content

Commit

Permalink
mt76: mt7915: simplify mt7915_lmac_mapping
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
nbd168 committed Sep 4, 2020
1 parent b1f4256 commit 8f48855
Showing 1 changed file with 2 additions and 11 deletions.
13 changes: 2 additions & 11 deletions mt7915/mt7915.h
Expand Up @@ -252,17 +252,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;
Expand Down

0 comments on commit 8f48855

Please sign in to comment.