[EXTERNAL] Re: [PATCH 0/8] Updats SPL splashscreen framework for AM62x

[Tom Rini]

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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20230602/cc9eabec/attachment.sig>