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

Devarsh Thakkar
2023-04-03T10:47:21Z

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