U-Boot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Anshul Dalal <anshuld@ti.com>
To: Andrew Davis <afd@ti.com>, Anshul Dalal <anshuld@ti.com>
Cc: <b-liu@ti.com>, <bb@ti.com>, <m-chawdhry@ti.com>,
	<n-francis@ti.com>, <nm@ti.com>, <trini@konsulko.com>,
	<u-boot@lists.denx.de>, <vigneshr@ti.com>
Subject: Re: [PATCH v8 8/8] doc: ti: document R5 falcon mode for AM62 platforms
Date: Thu, 30 Oct 2025 10:38:12 +0530	[thread overview]
Message-ID: <DDVEAKLT0K9F.26406SBEB79LS@ti.com> (raw)
In-Reply-To: <f31edd75-97a4-46b7-acf9-6513a9125a4e@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

      reply	other threads:[~2025-10-30  5:08 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-10-27 12:59 [PATCH v8 0/8] Allow falcon boot from R5 SPL on TI's AM62 devices Anshul Dalal
2025-10-27 12:59 ` [PATCH v8 1/8] arch: arm: k3-binman: add fit for falcon boot Anshul Dalal
2025-10-27 12:59 ` [PATCH v8 2/8] configs: add falcon mode fragment for k3 devices Anshul Dalal
2025-10-27 12:59 ` [PATCH v8 3/8] mach-k3: common: enable falcon mode from R5 SPL Anshul Dalal
2025-10-27 12:59 ` [PATCH v8 4/8] mach-k3: common: support only MMC in R5 falcon mode Anshul Dalal
2025-10-27 13:00 ` [PATCH v8 5/8] mach-k3: r5: common: add fdt fixups for " Anshul Dalal
2025-10-27 13:00 ` [PATCH v8 6/8] mach-k3: r5: common: add bootargs to kernel's dtb Anshul Dalal
2025-10-27 13:00 ` [PATCH v8 7/8] Makefile: add tifalcon.bin to CLEAN_FILES Anshul Dalal
2025-10-27 13:00 ` [PATCH v8 8/8] doc: ti: document R5 falcon mode for AM62 platforms Anshul Dalal
2025-10-29 18:16   ` Andrew Davis
2025-10-30  5:08     ` Anshul Dalal [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=DDVEAKLT0K9F.26406SBEB79LS@ti.com \
    --to=anshuld@ti.com \
    --cc=afd@ti.com \
    --cc=b-liu@ti.com \
    --cc=bb@ti.com \
    --cc=m-chawdhry@ti.com \
    --cc=n-francis@ti.com \
    --cc=nm@ti.com \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    --cc=vigneshr@ti.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox