linux-kernel - Re: [PATCH v6] remoteproc: Use of_reserved_mem_region

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <4c787312-2340-437d-9d52-9917d29e3eac@ti.com>
Date: Wed, 19 Nov 2025 13:32:18 +0530
From: Beleswar Prasad Padhi <b-padhi@...com>
To: "Rob Herring (Arm)" <robh@...nel.org>, Bjorn Andersson
	<andersson@...nel.org>, Mathieu Poirier <mathieu.poirier@...aro.org>, "Shawn
 Guo" <shawnguo@...nel.org>, Sascha Hauer <s.hauer@...gutronix.de>,
	"Pengutronix Kernel Team" <kernel@...gutronix.de>, Fabio Estevam
	<festevam@...il.com>, Geert Uytterhoeven <geert+renesas@...der.be>, Magnus
 Damm <magnus.damm@...il.com>, Patrice Chotard <patrice.chotard@...s.st.com>,
	Maxime Coquelin <mcoquelin.stm32@...il.com>, Alexandre Torgue
	<alexandre.torgue@...s.st.com>
CC: Arnaud Pouliquen <arnaud.pouliquen@...s.st.com>, Peng Fan
	<peng.fan@....com>, <linux-remoteproc@...r.kernel.org>,
	<imx@...ts.linux.dev>, <linux-arm-kernel@...ts.infradead.org>,
	<linux-kernel@...r.kernel.org>, <linux-arm-msm@...r.kernel.org>,
	<linux-renesas-soc@...r.kernel.org>,
	<linux-stm32@...md-mailman.stormreply.com>
Subject: Re: [PATCH v6] remoteproc: Use of_reserved_mem_region_* functions for
 "memory-region"


On 31/10/25 23:29, 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@...s.st.com>
> Tested-by: Peng Fan <peng.fan@....com> # i.MX93-11x11-EVK for imx_rproc.c
> Reviewed-by: Geert Uytterhoeven <geert+renesas@...der.be> # rcar
> Signed-off-by: Rob Herring (Arm) <robh@...nel.org>


All of the tests pass in our CI. Feel free to use:

Tested-by: Beleswar Padhi <b-padhi@...com> # TI

Thanks,
Beleswar


> ---
> v6:
>  - Rebase on v6.18-rc1
>  - The error in v5 was fixed in v6.17 commit aea70964b5a7
>    ("of: reserved_mem: Add missing IORESOURCE_MEM flag on resources")
>
> [...]