Re: [PATCH v2 0/4] of: Common "memory-region" parsing

[Peng Fan <peng.fan@oss.nxp.com>]

On Wed, Apr 23, 2025 at 02:42:12PM -0500, Rob Herring (Arm) wrote:
>While there's a common function to parse "memory-region" properties for
>DMA pool regions, there's not anything for driver private regions. As a
>result, drivers have resorted to parsing "memory-region" properties
>themselves repeating the same pattern over and over. To fix this, this
>series adds 2 functions to handle those cases:
>of_reserved_mem_region_to_resource() and of_reserved_mem_region_count().
>
>I've converted the whole tree, but just including remoteproc here as
>it has the most cases. I intend to apply the first 3 patches for 6.16
>so the driver conversions can be applied for 6.17.
>
>A git tree with all the drivers converted is here[1].
>
>v2:
>- Fix of_dma_set_restricted_buffer() to maintain behavior on warning msg
>- Export devm_ioremap_resource_wc()
>- Rework handling of resource name to drop unit-address from name as it 
>  was before.
>- Link to v1: 
>  https://lore.kernel.org/all/20250317232426.952188-1-robh@kernel.org
>
>[1] git://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git dt/memory-region
>
>Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
>---
>Rob Herring (Arm) (4):
>      of: reserved_mem: Add functions to parse "memory-region"
>      of: Simplify of_dma_set_restricted_buffer() to use of_for_each_phandle()
>      devres: Export devm_ioremap_resource_wc()
>      remoteproc: Use of_reserved_mem_region_* functions for "memory-region"
>
> drivers/of/device.c                       | 31 +++++-------
> drivers/of/of_reserved_mem.c              | 80 +++++++++++++++++++++++++++++++
> drivers/remoteproc/imx_dsp_rproc.c        | 45 +++++++----------
> drivers/remoteproc/imx_rproc.c            | 68 +++++++++++---------------

Tested-by: Peng Fan <peng.fan@nxp.com> (i.MX93-11x11-EVK for imx_rproc.c)

Thanks,
Peng