]> www.infradead.org Git - users/willy/xarray.git/commitdiff
mt76: mt7921: move hw configuration in mt7921_register_device
authorLorenzo Bianconi <lorenzo@kernel.org>
Thu, 15 Apr 2021 09:03:58 +0000 (11:03 +0200)
committerFelix Fietkau <nbd@nbd.name>
Wed, 21 Apr 2021 09:42:22 +0000 (11:42 +0200)
Get rid of init work since firmware loading is already performed in
mt7921_init_hardware

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

index 0aedddb908589016b23a30b05d71b9d3c677a584..eab6e2dcdb9646f057dbbf412c31ce969ff18d93 100644 (file)
@@ -166,20 +166,10 @@ void mt7921_mac_init(struct mt7921_dev *dev)
        mt76_connac_mcu_set_rts_thresh(&dev->mt76, 0x92b, 0);
 }
 
-static void mt7921_init_work(struct work_struct *work)
-{
-       struct mt7921_dev *dev = container_of(work, struct mt7921_dev,
-                                init_work);
-
-       mt7921_mcu_set_eeprom(dev);
-       mt7921_mac_init(dev);
-}
-
 static int mt7921_init_hardware(struct mt7921_dev *dev)
 {
        int ret, idx;
 
-       INIT_WORK(&dev->init_work, mt7921_init_work);
        spin_lock_init(&dev->token_lock);
        idr_init(&dev->token);
 
@@ -202,6 +192,10 @@ static int mt7921_init_hardware(struct mt7921_dev *dev)
        if (ret < 0)
                return ret;
 
+       ret = mt7921_mcu_set_eeprom(dev);
+       if (ret)
+               return ret;
+
        /* Beacon and mgmt frames should occupy wcid 0 */
        idx = mt76_wcid_alloc(dev->mt76.wcid_mask, MT7921_WTBL_STA - 1);
        if (idx)
@@ -212,6 +206,8 @@ static int mt7921_init_hardware(struct mt7921_dev *dev)
        dev->mt76.global_wcid.tx_info |= MT_WCID_TX_INFO_SET;
        rcu_assign_pointer(dev->mt76.wcid[idx], &dev->mt76.global_wcid);
 
+       mt7921_mac_init(dev);
+
        return 0;
 }
 
@@ -266,8 +262,6 @@ int mt7921_register_device(struct mt7921_dev *dev)
        if (ret)
                return ret;
 
-       ieee80211_queue_work(mt76_hw(dev), &dev->init_work);
-
        return mt7921_init_debugfs(dev);
 }
 
index 5cc01efee989f509c2bc67c275ce16e17c9e25df..c34cf3e3a26b2f3f93f5845210ba57fb729fbb10 100644 (file)
@@ -156,7 +156,6 @@ struct mt7921_dev {
 
        u16 chainmask;
 
-       struct work_struct init_work;
        struct work_struct reset_work;
 
        struct list_head sta_poll_list;