mt76: mt7615: run mt7615_mcu_set_wmm holding mt76 mutex
authorLorenzo Bianconi <lorenzo@kernel.org>
Fri, 3 Jul 2020 08:15:51 +0000 (10:15 +0200)
committerFelix Fietkau <nbd@nbd.name>
Tue, 21 Jul 2020 17:01:17 +0000 (19:01 +0200)
Make sure to run mt7615_mcu_set_wmm() holding mt76 mutex in order to
wake the device from low power state and avoid races

Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org>
Signed-off-by: Felix Fietkau <nbd@nbd.name>
drivers/net/wireless/mediatek/mt76/mt7615/main.c

index eb56cd7..c52b5c6 100644 (file)
@@ -433,11 +433,17 @@ mt7615_conf_tx(struct ieee80211_hw *hw, struct ieee80211_vif *vif, u16 queue,
 {
        struct mt7615_vif *mvif = (struct mt7615_vif *)vif->drv_priv;
        struct mt7615_dev *dev = mt7615_hw_dev(hw);
+       int err;
+
+       mt7615_mutex_acquire(dev);
 
        queue = mt7615_lmac_mapping(dev, queue);
        queue += mvif->wmm_idx * MT7615_MAX_WMM_SETS;
+       err = mt7615_mcu_set_wmm(dev, queue, params);
 
-       return mt7615_mcu_set_wmm(dev, queue, params);
+       mt7615_mutex_release(dev);
+
+       return err;
 }
 
 static void mt7615_configure_filter(struct ieee80211_hw *hw,