From 587d588fe4ccc1587a2ad189863aeb82a41e7252 Mon Sep 17 00:00:00 2001 From: Leah Rowe Date: Thu, 21 Nov 2024 17:56:50 +0000 Subject: rom.sh: Don't build txtmode U-Boot images U-Boot needs a VESA framebuffer or native coreboot framebuffer to work properly. Signed-off-by: Leah Rowe --- include/rom.sh | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) (limited to 'include') diff --git a/include/rom.sh b/include/rom.sh index c2896b85..6ffbb228 100644 --- a/include/rom.sh +++ b/include/rom.sh @@ -95,7 +95,6 @@ mkcorebootbin() initmode="${defconfig##*/}"; displaymode="${initmode##*_}" initmode="${initmode%%_*}" - [ -n "$displaymode" ] && displaymode="_$displaymode" cbfstool="elf/cbfstool/$tree/cbfstool" [ "$payload_uboot_i386" = "y" ] && \ @@ -155,7 +154,8 @@ add_seabios() [ "$payload_grub" = "y" ] && add_grub cprom - [ "$payload_uboot_amd64" = "y" ] && pname="seauboot" && cprom "seauboot" + [ "$payload_uboot_amd64" = "y" ] && [ "$displaymode" != "txtmode" ] && \ + pname="seauboot" && cprom "seauboot" [ "$payload_grub" = "y" ] && pname="seagrub" && mkseagrub; : } @@ -177,6 +177,11 @@ mkseagrub() add_uboot() { + if [ "$displaymode" = "txtmode" ]; then + printf "cb/$target: Cannot use U-Boot in text mode\n" 1>&2 + return 0 + fi + # TODO: re-work to allow each coreboot target to say which ub tree # instead of hardcoding as in the current logic below: @@ -211,7 +216,8 @@ add_uboot() cprom() { - newrom="bin/$target/${pname}_${target}_$initmode$displaymode.rom" + newrom="bin/$target/${pname}_${target}_$initmode.rom" + [ -n "$displaymode" ] && newrom="${newrom%.rom}_$displaymode.rom" [ $# -gt 0 ] && [ "$1" != "seauboot" ] && \ newrom="${newrom%.rom}_${1%.gkb}.rom" -- cgit v1.2.1