Messages in this thread Patch in this message | | | From | cgel.zte@gmail ... | Subject | [PATCH] staging/ks7010: Remove redundant 'flush_workqueue()' calls | Date | Thu, 10 Feb 2022 06:04:11 +0000 |
| |
From: "Minghao Chi (CGEL ZTE)" <chi.minghao@zte.com.cn>
'destroy_workqueue()' already drains the queue before destroying it, so there is no need to flush it explicitly.
Remove the redundant 'flush_workqueue()' calls.
Reported-by: Zeal Robot <zealci@zte.com.cn> Signed-off-by: Minghao Chi (CGEL ZTE) <chi.minghao@zte.com.cn> --- drivers/staging/ks7010/ks7010_sdio.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/staging/ks7010/ks7010_sdio.c b/drivers/staging/ks7010/ks7010_sdio.c index 98d759e7cc95..9fb118e77a1f 100644 --- a/drivers/staging/ks7010/ks7010_sdio.c +++ b/drivers/staging/ks7010/ks7010_sdio.c @@ -1102,10 +1102,8 @@ static void ks7010_sdio_remove(struct sdio_func *func) if (ret) /* memory allocation failure */ goto err_free_card; - if (priv->wq) { - flush_workqueue(priv->wq); + if (priv->wq) destroy_workqueue(priv->wq); - } hostif_exit(priv); -- 2.25.1
| |