Re: [PATCH V5 02/13] drivers: video: tidss: Kconfig: Configs to enable TIDSS at SPL

[Devarsh Thakkar <devarsht@ti.com>]

Hi Nikhil,

On 03/04/23 13:44, Nikhil M Jain wrote:
> To enable tidss video driver only at SPL stage, add necessary config,
> CONFIG_SPL_VIDEO_TIDSS.
> 
> Signed-off-by: Nikhil M Jain <n-jain1@ti.com>
> Reviewed-by: Simon Glass <sjg@chromium.org>
> ---
> V5:
> - No change
> 
> V4:
> - Add Reviewed-By tag
> 
> V3:
> - No change
> 
> V2:
> - No change
> 
>  drivers/video/tidss/Kconfig | 6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/drivers/video/tidss/Kconfig b/drivers/video/tidss/Kconfig
> index 2a5e56ea4e..748c332281 100644
> --- a/drivers/video/tidss/Kconfig
> +++ b/drivers/video/tidss/Kconfig
> @@ -16,3 +16,9 @@ menuconfig VIDEO_TIDSS
>  	  DPI . This option enables these supports which can be used on
>  	  devices which have OLDI or HDMI display connected.
>  
> +config SPL_VIDEO_TIDSS
> +	bool "Enable TIDSS video support in SPL Stage"
> +	depends on SPL_VIDEO
> +	help
> +	  This options enables tidss driver in SPL stage. If
> +	  you need to use tidssat SPL stage use this config.

Minor typo <space missing>
"you need to use tidss display driver at SPL stage"
After that is fixed, feel free to add
Reviewed-by: Devarsh Thakkar <devarsht@ti.com>

Regards
Devarsh