mbox series

[V5,0/6] Low Power Mode: Package TIFS Stub in BeaglePlay

Message ID 20240705164833.2386549-1-d-gole@ti.com
Headers show
Series Low Power Mode: Package TIFS Stub in BeaglePlay | expand

Message

Dhruva Gole July 5, 2024, 4:48 p.m. UTC 
This series includes the binman related changes required to package TIFS
Stub to support Low Power Modes on BeaglePlay.
It also documents the boot flow and tispl packaging details regarding
the same.

Changelog:
* Add more documentation around last time's discussion[1] on packaging TIFS
  Stub in DM.

Link to previous series:
https://lore.kernel.org/u-boot/20240704084207.2211801-1-d-gole@ti.com/

[1] https://lore.kernel.org/u-boot/20240705151753.7v3geqkupqu673x3@ambiance/

Thanks for the explanation given in [1] Sebin,
Cc: Sebin Francis <sebin.francis@ti.com>

Dhruva Gole (6):
  arm: dts: k3-am625-beagleplay: Package TIFS Stub
  doc: beagle: am62x_beagleplay: Update the boot flow to show TIFS Stub
  doc: beagle: am62x_beagleplay: Add TIFS Stub in image format
  doc: ti: k3: Add TIFS Stub documentation
  doc: ti: am62*: Mention TIFS Stub in img fmts and boot flow
  MAINTAINERS: Include the TI docs under ARM TI

 MAINTAINERS                                  |    1 +
 arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi |   33 +-
 doc/board/beagle/am62x_beagleplay.rst        |    4 +-
 doc/board/ti/am62ax_sk.rst                   |    4 +-
 doc/board/ti/am62px_sk.rst                   |    4 +-
 doc/board/ti/am62x_sk.rst                    |    4 +-
 doc/board/ti/img/boot_diagram_am62.svg       | 1983 ++++++++++++++++++
 doc/board/ti/img/tifsstub_dm_tispl.bin.svg   |  353 ++++
 doc/board/ti/k3.rst                          |   11 +
 9 files changed, 2388 insertions(+), 9 deletions(-)
 create mode 100644 doc/board/ti/img/boot_diagram_am62.svg
 create mode 100644 doc/board/ti/img/tifsstub_dm_tispl.bin.svg


base-commit: a9b3723e74aacb5197dc77c9979036dc6b3d9c03

Comments

Nishanth Menon July 17, 2024, 6:49 p.m. UTC | #1 
On 22:18-20240705, Dhruva Gole wrote:
> This series includes the binman related changes required to package TIFS
> Stub to support Low Power Modes on BeaglePlay.
> It also documents the boot flow and tispl packaging details regarding
> the same.
> 
> Changelog:
> * Add more documentation around last time's discussion[1] on packaging TIFS
>   Stub in DM.
> 
> Link to previous series:
> https://lore.kernel.org/u-boot/20240704084207.2211801-1-d-gole@ti.com/
> 
> [1] https://lore.kernel.org/u-boot/20240705151753.7v3geqkupqu673x3@ambiance/
> 
> Thanks for the explanation given in [1] Sebin,
> Cc: Sebin Francis <sebin.francis@ti.com>
> 
> Dhruva Gole (6):
>   arm: dts: k3-am625-beagleplay: Package TIFS Stub
>   doc: beagle: am62x_beagleplay: Update the boot flow to show TIFS Stub
>   doc: beagle: am62x_beagleplay: Add TIFS Stub in image format
>   doc: ti: k3: Add TIFS Stub documentation
>   doc: ti: am62*: Mention TIFS Stub in img fmts and boot flow
> 
	Please change the ordering of the patches:

MAINTAINERS: Include the TI docs under ARM TI
doc: ti: k3: Add TIFS Stub documentation
doc: ti: am62*: Mention TIFS Stub in img fmts and boot flow

then do the stuff for beagleplay - first fix, then add new stuff, NOT
the other way around - yes, i claim that a feature added without
documentation is a bug.
Dhruva Gole July 18, 2024, 10:36 a.m. UTC | #2 
On Jul 17, 2024 at 13:49:07 -0500, Nishanth Menon wrote:
> On 22:18-20240705, Dhruva Gole wrote:
> > This series includes the binman related changes required to package TIFS
> > Stub to support Low Power Modes on BeaglePlay.
> > It also documents the boot flow and tispl packaging details regarding
> > the same.
> > 
> > Changelog:
> > * Add more documentation around last time's discussion[1] on packaging TIFS
> >   Stub in DM.
> > 
> > Link to previous series:
> > https://lore.kernel.org/u-boot/20240704084207.2211801-1-d-gole@ti.com/
> > 
> > [1] https://lore.kernel.org/u-boot/20240705151753.7v3geqkupqu673x3@ambiance/
> > 
> > Thanks for the explanation given in [1] Sebin,
> > Cc: Sebin Francis <sebin.francis@ti.com>
> > 
> > Dhruva Gole (6):
> >   arm: dts: k3-am625-beagleplay: Package TIFS Stub
> >   doc: beagle: am62x_beagleplay: Update the boot flow to show TIFS Stub
> >   doc: beagle: am62x_beagleplay: Add TIFS Stub in image format
> >   doc: ti: k3: Add TIFS Stub documentation
> >   doc: ti: am62*: Mention TIFS Stub in img fmts and boot flow
> > 
> 	Please change the ordering of the patches:
> 
> MAINTAINERS: Include the TI docs under ARM TI
> doc: ti: k3: Add TIFS Stub documentation
> doc: ti: am62*: Mention TIFS Stub in img fmts and boot flow
> 
> then do the stuff for beagleplay - first fix, then add new stuff, NOT
> the other way around - yes, i claim that a feature added without
> documentation is a bug.

Understood, Will reorder and address other concerns in next revision.