Hi Nikhil, On Wed, 15 Mar 2023 at 00:07, Nikhil M Jain <n-ja...@ti.com> wrote: > > Hi Simon, > > On 15/03/23 03:38, Simon Glass wrote: > > Hi Nikhil, > > > > On Mon, 13 Mar 2023 at 04:15, Nikhil M Jain <n-ja...@ti.com> wrote: > >> > >> To enable splash screen at SPL stage move video driver and splash screen > >> framework at SPL, which will bring up image on display very quickly and > >> thus have early display support in SPL. > >> > >> Nikhil M Jain (9): > >> drivers: video: Kconfig: Necessary configs for video at SPL > >> drivers: video: tidss: Kconfig: Configs to enable TIDSS at SPL > >> cmd: Kconfig: Add necessary configs for splash screen at SPL > >> drivers: video: Makefile: Compile video driver files at SPL > >> drivers: video: tidss: Makefile: Add condition to compile TIDSS at SPL > >> cmd: Makefile: Add rules to build bmp.c and read.c at SPL > >> common: splash: Enable splash_display at SPL stage > >> drivers: video: video-uclass: Disable u-boot logo at SPL > >> board: ti: am62x: evm: OSPI support for splash screen > >> > >> board/ti/am62x/evm.c | 6 ++++++ > >> cmd/Kconfig | 17 +++++++++++++++++ > >> cmd/Makefile | 2 ++ > >> common/splash.c | 2 +- > >> drivers/video/Kconfig | 32 ++++++++++++++++++++++++++++---- > >> drivers/video/Makefile | 6 ++++++ > >> drivers/video/tidss/Kconfig | 6 ++++++ > >> drivers/video/tidss/Makefile | 1 + > >> drivers/video/video-uclass.c | 2 +- > >> include/splash.h | 2 +- > >> 10 files changed, 69 insertions(+), 7 deletions(-) > > > > I'm not necessarily arguing against this, but what is the need for > > this? How many milliseconds earlier does the image appear with this > > patch? What is the bottleneck? We should be able to get to U-Boot > > proper very quickly. > > > There is a significant difference in time, by adding support in SPL > splash screen comes up by approx 650ms and at u-boot proper it comes at > 2.6s, measured from first print in console as seen on AM62x. Also we > plan to skip u-boot proper and load kernel directly.
Yes that really is terrible. It should be under a second for U-Boot proper! Have you tried using bootstage to report the numbers? Have you tried using tracing to figure out what is wrong? Is it just slow storage? Regards, Simon