I'm curious if the code below (and nevermind that dev_dbg I added) should also check the omac_idx for STATION type, since it is possible to have lots of stations on an mt7915 radio? static void mt7915_update_bss_color(struct ieee80211_hw *hw, struct ieee80211_vif *vif, struct cfg80211_he_bss_color *bss_color) { struct mt7915_dev *dev = mt7915_hw_dev(hw); switch (vif->type) { case NL80211_IFTYPE_AP: { struct mt7915_vif *mvif = (struct mt7915_vif *)vif->drv_priv; if (mvif->mt76.omac_idx > HW_BSSID_MAX) { dev_dbg(dev->mt76.dev, "update-bss-color, omac idx: %d > HW_BSSID_MAX: %d, bss color NOT updated.\n", mvif->mt76.omac_idx, HW_BSSID_MAX); return; } fallthrough; } case NL80211_IFTYPE_STATION: mt7915_mcu_update_bss_color(dev, vif, bss_color); break; default: break; } } Thanks, Ben -- Ben Greear <greearb@xxxxxxxxxxxxxxx> Candela Technologies Inc http://www.candelatech.com