wifi: rtw88: add missing destroy_workqueue() on error path in rtw_core_init()
authorYang Yingliang <yangyingliang@huawei.com>
Fri, 26 Aug 2022 02:38:17 +0000 (10:38 +0800)
committerKalle Valo <kvalo@kernel.org>
Fri, 2 Sep 2022 08:45:30 +0000 (11:45 +0300)
Add the missing destroy_workqueue() before return from rtw_core_init()
in error path.

Fixes: fe101716c7c9 ("rtw88: replace tx tasklet with work queue")
Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
Reviewed-by: Ping-Ke Shih <pkshih@realtek.com>
Signed-off-by: Kalle Valo <kvalo@kernel.org>
Link: https://lore.kernel.org/r/20220826023817.3908255-1-yangyingliang@huawei.com
drivers/net/wireless/realtek/rtw88/main.c

index 5a74dda977563a97e2e400476b7fe2942b60862f..67151dbf83842a783922d203431eeb4f05df3420 100644 (file)
@@ -2093,7 +2093,7 @@ int rtw_core_init(struct rtw_dev *rtwdev)
        ret = rtw_load_firmware(rtwdev, RTW_NORMAL_FW);
        if (ret) {
                rtw_warn(rtwdev, "no firmware loaded\n");
-               return ret;
+               goto out;
        }
 
        if (chip->wow_fw_name) {
@@ -2103,11 +2103,15 @@ int rtw_core_init(struct rtw_dev *rtwdev)
                        wait_for_completion(&rtwdev->fw.completion);
                        if (rtwdev->fw.firmware)
                                release_firmware(rtwdev->fw.firmware);
-                       return ret;
+                       goto out;
                }
        }
 
        return 0;
+
+out:
+       destroy_workqueue(rtwdev->tx_wq);
+       return ret;
 }
 EXPORT_SYMBOL(rtw_core_init);