Re: [PATCH 6/8] drivers: video: Kconfig: Add config remove video

[Devarsh Thakkar <devarsht@ti.com>]

Hi Nikhil,

Thanks for the patch.

Subject: Add Kconfig for calling video remove method

On 11/05/23 15:29, Nikhil M Jain wrote:
> Add VIDEO_REMOVE configs to allow user to control removing of video
> driver, in between stages.

This is required since user may want to either call the remove method
of video driver and reset the display or not call the remove method
to continue displaying until next stage.

> 
> Signed-off-by: Nikhil M Jain <n-jain1@ti.com>
> ---
>  drivers/video/Kconfig | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig
> index fcc0e85d2e..c5863f4dd5 100644
> --- a/drivers/video/Kconfig
> +++ b/drivers/video/Kconfig
> @@ -840,6 +840,12 @@ config IHS_VIDEO_OUT
>  	  out On-screen Display (OSD) used on gdsys FPGAs to control dynamic
>  	  textual overlays of the display outputs.
>  
> +config VIDEO_REMOVE
> +	bool "Remove video driver"
> +	help
> +		Use this option to specify if you want to remove video driver before
> +		loading OS.

to call remove method of video driver in u-boot proper stage

> +
>  config SPLASH_SCREEN
>  	bool "Show a splash-screen image"
>  	help
> @@ -1063,6 +1069,12 @@ config SPL_SYS_WHITE_ON_BLACK
>  	 This can be better in low-light situations or to reduce eye strain in
>  	 some cases.
>  
> +config SPL_VIDEO_REMOVE
> +	bool "Remove video driver after SPL stage"
> +	help
> +	 if this  option is enabled video driver will be removed at the end of
> +	 SPL stage, beforeloading the next stage.
> +

to call remove method of video driver in u-boot SPL stage
With suggested changes,
Reviewed-by: Devarsh Thakkar <devarsht@ti.com>

Regards
Devarsh

>  if SPL_SPLASH_SCREEN
>  
>  config SPL_SPLASH_SCREEN_ALIGN