staging: greybus: Remove redundant 'flush_workqueue()' calls
authorMinghao Chi (CGEL ZTE) <chi.minghao@zte.com.cn>
Thu, 10 Feb 2022 06:02:05 +0000 (06:02 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 15 Feb 2022 16:05:56 +0000 (17:05 +0100)
'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>
Reviewed-by: Rui Miguel Silva <rmfrfs@gmail.com>
Signed-off-by: Minghao Chi (CGEL ZTE) <chi.minghao@zte.com.cn>
Link: https://lore.kernel.org/r/20220210060205.1607792-1-chi.minghao@zte.com.cn
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/greybus/sdio.c

index 37bf04c22dbccf1c9e2a8ccbc3900e1aaa7a1fb8..25bee5335c7050130b90b4170df9dd2e57e0a23c 100644 (file)
@@ -858,7 +858,6 @@ static void gb_sdio_remove(struct gbphy_device *gbphy_dev)
        gb_connection_set_data(connection, NULL);
        mutex_unlock(&host->lock);
 
-       flush_workqueue(host->mrq_workqueue);
        destroy_workqueue(host->mrq_workqueue);
        gb_connection_disable_rx(connection);
        mmc_remove_host(mmc);