[PATCH v4 09/15] cmd: Makefile: Rule to compile bmp_cmd

Devarsh Thakkar devarsht at ti.com
Thu Mar 30 12:06:40 CEST 2023


Hi Nikhil,

One more comment on this patch.

On 29/03/23 17:31, Nikhil M Jain wrote:
> Change rule to compile bmp_cmd when CONFIG_CMD_BMP is defined, in place
> of bmp.c.
> 
> Signed-off-by: Nikhil M Jain <n-jain1 at ti.com>
> ---
> V4:
> - No change
> 
> V3 (patch introduced):
> - Rule to compile bmp_cmd.c
> 
>  cmd/Makefile | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/cmd/Makefile b/cmd/Makefile
> index 36d2daf22a..c773acb1a1 100644
> --- a/cmd/Makefile
> +++ b/cmd/Makefile
> @@ -29,7 +29,7 @@ obj-$(CONFIG_CMD_BIND) += bind.o
>  obj-$(CONFIG_CMD_BINOP) += binop.o
>  obj-$(CONFIG_CMD_BLOBLIST) += bloblist.o
>  obj-$(CONFIG_CMD_BLOCK_CACHE) += blkcache.o
> -obj-$(CONFIG_CMD_BMP) += bmp.o
> +obj-$(CONFIG_CMD_BMP) += bmp_cmd.o
>  obj-$(CONFIG_CMD_BOOTCOUNT) += bootcount.o
>  obj-$(CONFIG_CMD_BOOTEFI) += bootefi.o
>  obj-$(CONFIG_CMD_BOOTMENU) += bootmenu.o
> @@ -235,6 +235,7 @@ obj-$(CONFIG_$(SPL_)CMD_TLV_EEPROM) += tlv_eeprom.o
>  
>  # core command
>  obj-y += nvedit.o
> +obj-y += read.o
commit message doesnt describe above change, is it really required ?

Regards
Devarsh

>  
>  obj-$(CONFIG_CMD_BCM_EXT_UTILS) += broadcom/
>  


More information about the U-Boot mailing list