Re: [PATCH 02/16] doc: board: ti: am62x/j7200: Update with common boot flow diagram

Neha Malcom Francis
2023-07-27T10:03:15Z

Hi Nishanth

On 27/07/23 15:10, Neha Malcom Francis wrote:
> These look nice!
>
> On 27/07/23 13:30, Nishanth Menon wrote:
>> Update the bootflow svg diagram and reuse across the platforms as they
>> are common.
>>
>> Signed-off-by: Nishanth Menon <nm@ti.com>
>> ---
>>   doc/board/ti/am62x_sk.rst                    |   68 +-
>>   doc/board/ti/img/boot_diagram_k3_current.svg | 1987 ++++++++++++++++++
>>   doc/board/ti/j7200_evm.rst                   |   85 +-
>>   3 files changed, 1989 insertions(+), 151 deletions(-)
>>   create mode 100644 doc/board/ti/img/boot_diagram_k3_current.svg
>>
> [...]
>> - |    +---------+         |                       |
>> |                       |
>> - |                        |                       |
>> |                       |
>> -
>> +------------------------------------------------------------------------+-----------------------+
>> +.. image:: img/boot_diagram_k3_current.svg
>>   - Here DMSC acts as master and provides all the critical services. R5/A72
>>     requests DMSC to get these services done as shown in the above diagram.
>
>
> Reviewed-by: Neha Malcom Francis <n-francis@ti.com>
>

Although one change (correct me if I am wrong), the TIFS + board-cfg is packed
*after* R5-SPL + DTB FIT; so maybe you could reverse the bottom two blocks there
in tiboot3.bin. Same goes for the other combined tiboot3.bin diagrams.

--
Thanking You
Neha Malcom Francis