[PATCH 0/9] Enable splash screen

[Simon Glass]

Hi Nikhil,

On Wed, 15 Mar 2023 at 00:07, Nikhil M Jain <n-jain1 at 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-jain1 at 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