| Message ID | 20240222130811.3878567-5-devarsht@ti.com |
|---|---|
| State | Under Review |
| Delegated to: | Anatolij Gustschin |
| Headers | show |
| Series | Simplefb and fb reservation related updates | expand |
Hi, On 22/02/24 18:38, Devarsh Thakkar wrote: > Update simple-framebuffer device-tree node by enumerating framebuffer > related information in existing simple-framebuffer node in Linux > device-tree file and enabling it. > > In case there is no simple-framebuffer stub detected in Linux kernel > device-tree and video is still active, then update the device-tree to > reserve the framebuffer region for the active splash screen. > > This helps preserve the splash screen till the display server takes over > after OS is booted. > > Signed-off-by: Devarsh Thakkar <devarsht@ti.com> > --- > board/ti/am62x/evm.c | 19 +++++++++++++++++++ > 1 file changed, 19 insertions(+) > > diff --git a/board/ti/am62x/evm.c b/board/ti/am62x/evm.c > index 88e02155ee..c825af3368 100644 > --- a/board/ti/am62x/evm.c > +++ b/board/ti/am62x/evm.c > @@ -15,6 +15,7 @@ > #include <cpu_func.h> > #include <k3-ddrss.h> > #include <fdt_support.h> > +#include <fdt_simplefb.h> > #include <asm/io.h> > #include <asm/arch/hardware.h> > #include <dm/uclass.h> > @@ -118,3 +119,21 @@ void spl_perform_fixups(struct spl_image_info *spl_image) > #endif > } > #endif > + > +#if defined(CONFIG_OF_BOARD_SETUP) > +int ft_board_setup(void *blob, struct bd_info *bd) > +{ > + int ret = -1; > + > + if (IS_ENABLED(CONFIG_FDT_SIMPLEFB)) > + ret = fdt_simplefb_enable_and_mem_rsv(blob); > + This needs to be protected with IS_ENABLED(CONFIG_VIDEO) too to avoid below errors when CONFIG_VIDEO is disabled : /home/devarsht/ti/ti-u-boot/board/ti/am62x/evm.c:102: undefined reference to `video_is_active' aarch64-none-linux-gnu-ld.bfd: /home/devarsht/ti/ti-u-boot/board/ti/am62x/evm.c:104: undefined reference to `fdt_add_fb_mem_rsv' \ I will send out a V2 of this patch. Regards Devarsh > + /* If simplefb is not enabled and video is active, then at least reserve > + * the framebuffer region to preserve the splash screen while OS is booting > + */ > + if (ret && video_is_active()) > + return fdt_add_fb_mem_rsv(blob); > + > + return 0; > +} > +#endif
diff --git a/board/ti/am62x/evm.c b/board/ti/am62x/evm.c index 88e02155ee..c825af3368 100644 --- a/board/ti/am62x/evm.c +++ b/board/ti/am62x/evm.c @@ -15,6 +15,7 @@ #include <cpu_func.h> #include <k3-ddrss.h> #include <fdt_support.h> +#include <fdt_simplefb.h> #include <asm/io.h> #include <asm/arch/hardware.h> #include <dm/uclass.h> @@ -118,3 +119,21 @@ void spl_perform_fixups(struct spl_image_info *spl_image) #endif } #endif + +#if defined(CONFIG_OF_BOARD_SETUP) +int ft_board_setup(void *blob, struct bd_info *bd) +{ + int ret = -1; + + if (IS_ENABLED(CONFIG_FDT_SIMPLEFB)) + ret = fdt_simplefb_enable_and_mem_rsv(blob); + + /* If simplefb is not enabled and video is active, then at least reserve + * the framebuffer region to preserve the splash screen while OS is booting + */ + if (ret && video_is_active()) + return fdt_add_fb_mem_rsv(blob); + + return 0; +} +#endif
Update simple-framebuffer device-tree node by enumerating framebuffer related information in existing simple-framebuffer node in Linux device-tree file and enabling it. In case there is no simple-framebuffer stub detected in Linux kernel device-tree and video is still active, then update the device-tree to reserve the framebuffer region for the active splash screen. This helps preserve the splash screen till the display server takes over after OS is booted. Signed-off-by: Devarsh Thakkar <devarsht@ti.com> --- board/ti/am62x/evm.c | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+)