On Thu, May 04, 2023 at 06:40:56PM -0600, Simon Glass wrote:
> Hi,
> 
> On Thu, 20 Apr 2023 at 06:11, 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.
> >
> > Changes in V9
> > - Update as per review comments.
> > - Add Reviewed-by tag in appropriate patch.
> >
> > Changes in V8
> > - Update as per review comments.
> >
> > Changes in V7
> > - Replace #ifdefs' and #ifs' with if in common/splash and common/bmp.
> > - Add Reviewed-by and Tested-by tag in apprpriate patch.
> >
> > Changes in V6
> > - Fix commit messages.
> > - Fix compile rules for video related files in patch
> >   drivers: video: Makefile: Rule to compile necessary video driver
> >   files.
> >
> > Changes in V5
> > - Drop patch cmd: Makefile: Rule to compile bmp_cmd.
> > - Squash common: splash: Replace CONFIG_CMD_BMP patch into patches
> >   common: Enable splash functions at SPL
> >   include: Enable video related global data variable and splash at SPL
> > - Fix merge issue for next branch.
> > - Replace cmd/bmp_cmd with cmd/bmp.
> > - Use CONFIG_$(SPL_TPL_) to compile files in
> >   drivers: video: Makefile: Rule to compile necessary video driver files.
> > - Add Reviewed-by tag in appropriate patch.
> >
> > Changes in V4
> > - Add Reviewed-by tag in appropriate patch.
> >
> > Changes in V3
> > - Add separate video configs for SPL splash screen.
> > - Add rule to compile video driver in drivers/Makefile at SPL.
> > - Add rule to compile splash.c and splash_source.c.
> > - Squash drivers: video: video-uclass: Disable u-boot logo at SPL into
> >   drivers: video: Use CONFIG_IS_ENABLED and CONFIG_VAL.
> > - Split cmd/bmp.c to separate bmp functions and commands.
> > - Add CONFIG_BMP and CONFIG_SPL_BMP.
> > - Add rule to compile necessary files required for video driver.
> > - Add rule to compile common/bmp.c.
> >
> > Changes in V2
> > - Removed artifacts from bad patch apply.
> >
> > Nikhil M Jain (14):
> >   drivers: video: Kconfig: Add configs for enabling video at SPL
> >   drivers: video: tidss: Kconfig: Configs to enable TIDSS at SPL
> >   drivers: Makefile: Add rule to compile video driver
> >   drivers: video: Makefile: Rule to compile necessary video driver files
> >   drivers: video: tidss: Makefile: Add condition to compile TIDSS at SPL
> >   common: Makefile: Add rule to compile splash and splash_source at SPL
> >   common: Kconfig: Add BMP configs
> >   cmd: bmp: Split bmp commands and functions
> >   common: Makefile: Rule to compile bmp.c
> >   drivers: video: Enable necessary video functions at SPL
> >   common: Enable splash functions at SPL
> >   include: Enable video related global data variable and splash at SPL
> >   board: ti: am62x: evm: OSPI support for splash screen
> >   common: Replace #ifdef and #if with if's
> >
> >  board/ti/am62x/evm.c              |   8 +-
> >  cmd/bmp.c                         | 162 +---------------------
> >  common/Kconfig                    |  12 ++
> >  common/Makefile                   |   3 +
> >  common/bmp.c                      | 149 ++++++++++++++++++++
> >  common/splash.c                   |  20 +--
> >  drivers/Makefile                  |   1 +
> >  drivers/video/Kconfig             | 223 +++++++++++++++++++++++++++++-
> >  drivers/video/Makefile            |  16 +--
> >  drivers/video/console_core.c      |   6 +-
> >  drivers/video/tidss/Kconfig       |   6 +
> >  drivers/video/tidss/Makefile      |   2 +-
> >  drivers/video/vidconsole-uclass.c |   2 +-
> >  drivers/video/video-uclass.c      |  14 +-
> >  drivers/video/video_bmp.c         |   8 +-
> >  include/asm-generic/global_data.h |   4 +-
> >  include/splash.h                  |  15 +-
> >  include/video.h                   |   8 ++
> >  18 files changed, 447 insertions(+), 212 deletions(-)
> >  create mode 100644 common/bmp.c
> >
> > --
> > 2.34.1
> >
> 
> Can this be applied, please? We have rc1 out now so this should really
> be in there by now.
> 
> I have another series that depends on it, and it is also pending application.

Anatolij ?

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to