remoteproc: imx_rproc: Fix a resource leak in the remove function
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Sat, 16 Oct 2021 06:44:28 +0000 (08:44 +0200)
committerMathieu Poirier <mathieu.poirier@linaro.org>
Wed, 17 Nov 2021 16:36:27 +0000 (09:36 -0700)
'priv->workqueue' is destroyed in the error handling path of the probe but
not in the remove function.

Add the missing call to release some resources.

Cc: stable <stable@vger.kernel.org>
Fixes: 2df7062002d0 ("remoteproc: imx_proc: enable virtio/mailbox")
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Reviewed-by: Peng Fan <peng.fan@nxp.com>
Tested-by: Peng Fan <peng.fan@nxp.com>
Link: https://lore.kernel.org/r/d28ca94a4031bd7297d47c2164e18885a5a6ec19.1634366546.git.christophe.jaillet@wanadoo.fr
Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>
drivers/remoteproc/imx_rproc.c

index ff8170dbbc3c3e2ca2c5ed17cd682c2c87a98836..0a45bc0d3f73fc80f83f0655f88ef03946f2c024 100644 (file)
@@ -804,6 +804,7 @@ static int imx_rproc_remove(struct platform_device *pdev)
        clk_disable_unprepare(priv->clk);
        rproc_del(rproc);
        imx_rproc_free_mbox(rproc);
+       destroy_workqueue(priv->workqueue);
        rproc_free(rproc);
 
        return 0;