[PATCH 2/2] sunxi: dts: H616: Drop reserved-memory node

[Jagan Teki]

On Tue, Mar 30, 2021 at 7:21 PM Samuel Holland <samuel at sholland.org> wrote:
>
> On 3/30/21 8:28 AM, Jagan Teki wrote:
> > On Tue, Mar 30, 2021 at 6:31 PM Andre Przywara <andre.przywara at arm.com> wrote:
> >>
> >> Trusted Firmware now adds the /reserved-memory subnode to the DT at
> >> runtime[1], putting in the right values.
> >>
> >> Drop our hard-coded version, as this might clash with the actual values
> >> (which have also changed), and rely on TF-A to add the node.
> >>
> >> [1] https://review.trustedfirmware.org/c/TF-A/trusted-firmware-a/+/7770
> >>
> >> Signed-off-by: Andre Przywara <andre.przywara at arm.com>
>
> Reviewed-by: Samuel Holland <samuel at sholland.org>
>
> >> ---
> >>  arch/arm/dts/sun50i-h616.dtsi | 12 ------------
> >>  1 file changed, 12 deletions(-)
> >>
> >> diff --git a/arch/arm/dts/sun50i-h616.dtsi b/arch/arm/dts/sun50i-h616.dtsi
> >> index 953e8fac20f..dd4d2f31111 100644
> >> --- a/arch/arm/dts/sun50i-h616.dtsi
> >> +++ b/arch/arm/dts/sun50i-h616.dtsi
> >> @@ -51,18 +51,6 @@
> >>                 };
> >>         };
> >>
> >> -       reserved-memory {
> >> -               #address-cells = <2>;
> >> -               #size-cells = <2>;
> >> -               ranges;
> >> -
> >> -               /* 512KiB reserved for ARM Trusted Firmware (BL31) */
> >> -               secmon_reserved: secmon at 40000000 {
> >> -                       reg = <0x0 0x40000000 0x0 0x80000>;
> >> -                       no-map;
> >> -               };
> >> -       };
> >> -
> >
> > As said always. it's better to not touch Linux dts files. If the same
> > fix same available in Linux add SHA1 on the commit message otherwise
> > keep /delete-node on -u-boot.dtsi. This how we are maintaining sofar
> > at least on sunxi.
>
> This file has not yet been added to the Linux tree, so that rule does not apply
> in this case.

Reviewed-by: Jagan Teki <jagan at amarulasolutions.com>