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 <n...@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-fran...@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

Reply via email to