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

Neha Malcom Francis
2023-07-27T10:05:00Z

On 27/07/23 15:32, Neha Malcom Francis wrote:
> 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.
>

This comment was meant for [PATCH 09/16]

--
Thanking You
Neha Malcom Francis