On Fri, Jun 02, 2023 at 11:49:49AM +0530, Nikhil M Jain wrote: > 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.
OK, hopefully it all applies correctly still once that's merged. > 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? Done. -- Tom
signature.asc
Description: PGP signature