Hi Tom,

On 02/06/23 10:09, Nikhil M Jain wrote:
Hi Tom,

On 01/06/23 22:10, Tom Rini wrote:
On Thu, May 11, 2023 at 03:29:50PM +0530, Nikhil M Jain wrote:

This patch series aims at updating SPL splashscreen framework for AM62x.

Nikhil M Jain (8):
   common: spl: spl: Update stack pointer address
   arch: arm: mach-k3: common: Return a pointer after setting page table
   board: ti: am62x: evm: Update function calls for splash screen
   include: video: Reserve video using blob
   common: board_f: Pass frame buffer info from SPL to u-boot
   drivers: video: Kconfig: Add config remove video
   common: spl: spl: Remove video driver
   configs: am62x_evm_a53: Add bloblist address

  arch/arm/mach-k3/am625_init.c   |  1 +
  arch/arm/mach-k3/common.c       |  2 ++
  board/ti/am62x/evm.c            | 46 +++++++++++++--------------------
  common/board_f.c                | 13 +++++++++-
  common/spl/spl.c                |  3 ++-
  configs/am62x_evm_a53_defconfig |  1 +
  drivers/video/Kconfig           | 12 +++++++++
  drivers/video/video-uclass.c    | 24 +++++++++++++++++
  include/video.h                 |  9 +++++++
  9 files changed, 81 insertions(+), 30 deletions(-)

This series causes problems with sandbox even building, please rebase on
current next and put this through CI once you're sure both the TI cases
and at least sandbox also build, thanks.

I will rebase it and ensure it is getting built successfully.

Thanks,
Nikhil

Sorry I forgot to mention but this patch series actually depends on
https://patchwork.ozlabs.org/project/uboot/list/?series=353559
and without above series it won't build cleanly.

Also to add one of the patches from the series [1] had one comment as below:

>Vignesh, Tom,

"Could you please have a look at this patch and comment ? Does the caller of this function need to set gd->start_addr_sp or it's ok to set in here only?"

Could you please share your opinion for the same?

[1] https://lore.kernel.org/u-boot/ae6aa5f2-8bda-850c-5aae-2327ced39...@ti.com/

Thanks,
Nikhil

Reply via email to