mbox series

[0/4] Simplefb and fb reservation related updates

Message ID 20240222130811.3878567-1-devarsht@ti.com
Headers show
Series Simplefb and fb reservation related updates | expand

Message

Devarsh Thakkar Feb. 22, 2024, 1:08 p.m. UTC
This adds support for simple-framebuffer reservation using video handoff
when splash is enabled at SPL stage.

Also adds helper function to only enable framebuffer reservation without
enabling simple-framebuffer in case user want to continue display
bootloader splash without displaying anything else in between until
kernel boots up.

Lastly, it enables above support on AM62x.

Devarsh Thakkar (4):
  boot: fdt_simplefb: Enumerate framebuffer info from video handoff
  video: Assume video to be active if SPL is passing video hand-off
  boot: Move framebuffer reservation to separate helper
  board: ti: am62x: evm: Update simple-framebuffer node in device-tree

 board/ti/am62x/evm.c         | 19 +++++++++++++++
 boot/fdt_simplefb.c          | 46 ++++++++++++++++++++----------------
 boot/fdt_support.c           | 21 ++++++++++++++++
 drivers/video/video-uclass.c |  4 ++++
 include/fdt_support.h        |  2 ++
 5 files changed, 72 insertions(+), 20 deletions(-)

Comments

Devarsh Thakkar March 8, 2024, 11:08 a.m. UTC | #1
Hi Tom, Anatolij, Simon,

On 22/02/24 18:38, Devarsh Thakkar wrote:
> This adds support for simple-framebuffer reservation using video handoff
> when splash is enabled at SPL stage.
> 
> Also adds helper function to only enable framebuffer reservation without
> enabling simple-framebuffer in case user want to continue display
> bootloader splash without displaying anything else in between until
> kernel boots up.
> 
> Lastly, it enables above support on AM62x.
> 

I saw a RB on first 3 patches so just wanted to check if it looks good to you
too then is it possible to pull in the first 3 patches if no further comments ?

Regards
Devarsh

> Devarsh Thakkar (4):
>   boot: fdt_simplefb: Enumerate framebuffer info from video handoff
>   video: Assume video to be active if SPL is passing video hand-off
>   boot: Move framebuffer reservation to separate helper
>   board: ti: am62x: evm: Update simple-framebuffer node in device-tree
> 
>  board/ti/am62x/evm.c         | 19 +++++++++++++++
>  boot/fdt_simplefb.c          | 46 ++++++++++++++++++++----------------
>  boot/fdt_support.c           | 21 ++++++++++++++++
>  drivers/video/video-uclass.c |  4 ++++
>  include/fdt_support.h        |  2 ++
>  5 files changed, 72 insertions(+), 20 deletions(-)
>
Devarsh Thakkar April 5, 2024, 2:49 p.m. UTC | #2
Gentle Reminder.

On 08/03/24 16:38, Devarsh Thakkar wrote:
> Hi Tom, Anatolij, Simon,
> 
> On 22/02/24 18:38, Devarsh Thakkar wrote:
>> This adds support for simple-framebuffer reservation using video handoff
>> when splash is enabled at SPL stage.
>>
>> Also adds helper function to only enable framebuffer reservation without
>> enabling simple-framebuffer in case user want to continue display
>> bootloader splash without displaying anything else in between until
>> kernel boots up.
>>
>> Lastly, it enables above support on AM62x.
>>
> 
> I saw a RB on first 3 patches so just wanted to check if it looks good to you
> too then is it possible to pull in the first 3 patches if no further comments ?
> 
> Regards
> Devarsh
> 
>> Devarsh Thakkar (4):
>>   boot: fdt_simplefb: Enumerate framebuffer info from video handoff
>>   video: Assume video to be active if SPL is passing video hand-off
>>   boot: Move framebuffer reservation to separate helper
>>   board: ti: am62x: evm: Update simple-framebuffer node in device-tree
>>
>>  board/ti/am62x/evm.c         | 19 +++++++++++++++
>>  boot/fdt_simplefb.c          | 46 ++++++++++++++++++++----------------
>>  boot/fdt_support.c           | 21 ++++++++++++++++
>>  drivers/video/video-uclass.c |  4 ++++
>>  include/fdt_support.h        |  2 ++
>>  5 files changed, 72 insertions(+), 20 deletions(-)
>>
Anatolij Gustschin April 20, 2024, 11:09 p.m. UTC | #3
On Fri, 8 Mar 2024 16:38:28 +0530
Devarsh Thakkar devarsht@ti.com wrote:

> Hi Tom, Anatolij, Simon,
> 
> On 22/02/24 18:38, Devarsh Thakkar wrote:
> > This adds support for simple-framebuffer reservation using video handoff
> > when splash is enabled at SPL stage.
> > 
> > Also adds helper function to only enable framebuffer reservation without
> > enabling simple-framebuffer in case user want to continue display
> > bootloader splash without displaying anything else in between until
> > kernel boots up.
> > 
> > Lastly, it enables above support on AM62x.
> >   
> 
> I saw a RB on first 3 patches so just wanted to check if it looks good to you
> too then is it possible to pull in the first 3 patches if no further comments ?

patches 1 to 3 applied to u-boot-video. Fixed build errors with patch 3.

--
Anatolij