Re: [PATCH v2 2/2] arm64: dts: exynos850: Add acpm SRAM node

Krzysztof Kozlowski <[email protected]> Fri, 24 Jul 2026 10:38:39 +0200

Newsgroups	org.kernel.vger.linux-samsung-soc,org.infradead.lists.linux-arm-kernel,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel
Message-ID	<20260724-hissing-fascinating-terrier-6b754f@quoll>

On Thu, Jul 23, 2026 at 04:29:47PM +0100, Alexey Klimov wrote:
> SRAM is used by the ACPM protocol to retrieve the ACPM channels
> information and configuration data. Add the SRAM node.
> 
> Signed-off-by: Alexey Klimov <[email protected]>
> ---
>  arch/arm64/boot/dts/exynos/exynos850.dtsi | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/exynos/exynos850.dtsi b/arch/arm64/boot/dts/exynos/exynos850.dtsi
> index 3881f573ec08..9a8d92aae9a8 100644
> --- a/arch/arm64/boot/dts/exynos/exynos850.dtsi
> +++ b/arch/arm64/boot/dts/exynos/exynos850.dtsi
> @@ -917,6 +917,11 @@ spi_2: spi@11d20000 {
>  			};
>  		};
>  	};
> +
> +	apm_sram: sram@2039000 {

Looks oddly placed, suggesting this is not part of SoC, but all MMIO
nodes must be part of the SoC if this is SoC. Compatible says this is
SoC.

Best regards,
Krzysztof