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

Nishanth Menon
2023-07-27T13:13:10Z

On 15:34-20230727, Neha Malcom Francis wrote:
> 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]

Got it, i decoded the ascii text wrong. will fixup.

--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D