[EXTERNAL] Re: [PATCH 09/10] config: add falcon boot config fragment for am62x

[Anshul Dalal]

On Fri Mar 7, 2025 at 9:28 PM IST, Tom Rini wrote:
> On Fri, Mar 07, 2025 at 01:25:37PM +0530, Anshul Dalal wrote:
> > This fragment enables falcon boot for the am62x, 62a and 62p.
> > 
> > To have enough stack and heap space for loading kernel image as FIT, the
> > memory map was modified as shown below, the PRELOADED_BL33_BASE in ATF
> > has to also be updated to 0x80200000 since the kernel requires to be
> > loaded at 2MiB aligned address.
> > 
> > Memory Map (changed addresses marked with *):
> > 
> > 0x80000000 +-------------------------------+ Start of DDR
> >   512KiB   |   ATF reserved memory space   | CONFIG_K3_ATF_LOAD_ADDR
> > 0x80080000 +-------------------------------+
> >   1.5MiB   |            Unused             |
> > 0x80200000 +-------------------------------+ PRELOADED_BL33_BASE* in ATF
> >            |                               | CONFIG_SYS_LOAD_ADDR* (load
> >    30MiB   |   Kernel + initramfs Image    | address for kernel)
> >            |                               |
> > 0x82000000 +-------------------------------+ K3_HW_CONFIG_BASE in ATF
> >            |                               | CONFIG_SPL_PAYLOAD_ARGS_ADDR*
> >     1MiB   |          Kernel DTB           |
> >            |                               |
> > 0x82100000 +-------------------------------+ CONFIG_SPL_LOAD_FIT_ADDRESS*
> >            |                               |
> >    55MiB   |Initial Download Region for FIT|
> >            |                               |
> > 0x85800000 +-------------------------------+
> >            |                               |
> >            |  R5 U-Boot SPL Stack + Heap   |
> >    56MiB   |       (size defined by        |
> >            | SPL_STACK_R_MALLOC_SIMPLE_LEN)|
> >            |                               |
> > 0x89000000 +-------------------------------+ CONFIG_SPL_STACK_R_ADDR*
> >   344MiB   |       Device Manager (DM)     |
> > 0x9e800000 +-------------------------------+ BL32_BASE in ATF
> >    24MiB   |             OPTEE             |
> > 0xa0000000 +-------------------------------+ End of DDR (512MiB)
>
> This information is good and belongs in a relevant rst file, not just
> commit message.

I wanted to get some feedback first before updating the documentation in
case there were any changes to the boot flow. Once we have finalized
that, I will update the documentation in an upcoming revision of this
patch set.

Anshul