[PATCH] remoteproc: imx_dsp: acquire runtime PM before adding carveouts

[Pengpeng Hou]

The prepare callback publishes DSP carveouts before acquiring the
runtime-PM reference that establishes required clocks and mailbox state.
A PM failure therefore leaves the prepare transaction partially
published.

Acquire runtime PM first and drop that reference if carveout
construction fails, so a failed prepare does not add resources before
its hardware prerequisite.

Fixes: ec0e5549f358 ("remoteproc: imx_dsp_rproc: Add remoteproc driver for DSP 
on i.MX")
Signed-off-by: Pengpeng Hou <pengp...@iscas.ac.cn>
---
 drivers/remoteproc/imx_dsp_rproc.c | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/drivers/remoteproc/imx_dsp_rproc.c 
b/drivers/remoteproc/imx_dsp_rproc.c
index fd60c67ba8a9f..96d619daf3f5b 100644
--- a/drivers/remoteproc/imx_dsp_rproc.c
+++ b/drivers/remoteproc/imx_dsp_rproc.c
@@ -726,14 +726,17 @@ static int imx_dsp_rproc_prepare(struct rproc *rproc)
        struct device *dev = rproc->dev.parent;
        int ret;
 
+       ret = pm_runtime_resume_and_get(dev);
+       if (ret < 0)
+               return ret;
+
        ret = imx_dsp_rproc_add_carveout(priv);
        if (ret) {
                dev_err(dev, "failed on imx_dsp_rproc_add_carveout\n");
+               pm_runtime_put_sync(dev);
                return ret;
        }
 
-       pm_runtime_get_sync(dev);
-
        return 0;
 }

base-commit: 08dbfad3f5040f5bdb6c529da20d6d4e81fefd72
-- 
2.50.1