From 895d3c67cd58fee6815d0e2799ea7d22474accb9 Mon Sep 17 00:00:00 2001 From: Simon Glass Date: Sat, 13 Jun 2026 07:05:12 -0600 Subject: x86: edison: Avoid passing FORCE to the align command Commit bd3f9ee679b4 ("kbuild: Bump the build system to 6.1") adds FORCE as a prerequisite to the u-boot-align.bin rule, but cmd_mkalign_eds expands $^, so FORCE leaks in as a stray operand. The dd then fails (silently, since its stderr is discarded) and the following mv never runs, so u-boot.bin never gains its 4096-byte zero prefix. The mask ROM enters U-Boot 0x1000 (4KB) into the image and the board never starts, sitting in BootROM download mode (DnX). Use $< instead of $^, which excludes FORCE and restores the prefix. Fixes: bd3f9ee679b4 ("kbuild: Bump the build system to 6.1") Signed-off-by: Simon Glass Reviewed-by: Andy Shevchenko --- board/intel/edison/config.mk | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/board/intel/edison/config.mk b/board/intel/edison/config.mk index 3afebf0f0b1..fff187caa15 100644 --- a/board/intel/edison/config.mk +++ b/board/intel/edison/config.mk @@ -7,8 +7,8 @@ # Add 4096 bytes of zeroes to u-boot.bin quiet_cmd_mkalign_eds = EDSALGN $@ cmd_mkalign_eds = \ - dd if=$^ of=$@ bs=4k seek=1 2>/dev/null && \ - mv $@ $^ + dd if=$< of=$@ bs=4k seek=1 2>/dev/null && \ + mv $@ $< INPUTS-y += u-boot-align.bin u-boot-align.bin: u-boot.bin FORCE -- cgit v1.3.1 From 3ac35afdd2cc73db0ac3911937ce75b58a6f90b8 Mon Sep 17 00:00:00 2001 From: Simon Glass Date: Sat, 13 Jun 2026 07:05:13 -0600 Subject: arm: mxs: Avoid passing FORCE to commands Commit bd3f9ee679b4 ("kbuild: Bump the build system to 6.1") adds FORCE as a prerequisite to the if_changed rules here, but the commands use $^ so FORCE leaks in as a stray operand. The dd in cmd_mkalign_mxs fails (silently, since its stderr is discarded) and the binary is never aligned, stat in cmd_mkivt_mxs emits an error and cst receives a bogus argument. Use $< and $(real-prereqs) instead, which exclude FORCE. Fixes: bd3f9ee679b4 ("kbuild: Bump the build system to 6.1") Signed-off-by: Simon Glass --- arch/arm/cpu/arm926ejs/mxs/Makefile | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/arch/arm/cpu/arm926ejs/mxs/Makefile b/arch/arm/cpu/arm926ejs/mxs/Makefile index 0df501e91db..6bd95a5185b 100644 --- a/arch/arm/cpu/arm926ejs/mxs/Makefile +++ b/arch/arm/cpu/arm926ejs/mxs/Makefile @@ -26,7 +26,7 @@ MKIMAGE_TARGET-$(CONFIG_MX28) = mxsimage$(CONFIG_SPL_FRAMEWORK:%=-spl).mx28.cfg # quiet_cmd_mkivt_mxs = MXSIVT $@ cmd_mkivt_mxs = \ - sz=`expr \`stat -c "%s" $^\` + 64 + 3904 + 128` ; \ + sz=`expr \`stat -c "%s" $<\` + 64 + 3904 + 128` ; \ echo -n "0x402000d1 $2 0 0 0 $3 $4 0 $$sz 0 0 0 0 0 0 0" | \ tr -s " " | xargs -d " " -i printf "%08x\n" "{}" | rev | \ sed "s/\(.\)\(.\)/\\\\\\\\x\2\1\n/g" | xargs -i printf "{}" >$@ @@ -34,15 +34,15 @@ cmd_mkivt_mxs = \ # Align binary to 64B quiet_cmd_mkalign_mxs = MXSALGN $@ cmd_mkalign_mxs = \ - dd if=$^ of=$@ ibs=64 conv=sync 2>/dev/null && \ - mv $@ $^ + dd if=$< of=$@ ibs=64 conv=sync 2>/dev/null && \ + mv $@ $< # Assemble the CSF file quiet_cmd_mkcsfreq_mxs = MXSCSFR $@ cmd_mkcsfreq_mxs = \ - ivt=$(word 1,$^) ; \ - bin=$(word 2,$^) ; \ - csf=$(word 3,$^) ; \ + ivt=$(word 1,$(real-prereqs)) ; \ + bin=$(word 2,$(real-prereqs)) ; \ + csf=$(word 3,$(real-prereqs)) ; \ sed "s@VENDOR@$(VENDOR)@g;s@BOARD@$(BOARD)@g" "$$csf" | \ sed '/^\#\#Blocks/ d' > $@ ; \ echo " Blocks = $2 0x0 `stat -c '%s' $$bin` \"$$bin\" , \\" >> $@ ; \ @@ -50,7 +50,7 @@ cmd_mkcsfreq_mxs = \ # Sign files quiet_cmd_mkcst_mxs = MXSCST $@ -cmd_mkcst_mxs = cst -o $@ < $^ \ +cmd_mkcst_mxs = cst -o $@ < $< \ $(if $(KBUILD_VERBOSE:1=), >/dev/null) spl/u-boot-spl.ivt: spl/u-boot-spl.bin FORCE -- cgit v1.3.1