Re: [PATCH v8 8/8] doc: ti: document R5 falcon mode for AM62 platforms

[Anshul Dalal <anshuld@ti.com>]

On Wed Oct 29, 2025 at 11:46 PM IST, Andrew Davis wrote:
> On 10/27/25 8:00 AM, Anshul Dalal wrote:
>> This patch adds user documentation for falcon boot for AM62 platforms.
>> The main section is added to am62x_sk.rst and other documents just
>> include the section. Steps to build falcon support, usage and the
>> modified R5 memory map have been documented.
>> 
>> Two svg images have also been added for reference, one for the modified
>> tifalcon.bin and other for the falcon specific fitImage format.
>> 
>> Signed-off-by: Anshul Dalal <anshuld@ti.com>
>> ---
>>   doc/board/ti/am62ax_sk.rst                    |  35 ++
>>   doc/board/ti/am62px_sk.rst                    |  35 ++
>>   doc/board/ti/am62x_sk.rst                     | 303 ++++++++++++++++++
>>   doc/board/ti/img/fitImage_falcon.svg          | 180 +++++++++++
>>   doc/board/ti/img/tifsstub_dm_tifalcon.bin.svg | 285 ++++++++++++++++
>>   5 files changed, 838 insertions(+)
>>   create mode 100644 doc/board/ti/img/fitImage_falcon.svg
>>   create mode 100644 doc/board/ti/img/tifsstub_dm_tifalcon.bin.svg
>> 
>> diff --git a/doc/board/ti/am62ax_sk.rst b/doc/board/ti/am62ax_sk.rst
>> index 4dd998bcce9..45992ec304b 100644
>> --- a/doc/board/ti/am62ax_sk.rst
>> +++ b/doc/board/ti/am62ax_sk.rst
>> @@ -187,6 +187,41 @@ https://www.ti.com/lit/pdf/spruj16 under the `Boot Mode Pins` section.
>>   
>>   For SW2 and SW1, the switch state in the "ON" position = 1.
>>   
>> +Falcon Mode
>> +-----------
>> +
>> +Falcon Mode on AM62ax platforms bypasses the A53 SPL and U-Boot with the overall
>> +boot flow as below:
>> +
>> +**Normal boot flow:**
>> +
>> +* R5 SPL -> TFA -> OP-TEE -> A53 SPL -> U-Boot -> Kernel
>
> It might be good to point out that the TFA and after is on a different
> core than the R5 SPL, it might be obvious to us based on names, but
> never hurts to make flow graph match, and also in some way show the
> file names that contain the different parts, maybe somehting like:
>
>
>
>          | tiboot3.bin |          tispl.bin       |  u-boot.img  |  Image  |
>          |             |                          |              |         |
> R5 Core      SPL
> -------         \
> A53 Core          -- -> TFA -> OP-TEE -> A53 SPL  --> U-Boot  -->  Kernel
>

That is indeed more clear, I will incorporate it in the next revision.

>
> Also if this section is the same for all these boards, could it also
> be factored out into am62x_sk.rst?
>

Yeah, we can reuse it as well.

Regards,
Anshul