2021-06-04 16:46:34

by Aswath Govindraju

Subject: [PATCH] arm64: dts: ti: k3-am64-main: Update the location of ATF in SRAM and increase its max size

Due to a limitation for USB DFU boot mode, SPL load address has to be less
than or equal to 0x70001000. So, load address of SPL and ATF have been
moved to 0x70000000 and 0x701a0000 respectively.

Also, the maximum size of ATF has been increased to 0x1c000 [1].

Therefore, update ATF's location and maximum size accordingly in the device
tree file.

[1] - https://git.trustedfirmware.org/TF-A/trusted-firmware-a.git/commit/?id=2fb5312f61a7de8b7a70e1639199c4f14a10b6f9

Signed-off-by: Aswath Govindraju <[email protected]>
---

Link to corresponding U-Boot series that makes the ATF load address update,
- https://patchwork.ozlabs.org/project/uboot/list/?series=247265

arch/arm64/boot/dts/ti/k3-am64-main.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
index ca59d1f711f8..7ae28992097f 100644
--- a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
@@ -14,7 +14,7 @@
ranges = <0x0 0x00 0x70000000 0x200000>;

atf-sram@0 {
- reg = <0x0 0x1a000>;
+ reg = <0x1a0000 0x1c000>;
};
};

--
2.17.1

2021-06-07 13:31:10

by Nishanth Menon

Subject: Re: [PATCH] arm64: dts: ti: k3-am64-main: Update the location of ATF in SRAM and increase its max size

On 22:13-20210604, Aswath Govindraju wrote:
> Due to a limitation for USB DFU boot mode, SPL load address has to be less
> than or equal to 0x70001000. So, load address of SPL and ATF have been
> moved to 0x70000000 and 0x701a0000 respectively.
>
> Also, the maximum size of ATF has been increased to 0x1c000 [1].
>
> Therefore, update ATF's location and maximum size accordingly in the device
> tree file.
>
> [1] - https://git.trustedfirmware.org/TF-A/trusted-firmware-a.git/commit/?id=2fb5312f61a7de8b7a70e1639199c4f14a10b6f9
>
> Signed-off-by: Aswath Govindraju <[email protected]>
> ---
>
> Link to corresponding U-Boot series that makes the ATF load address update,
> - https://patchwork.ozlabs.org/project/uboot/list/?series=247265
>
> arch/arm64/boot/dts/ti/k3-am64-main.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
> index ca59d1f711f8..7ae28992097f 100644
> --- a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
> @@ -14,7 +14,7 @@
> ranges = <0x0 0x00 0x70000000 0x200000>;
>
> atf-sram@0 {
@0 should be @1a0000 ?
> - reg = <0x0 0x1a000>;
> + reg = <0x1a0000 0x1c000>;
> };
> };
>
> --
> 2.17.1
>

--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D

2021-06-07 13:34:36

by Aswath Govindraju

Subject: Re: [PATCH] arm64: dts: ti: k3-am64-main: Update the location of ATF in SRAM and increase its max size

Hi Nishanth,

On 07/06/21 6:58 pm, Nishanth Menon wrote:
> On 22:13-20210604, Aswath Govindraju wrote:
>> Due to a limitation for USB DFU boot mode, SPL load address has to be less
>> than or equal to 0x70001000. So, load address of SPL and ATF have been
>> moved to 0x70000000 and 0x701a0000 respectively.
>>
>> Also, the maximum size of ATF has been increased to 0x1c000 [1].
>>
>> Therefore, update ATF's location and maximum size accordingly in the device
>> tree file.
>>
>> [1] - https://git.trustedfirmware.org/TF-A/trusted-firmware-a.git/commit/?id=2fb5312f61a7de8b7a70e1639199c4f14a10b6f9
>>
>> Signed-off-by: Aswath Govindraju <[email protected]>
>> ---
>>
>> Link to corresponding U-Boot series that makes the ATF load address update,
>> - https://patchwork.ozlabs.org/project/uboot/list/?series=247265
>>
>> arch/arm64/boot/dts/ti/k3-am64-main.dtsi | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
>> index ca59d1f711f8..7ae28992097f 100644
>> --- a/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
>> +++ b/arch/arm64/boot/dts/ti/k3-am64-main.dtsi
>> @@ -14,7 +14,7 @@
>> ranges = <0x0 0x00 0x70000000 0x200000>;
>>
>> atf-sram@0 {
> @0 should be @1a0000 ?

Sorry, forgot to make this change. Will correct this in the respin.

Thanks,
Aswath

>> - reg = <0x0 0x1a000>;
>> + reg = <0x1a0000 0x1c000>;
>> };
>> };
>>
>> --
>> 2.17.1
>>
>