Re: [PATCH v5] remoteproc: Use of_reserved_mem_region_* functions for "memory-region"

On Tue, Aug 19, 2025 at 04:38:27PM -0500, Rob Herring (Arm) wrote:

Use the newly added of_reserved_mem_region_to_resource() and
of_reserved_mem_region_count() functions to handle "memory-region"
properties.

The error handling is a bit different in some cases. Often
"memory-region" is optional, so failed lookup is not an error. But then
an error in of_reserved_mem_lookup() is treated as an error. However,
that distinction is not really important. Either the region is available
and usable or it is not. So now, it is just
of_reserved_mem_region_to_resource() which is checked for an error.

Acked-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com>
Tested-by: Peng Fan <peng.fan@nxp.com> # i.MX93-11x11-EVK for imx_rproc.c
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> # rcar
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
v5:
 - Drop some inadvertent whitespace changes in qcom_q6v5_pas.c and ti_k3.
 - Fix error handling in adsp_alloc_memory_region()
 - Drop unreachable returns after 'while (1)' loops

v4:
 - Rebase on v6.17-rc1. qcom_q6v5_pas.c conflicted needing s/adsp/pas/

v3:
 - Rebase on v6.16-rc1. Move TI K3 changes to new common file.
 - Fix double increment of "i" in xlnx_r5

v2:
 - Use strstarts instead of strcmp for resource names as they include
   the unit-address.
 - Drop the unit-address from resource name for imx and st drivers
---
 drivers/remoteproc/imx_dsp_rproc.c      | 47 +++++++----------
 drivers/remoteproc/imx_rproc.c          | 70 ++++++++++---------------
 drivers/remoteproc/qcom_q6v5_adsp.c     | 24 ++++-----
 drivers/remoteproc/qcom_q6v5_mss.c      | 60 +++++++--------------
 drivers/remoteproc/qcom_q6v5_pas.c      | 69 +++++++++---------------
 drivers/remoteproc/qcom_q6v5_wcss.c     | 25 ++++-----
 drivers/remoteproc/qcom_wcnss.c         | 23 ++++----
 drivers/remoteproc/rcar_rproc.c         | 38 +++++---------
 drivers/remoteproc/st_remoteproc.c      | 41 +++++++--------
 drivers/remoteproc/stm32_rproc.c        | 46 +++++++---------
 drivers/remoteproc/ti_k3_common.c       | 28 ++++------
 drivers/remoteproc/xlnx_r5_remoteproc.c | 53 +++++++------------
 12 files changed, 199 insertions(+), 325 deletions(-)

Reviewed-by: Dmitry Baryshkov <redacted> # msm


-- 
With best wishes
Dmitry