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

[Mathieu Poirier]

On Sun, Aug 30, 2026 at 09:33:10PM +0800, Pengpeng Hou wrote:
> 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);
> -

Applied.

Thanks,
Mathieu

>       return 0;
>  }
> 
> base-commit: 08dbfad3f5040f5bdb6c529da20d6d4e81fefd72
> -- 
> 2.50.1
>