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

Jiafei Pan posted 1 patch 1 month ago
drivers/remoteproc/imx_rproc.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
[PATCH v2] remoteproc: imx_rproc: Use device node name as processor name
Posted by Jiafei Pan 1 month ago
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
Posted by Mathieu Poirier 2 weeks, 3 days ago
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>
> ---
> 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(-)
>

Applied.

Thanks,
Mathieu
 
> 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
Posted by Peng Fan 1 month ago
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
>
Re: [PATCH v2] remoteproc: imx_rproc: Use device node name as processor name
Posted by Daniel Baluta 1 month ago
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;