diff options
| author | Simon Glass <[email protected]> | 2026-06-13 07:05:12 -0600 |
|---|---|---|
| committer | Tom Rini <[email protected]> | 2026-06-15 12:49:56 -0600 |
| commit | 895d3c67cd58fee6815d0e2799ea7d22474accb9 (patch) | |
| tree | e9f123293871b36b95de9d0920bb8f163ee849d5 | |
| parent | 5ca1a73c7d3064582498a8aa96c29e714402a6d3 (diff) | |
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 <[email protected]>
Reviewed-by: Andy Shevchenko <[email protected]>
| -rw-r--r-- | board/intel/edison/config.mk | 4 |
1 files 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 |
