Matt DeVillier has submitted this change. ( https://review.coreboot.org/c/coreboot/+/85957?usp=email )
Change subject: payloads/external/Makefile.mk: Update linuxboot warning ......................................................................
payloads/external/Makefile.mk: Update linuxboot warning
Only print the warning if Linuxboot payload is actually selected, because we don't care otherwise.
Signed-off-by: Maximilian Brune maximilian.brune@9elements.com Change-Id: I5008d685c52c1d4e0d7eba44c964c51a2a6f99c3 Reviewed-on: https://review.coreboot.org/c/coreboot/+/85957 Reviewed-by: Ana Carolina Cabral ana.cpmelo95@gmail.com Reviewed-by: Matt DeVillier matt.devillier@gmail.com Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Alicja Michalska ahplka19@gmail.com --- M payloads/external/Makefile.mk 1 file changed, 3 insertions(+), 0 deletions(-)
Approvals: Matt DeVillier: Looks good to me, approved build bot (Jenkins): Verified Alicja Michalska: Looks good to me, but someone else must approve Ana Carolina Cabral: Looks good to me, but someone else must approve
diff --git a/payloads/external/Makefile.mk b/payloads/external/Makefile.mk index 299f1ee..c279121 100644 --- a/payloads/external/Makefile.mk +++ b/payloads/external/Makefile.mk @@ -396,7 +396,10 @@ CONFIG_LINUXBOOT_CROSS_COMPILE_PATH=$(CROSS_COMPILE_$(LINUXBOOT_CROSS_COMPILE_ARCH-y)) endif # CONFIG_LINUXBOOT_CROSS_COMPILE_PATH else # CONFIG_LINUXBOOT_CROSS_COMPILE +ifeq ($(CONFIG_PAYLOAD_LINUXBOOT),y) + # only print warning when Linuxboot payload is actually selected $(warning "Using host toolchain to build Linuxboot") +endif # CONFIG_PAYLOAD_LINUXBOOT endif # CONFIG_LINUXBOOT_CROSS_COMPILE .PHONY: linuxboot payloads/external/LinuxBoot/build/Image payloads/external/LinuxBoot/build/initramfs linuxboot: