RE: [PATCH] remoteproc: imx: use device node name as processor name

RE: [PATCH] remoteproc: imx: use device node name as processor name

[Jiafei Pan]

> On 4/30/26 09:17, Jiafei Pan wrote:
> > As currently there are maybe multiple remote processors, so change
> > from using fixed name to using device node name as remote processor
> > name in order to make them can be distinguished by through of name in
> > sys filesystem.
> >
> > Signed-off-by: Jiafei Pan <Jiafei.Pan@nxp.com>
> 
> Please use git log --oneline drivers/remoteproc/imx_rproc.c and check the prefix
> subject for commit messges related to imx_rproc files.
> 
> 
> 
> 9b2451658afa9 remoteproc: imx_rproc: Use devm_add_action_or_reset() for
> scu cleanup 65af722aa86fc remoteproc: imx_rproc: Use
> devm_clk_get_enabled() and simplify cleanup
> b0106defc0ff6 remoteproc: imx_rproc: Use devm_add_action_or_reset() for
> mailbox cleanup
> 
> Thanks,
> Daniel.

Updated, thanks.
Jiafei.

[PATCH v2] remoteproc: imx_rproc: Use device node name as processor name

[Jiafei Pan]

As currently there are maybe multiple remote processors, so change from
using fixed name to using device node name as remote processor name in
order to make them can be distinguished by through of name in sys
filesystem.

Signed-off-by: Jiafei Pan <Jiafei.Pan@nxp.com>
---
Fixes sine v1:
* Update patch subject to aligin prefix name with existing patches

 drivers/remoteproc/imx_rproc.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/remoteproc/imx_rproc.c b/drivers/remoteproc/imx_rproc.c
index e8d239bef5c9..38713f6f1c50 100644
--- a/drivers/remoteproc/imx_rproc.c
+++ b/drivers/remoteproc/imx_rproc.c
@@ -1356,8 +1356,7 @@ static int imx_rproc_probe(struct platform_device *pdev)
 	unsigned long cpus_bits;
 	int ret;
 
-	/* set some other name then imx */
-	rproc = devm_rproc_alloc(dev, "imx-rproc", &imx_rproc_ops,
+	rproc = devm_rproc_alloc(dev, np->name, &imx_rproc_ops,
 				 NULL, sizeof(*priv));
 	if (!rproc)
 		return -ENOMEM;
-- 
2.43.0

[PATCH v2] remoteproc: imx_rproc: Use device node name as processor name

[Jiafei Pan]

As currently there are maybe multiple remote processors, so change from
using fixed name to using device node name as remote processor name in
order to make them can be distinguished by through of name in sys
filesystem.

Signed-off-by: Jiafei Pan <Jiafei.Pan@nxp.com>
---
Fixes sine v1:
* Update patch subject to aligin prefix name with existing patches

 drivers/remoteproc/imx_rproc.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/remoteproc/imx_rproc.c b/drivers/remoteproc/imx_rproc.c
index e8d239bef5c9..38713f6f1c50 100644
--- a/drivers/remoteproc/imx_rproc.c
+++ b/drivers/remoteproc/imx_rproc.c
@@ -1356,8 +1356,7 @@ static int imx_rproc_probe(struct platform_device *pdev)
 	unsigned long cpus_bits;
 	int ret;
 
-	/* set some other name then imx */
-	rproc = devm_rproc_alloc(dev, "imx-rproc", &imx_rproc_ops,
+	rproc = devm_rproc_alloc(dev, np->name, &imx_rproc_ops,
 				 NULL, sizeof(*priv));
 	if (!rproc)
 		return -ENOMEM;
-- 
2.43.0

Re: [PATCH v2] remoteproc: imx_rproc: Use device node name as processor name

[Daniel Baluta]

On 5/8/26 06:20, Jiafei Pan wrote:
> As currently there are maybe multiple remote processors, so change from
> using fixed name to using device node name as remote processor name in
> order to make them can be distinguished by through of name in sys
> filesystem.
> 
> Signed-off-by: Jiafei Pan <Jiafei.Pan@nxp.com>

Reviewed-by: Daniel Baluta <daniel.baluta@nxp.com>

> ---
> Fixes sine v1:
> * Update patch subject to aligin prefix name with existing patches
> 
>  drivers/remoteproc/imx_rproc.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/drivers/remoteproc/imx_rproc.c b/drivers/remoteproc/imx_rproc.c
> index e8d239bef5c9..38713f6f1c50 100644
> --- a/drivers/remoteproc/imx_rproc.c
> +++ b/drivers/remoteproc/imx_rproc.c
> @@ -1356,8 +1356,7 @@ static int imx_rproc_probe(struct platform_device *pdev)
>  	unsigned long cpus_bits;
>  	int ret;
>  
> -	/* set some other name then imx */
> -	rproc = devm_rproc_alloc(dev, "imx-rproc", &imx_rproc_ops,
> +	rproc = devm_rproc_alloc(dev, np->name, &imx_rproc_ops,
>  				 NULL, sizeof(*priv));
>  	if (!rproc)
>  		return -ENOMEM;

Re: [PATCH v2] remoteproc: imx_rproc: Use device node name as processor name

[Peng Fan]

On Fri, May 08, 2026 at 11:20:16AM +0800, Jiafei Pan wrote:
>As currently there are maybe multiple remote processors, so change from
>using fixed name to using device node name as remote processor name in
>order to make them can be distinguished by through of name in sys
>filesystem.
>
>Signed-off-by: Jiafei Pan <Jiafei.Pan@nxp.com>

Reviewed-by: Peng Fan <peng.fan@nxp.com>

BTW:
please don't create thread with v1.

2026-04-30  6:17 [PATCH] remoteproc: imx: use device node name as processor name Jiafei Pan
2026-04-30 14:17 ` Daniel Baluta
2026-05-08  3:16   ` Jiafei Pan
2026-05-08  3:18   ` [PATCH v2] remoteproc: imx_rproc: Use " Jiafei Pan
2026-05-08  3:20   ` Jiafei Pan
2026-05-08  6:28     ` Daniel Baluta

https://lore.kernel.org/imx/ac390c82-91d9-4d8c-9514-b7cac0a4dd8a@oss.nxp.com/T/#mf211972b33053e6cb0b4adbe1002623d911e0214

Regards
Peng

>---
>Fixes sine v1:
>* Update patch subject to aligin prefix name with existing patches
>
> drivers/remoteproc/imx_rproc.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
>diff --git a/drivers/remoteproc/imx_rproc.c b/drivers/remoteproc/imx_rproc.c
>index e8d239bef5c9..38713f6f1c50 100644
>--- a/drivers/remoteproc/imx_rproc.c
>+++ b/drivers/remoteproc/imx_rproc.c
>@@ -1356,8 +1356,7 @@ static int imx_rproc_probe(struct platform_device *pdev)
> 	unsigned long cpus_bits;
> 	int ret;
> 
>-	/* set some other name then imx */
>-	rproc = devm_rproc_alloc(dev, "imx-rproc", &imx_rproc_ops,
>+	rproc = devm_rproc_alloc(dev, np->name, &imx_rproc_ops,
> 				 NULL, sizeof(*priv));
> 	if (!rproc)
> 		return -ENOMEM;
>-- 
>2.43.0
>