On Fri, Oct 24, 2025 at 12:56:08PM +0530, Anshul Dalal wrote:

> Using CMD_* configs from spl doesn't make logical sense. Therefore this
> patch replaces the checks for CMD_BOOTx with newly added library symbols
> BOOTI, BOOTM and BOOTZ which are enabled by their respective CMD_* or
> SPL_* counterparts.

You didn't update the commit message here to reflect the new symbol
names.

> diff --git a/arch/arm/lib/Makefile b/arch/arm/lib/Makefile
> index 
> ade42d0ca4370b7df3aec59615f12e28a96874cd..9f3c30156d22330515e408ec030a3ced3b81ab38
>  100644
> --- a/arch/arm/lib/Makefile
> +++ b/arch/arm/lib/Makefile
> @@ -7,6 +7,13 @@ lib-$(CONFIG_USE_PRIVATE_LIBGCC) += ashldi3.o ashrdi3.o 
> lshrdi3.o \
>                                   lib1funcs.o uldivmod.o div0.o \
>                                   div64.o muldi3.o
>  
> +obj-$(CONFIG_$(PHASE_)LIB_BOOTI) += image.o
> +obj-$(CONFIG_$(PHASE_)LIB_BOOTZ) += zimage.o
> +
> +ifndef CONFIG_XPL_BUILD
> +obj-$(CONFIG_LIB_BOOTM) += bootm.o
> +endif

Since we have $(PHASE_) now for LIB_BOOTM this too should be like the
other two.

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to