'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>
Link: https://lore.kernel.org/r/20220210060411.1607928-1-chi.minghao@zte.com.cn
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
        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);