[PATCH v2 6/6] arm64: dts: qcom: r0q: fix reserved memory regions

Eric Gonçalves posted 6 patches 2 months ago
[PATCH v2 6/6] arm64: dts: qcom: r0q: fix reserved memory regions
Posted by Eric Gonçalves 2 months ago
ADSP and video memory regions in SoC dtsi is misplaced on this
platform, fix them by deleting those nodes and redefining them.

Signed-off-by: Eric Gonçalves <ghatto404@gmail.com>
---
 arch/arm64/boot/dts/qcom/sm8450-samsung-r0q.dts | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/arch/arm64/boot/dts/qcom/sm8450-samsung-r0q.dts b/arch/arm64/boot/dts/qcom/sm8450-samsung-r0q.dts
index a3b81403d180..8ed8a67aae0d 100644
--- a/arch/arm64/boot/dts/qcom/sm8450-samsung-r0q.dts
+++ b/arch/arm64/boot/dts/qcom/sm8450-samsung-r0q.dts
@@ -10,6 +10,9 @@
 #include "pm8350c.dtsi"
 #include "pmk8350.dtsi"
 
+/delete-node/ &adsp_mem;
+/delete-node/ &video_mem;
+
 / {
 	model = "Samsung Galaxy S22 5G";
 	compatible = "samsung,r0q", "qcom,sm8450";
@@ -64,6 +67,16 @@ splash-region@b8000000 {
 			reg = <0x0 0xb8000000 0x0 0x2b00000>;
 			no-map;
 		};
+
+		adsp_mem: memory@84500000 {
+			reg = <0x0 0x84500000 0x0 0x3b00000>;
+			no-map;
+		};
+
+		video_mem: memory@83e00000 {
+			reg = <0x0 0x83e00000 0x0 0x700000>;
+			no-map;
+		};
 	};
 
 	/*
-- 
2.51.0

Re: [PATCH v2 6/6] arm64: dts: qcom: r0q: fix reserved memory regions
Posted by Konrad Dybcio 2 months ago
On 10/14/25 6:41 AM, Eric Gonçalves wrote:
> ADSP and video memory regions in SoC dtsi is misplaced on this
> platform, fix them by deleting those nodes and redefining them.
> 
> Signed-off-by: Eric Gonçalves <ghatto404@gmail.com>
> ---
>  arch/arm64/boot/dts/qcom/sm8450-samsung-r0q.dts | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
> 
> diff --git a/arch/arm64/boot/dts/qcom/sm8450-samsung-r0q.dts b/arch/arm64/boot/dts/qcom/sm8450-samsung-r0q.dts
> index a3b81403d180..8ed8a67aae0d 100644
> --- a/arch/arm64/boot/dts/qcom/sm8450-samsung-r0q.dts
> +++ b/arch/arm64/boot/dts/qcom/sm8450-samsung-r0q.dts
> @@ -10,6 +10,9 @@
>  #include "pm8350c.dtsi"
>  #include "pmk8350.dtsi"
>  
> +/delete-node/ &adsp_mem;
> +/delete-node/ &video_mem;
> +
>  / {
>  	model = "Samsung Galaxy S22 5G";
>  	compatible = "samsung,r0q", "qcom,sm8450";
> @@ -64,6 +67,16 @@ splash-region@b8000000 {
>  			reg = <0x0 0xb8000000 0x0 0x2b00000>;
>  			no-map;
>  		};
> +
> +		adsp_mem: memory@84500000 {
> +			reg = <0x0 0x84500000 0x0 0x3b00000>;
> +			no-map;
> +		};
> +
> +		video_mem: memory@83e00000 {
> +			reg = <0x0 0x83e00000 0x0 0x700000>;
> +			no-map;
> +		};

Please keep the entries sorted, also with regards to the existing ones

Konrad
Re: [PATCH v2 6/6] arm64: dts: qcom: r0q: fix reserved memory regions
Posted by Ghatto 2 months ago
On Tue, Oct 14, 2025 at 7:02 AM Konrad Dybcio
<konrad.dybcio@oss.qualcomm.com> wrote:
>
> On 10/14/25 6:41 AM, Eric Gonçalves wrote:
> > ADSP and video memory regions in SoC dtsi is misplaced on this
> > platform, fix them by deleting those nodes and redefining them.
> >
> > Signed-off-by: Eric Gonçalves <ghatto404@gmail.com>
> > ---
> >  arch/arm64/boot/dts/qcom/sm8450-samsung-r0q.dts | 13 +++++++++++++
> >  1 file changed, 13 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/qcom/sm8450-samsung-r0q.dts b/arch/arm64/boot/dts/qcom/sm8450-samsung-r0q.dts
> > index a3b81403d180..8ed8a67aae0d 100644
> > --- a/arch/arm64/boot/dts/qcom/sm8450-samsung-r0q.dts
> > +++ b/arch/arm64/boot/dts/qcom/sm8450-samsung-r0q.dts
> > @@ -10,6 +10,9 @@
> >  #include "pm8350c.dtsi"
> >  #include "pmk8350.dtsi"
> >
> > +/delete-node/ &adsp_mem;
> > +/delete-node/ &video_mem;
> > +
> >  / {
> >       model = "Samsung Galaxy S22 5G";
> >       compatible = "samsung,r0q", "qcom,sm8450";
> > @@ -64,6 +67,16 @@ splash-region@b8000000 {
> >                       reg = <0x0 0xb8000000 0x0 0x2b00000>;
> >                       no-map;
> >               };
> > +
> > +             adsp_mem: memory@84500000 {
> > +                     reg = <0x0 0x84500000 0x0 0x3b00000>;
> > +                     no-map;
> > +             };
> > +
> > +             video_mem: memory@83e00000 {
> > +                     reg = <0x0 0x83e00000 0x0 0x700000>;
> > +                     no-map;
> > +             };
>
> Please keep the entries sorted, also with regards to the existing ones
Sure, thanks
>
> Konrad