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

Samuel Holland samuel at sholland.org
Tue Mar 30 15:51:48 CEST 2021


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.

Cheers,
Samuel

> Jagan.
> 



More information about the U-Boot mailing list