Re: [PATCH V5 03/13] drivers: Makefile: Add rule to compile video driver

Devarsh Thakkar
2023-04-03T10:48:54Z

Hi Nikhil,

On 03/04/23 13:44, Nikhil M Jain wrote:
> CONFIG_$(SPL_)VIDEO checks for CONFIG_SPL_VIDEO at SPL and CONFIG_VIDEO
> at u-boot proper and compiles video driver at respective stage.
>

Please update above commit message per the new patch change as
CONFIG_$(SPL_)VIDEO is not used now. After above change, kindly add,

> Signed-off-by: Nikhil M Jain <n-jain1@ti.com>
Reviewed-by: Devarsh Thakkar <devarsht@ti.com>

Regards
Devarsh
> ---
> V5:
> - Add rule to build video at SPL using CONFIG_SPL_VIDEO and retain
> obj-y +=video for u-boot proper.
>
> V4:
> - No change
>
> V3 (patch introduced):
> - Add rule to compile video driver at SPL.
>
> drivers/Makefile | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/Makefile b/drivers/Makefile
> index 58be410135..928a8b0e9d 100644
> --- a/drivers/Makefile
> +++ b/drivers/Makefile
> @@ -61,6 +61,7 @@ obj-$(CONFIG_SPL_USB_HOST) += usb/host/
> obj-$(CONFIG_SPL_SATA) += ata/ scsi/
> obj-$(CONFIG_SPL_LEGACY_BLOCK) += block/
> obj-$(CONFIG_SPL_THERMAL) += thermal/
> +obj-$(CONFIG_SPL_VIDEO) +=video/
>
> endif
> endif