From 67ffb5134c5238295591bbc3f3260d5651a0a89a Mon Sep 17 00:00:00 2001 From: Leah Rowe Date: Tue, 10 Oct 2023 06:04:47 +0100 Subject: build/fw/coreboot: warning about bin/ versus elf/ also rename elf/coreboot to something scary some users were flashing roms built under elf/, which lack payloads. lbmk builds no-payload roms (and payloads) under elf/ then inserts them, creating full (flashable) images under bin/ Signed-off-by: Leah Rowe --- script/update/project/trees | 9 +++++++++ 1 file changed, 9 insertions(+) (limited to 'script/update') diff --git a/script/update/project/trees b/script/update/project/trees index 7df3c8bf..97b12954 100755 --- a/script/update/project/trees +++ b/script/update/project/trees @@ -72,6 +72,8 @@ build_projects() build_targets() { elfdir="elf/${project}" + [ "${elfdir}" = "elf/coreboot" ] && \ + elfdir="elf/coreboot_nopayload_DO_NOT_FLASH" cfgsdir="config/${project}" [ -d "${cfgsdir}" ] || err "directory, ${cfgsdir}, does not exist" @@ -88,6 +90,13 @@ build_targets() [ $# -gt 0 ] && targets=$@ [ -z "${mode}" ] && x_ mkdir -p "${elfdir}/" + if [ "${project}" != "coreboot" ]; then + printf "DO NOT flash coreboot images under elf/. Use ROMs" + printf "under bin/. The elf/ ones DO NOT contain payloads" + printf "will brick your machine.\n" > \ + "${elfdir}/WARNING.txt" || : + fi + handle_targets } -- cgit v1.2.1