Re: [PATCH 2/2] arm64: dts: msm8994-angler: removed clash with smem_region

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Bjorn,

any chance you could merge this patchset? Both patches apply on the
top of current qcom/for-next tree, because (as I wrote 2 months ago)
you applied Jamie's patch [1].

Kind regards,
Petr

[1] https://lore.kernel.org/linux-arm-msm/167591660371.1230100.18259207897974955566.b4-ty@xxxxxxxxxx/

On Thu, 9 Feb 2023 at 09:12, Petr Vorel <pvorel@xxxxxxx> wrote:
>
> > On Tue, Jan 31, 2023 at 09:04:14PM +0100, Petr Vorel wrote:
> > > This fixes memory overlap error:
> > > [    0.000000] reserved@6300000 (0x0000000006300000--0x0000000007000000) overlaps with smem_region@6a00000 (0x0000000006a00000--0x0000000006c00000)
>
> > > smem_region is the same as in downstream (qcom,smem) [1], therefore
> > > split reserved memory into two sections on either side of smem_region.
>
> > > Not adding labels as it's not expected to be used.
>
> > > [1] https://android.googlesource.com/kernel/msm/+/refs/heads/android-msm-angler-3.10-marshmallow-mr1/arch/arm/boot/dts/qcom/msm8994.dtsi#948
>
> > > Fixes: 380cd3a34b7f ("arm64: dts: msm8994-angler: fix the memory map")
>
> > > Signed-off-by: Petr Vorel <pvorel@xxxxxxx>
> > > ---
> > >  arch/arm64/boot/dts/qcom/msm8992-lg-bullhead.dtsi          | 5 -----
> > >  arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dts | 4 ++--
> > >  arch/arm64/boot/dts/qcom/msm8994.dtsi                      | 5 +++++
> > >  3 files changed, 7 insertions(+), 7 deletions(-)
>
> > > diff --git a/arch/arm64/boot/dts/qcom/msm8992-lg-bullhead.dtsi b/arch/arm64/boot/dts/qcom/msm8992-lg-bullhead.dtsi
> > > index cdd796040703..a100b05abf56 100644
> > > --- a/arch/arm64/boot/dts/qcom/msm8992-lg-bullhead.dtsi
> > > +++ b/arch/arm64/boot/dts/qcom/msm8992-lg-bullhead.dtsi
> > > @@ -60,11 +60,6 @@ reserved@5000000 {
> > >                     reg = <0x0 0x05000000 0x0 0x1a00000>;
> > >                     no-map;
> > >             };
> > > -
> > > -           reserved@6c00000 {
> > > -                   reg = <0x0 0x06c00000 0x0 0x400000>;
> > > -                   no-map;
> > > -           };
>
> > I've not picked up the change that introduces this, and it seems that
> > there's some request for changes on that thread.
>
> > Please try to sync up with Jamie to get the patch updated, or let me
> > know how you would like me to proceed.
>
> You merged v2 [1] as cd451939fdda ("arm64: dts: msm8994-angler: removed clash
> with smem_region"), therefore my patchset is applicable the current
> qcom/for-next. Can you please merge it? Or am I missing something?
>
> Kind regards,
> Petr
>
> [1] https://lore.kernel.org/linux-arm-msm/167591660371.1230100.18259207897974955566.b4-ty@xxxxxxxxxx/
>
> > Thanks,
> > Bjorn
>
> > >     };
> > >  };
>
> > > diff --git a/arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dts b/arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dts
> > > index 59b9ed78cf0c..29e79ae0849d 100644
> > > --- a/arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dts
> > > +++ b/arch/arm64/boot/dts/qcom/msm8994-huawei-angler-rev-101.dts
> > > @@ -41,8 +41,8 @@ tzapp_mem: tzapp@4800000 {
> > >                     no-map;
> > >             };
>
> > > -           removed_region: reserved@6300000 {
> > > -                   reg = <0 0x06300000 0 0xD00000>;
> > > +           reserved@6300000 {
> > > +                   reg = <0 0x06300000 0 0x700000>;
> > >                     no-map;
> > >             };
> > >     };
> > > diff --git a/arch/arm64/boot/dts/qcom/msm8994.dtsi b/arch/arm64/boot/dts/qcom/msm8994.dtsi
> > > index 9ff9d35496d2..24c3fced8df7 100644
> > > --- a/arch/arm64/boot/dts/qcom/msm8994.dtsi
> > > +++ b/arch/arm64/boot/dts/qcom/msm8994.dtsi
> > > @@ -228,6 +228,11 @@ adsp_mem: memory@c9400000 {
> > >                     reg = <0 0xc9400000 0 0x3f00000>;
> > >                     no-map;
> > >             };
> > > +
> > > +           reserved@6c00000 {
> > > +                   reg = <0 0x06c00000 0 0x400000>;
> > > +                   no-map;
> > > +           };
> > >     };
>
> > >     smd {
> > > --
> > > 2.39.1
>



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [Linux for Sparc]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux