[PATCH] mx9: Correct repeatable build error
Simon Glass
sjg at chromium.org
Tue Jul 2 17:37:39 CEST 2024
For some reason every second time imx93_11x11_evk is built it gives an
error:
make O=/tmp/x BINMAN_ALLOW_MISSING=1
It seems to sometimes skip generation of the .cfgout file and then
eventually Binman complains:
ValueError: Error 1 running 'mkimage -d ./mkimage.spl.mkimage -n
spl/u-boot-spl.cfgout -T imx8image -e 0x2049A000
./mkimage-out.spl.mkimage': Fail open first container file
mx93a1-ahab-container.img
Correct this by using if_changed instead of if_changed_dep
The only reason this hasn't come up in CI is that buildman did not retry
failing builds of current source, but now it does.
Note: The logic in this Makefile should be moved to Binman, e.g. these
warnings duplicate Binman functionality:
WARNING 'bl31.bin' not found, resulting binary may be not-functional
WARNING 'tee.bin' not found, resulting binary may be not-functional
Signed-off-by: Simon Glass <sjg at chromium.org>
---
arch/arm/mach-imx/Makefile | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/mach-imx/Makefile b/arch/arm/mach-imx/Makefile
index ef0caed3f7f..640e332e78b 100644
--- a/arch/arm/mach-imx/Makefile
+++ b/arch/arm/mach-imx/Makefile
@@ -227,7 +227,7 @@ flash.bin: MKIMAGEOUTPUT = flash.log
spl/u-boot-spl.cfgout: $(IMX_CONFIG) FORCE
$(Q)mkdir -p $(dir $@)
- $(call if_changed_dep,cpp_cfg)
+ $(call if_changed_,cpp_cfg)
$(call if_changed,imx9_check)
spl/u-boot-spl-ddr.bin: spl/u-boot-spl.bin spl/u-boot-spl.cfgout FORCE
--
2.34.1
More information about the U-Boot
mailing list