[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <lapjaf4edsuvnw22khjiuzhtiwrxfyiijcwj6piw5n5yby5ik3@r2jvfgvx2gqt>
Date: Sun, 4 May 2025 17:48:33 +0300
From: Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>
To: Alexey Minnekhanov <alexeymin@...tmarketos.org>
Cc: Bjorn Andersson <andersson@...nel.org>,
Konrad Dybcio <konradybcio@...nel.org>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>,
Dmitry Baryshkov <lumag@...nel.org>, Dang Huynh <danct12@...eup.net>,
Alexey Minnekhanov <alexey.min@...il.com>,
Marijn Suijten <marijn.suijten@...ainline.org>,
linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, ~postmarketos/upstreaming@...ts.sr.ht
Subject: Re: [PATCH 1/3] arm64: dts: qcom: sdm630: Add modem metadata mem
On Sun, May 04, 2025 at 02:51:18PM +0300, Alexey Minnekhanov wrote:
> Similarly to MSM8998, add and use modem metadata memory region.
> This does not seemingly affect device functionality. But it fixes
> DTBs check warning:
>
> remoteproc@...0000: memory-region: [[45], [46]] is too short
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>
>
> Signed-off-by: Alexey Minnekhanov <alexeymin@...tmarketos.org>
> ---
> arch/arm64/boot/dts/qcom/sdm630.dtsi | 8 +++++++-
> 1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sdm630.dtsi b/arch/arm64/boot/dts/qcom/sdm630.dtsi
> index 2d3820536ddf4..8b1a45a4e56ed 100644
> --- a/arch/arm64/boot/dts/qcom/sdm630.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sdm630.dtsi
> @@ -509,6 +509,12 @@ zap_shader_region: gpu@...00000 {
> reg = <0x0 0xfed00000 0x0 0xa00000>;
> no-map;
> };
> +
> + mdata_mem: mpss-metadata {
> + alloc-ranges = <0x0 0xa0000000 0x0 0x20000000>;
> + size = <0x0 0x4000>;
> + no-map;
> + };
> };
>
> smem: smem {
> @@ -1056,7 +1062,7 @@ remoteproc_mss: remoteproc@...0000 {
> <&rpmpd SDM660_VDDMX>;
> power-domain-names = "cx", "mx";
>
> - memory-region = <&mba_region>, <&mpss_region>;
> + memory-region = <&mba_region>, <&mpss_region>, <&mdata_mem>;
>
> status = "disabled";
>
> --
> 2.49.0
>
--
With best wishes
Dmitry
Powered by blists - more mailing lists