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