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

[Nikhil M Jain]

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-2327ced39ec7@ti.com/

Thanks,
Nikhil