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

Heinrich Schuchardt
2023-07-28T07:31:24Z

On 7/27/23 11:40, 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.

We should add SPDX license information to all the SVG files that you
created.

e.g.

like the license in TI's *.rst files.

Could you, please, send a follow up patch for the series.

Best regards

Heinrich

>>
>> 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>
>