[PATCH 0/9] Enable splash screen

[Nikhil M Jain]

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.

> Regards,
> Simon

Regards,
Nikhil