Stefan Reinauer (stefan.reinauer@coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10828
-gerrit
commit 2b2e1e594e0825599c967c521b60adf02852b925 Author: Stefan Reinauer stefan.reinauer@coreboot.org Date: Tue Jun 30 15:58:56 2015 -0700
payloads: Reorganize Kconfig for external payloads
The integration of external payloads in coreboot is a bit messy. You have to change the to level Kconfig file for every payload (something that we recently fixed for mainboards and chipsets). This means that updating e.g. the SeaBIOS version requires a change outside of the SeaBIOS directory.
With this patch you can create a new directory under payloads/external and place a Kconfig and Kconfig.name file in there, and the payload will automatically show up when you do "make menuconfig".
Change-Id: I293abcb8eae581d4b3934e64897c0d339a27e7c1 Signed-off-by: Stefan Reinauer stefan.reinauer@coreboot.org --- payloads/external/FILO/Kconfig | 20 ++++ payloads/external/FILO/Kconfig.name | 8 ++ payloads/external/GRUB2/Kconfig | 15 +++ payloads/external/GRUB2/Kconfig.name | 8 ++ payloads/external/SeaBIOS/Kconfig | 58 ++++++++++ payloads/external/SeaBIOS/Kconfig.name | 9 ++ payloads/external/linux/Kconfig | 20 ++++ payloads/external/linux/Kconfig.name | 9 ++ payloads/external/tianocore/Kconfig | 6 + payloads/external/tianocore/Kconfig.name | 8 ++ src/Kconfig | 184 +------------------------------ 11 files changed, 164 insertions(+), 181 deletions(-)
diff --git a/payloads/external/FILO/Kconfig b/payloads/external/FILO/Kconfig new file mode 100644 index 0000000..0215aca --- /dev/null +++ b/payloads/external/FILO/Kconfig @@ -0,0 +1,20 @@ +choice + prompt "FILO version" + default FILO_STABLE + depends on PAYLOAD_FILO + +config FILO_STABLE + bool "0.6.0" + help + Stable FILO version + +config FILO_MASTER + bool "HEAD" + help + Newest FILO version + +endchoice + +config PAYLOAD_FILE + depends on PAYLOAD_FILO + default "payloads/external/FILO/filo/build/filo.elf" diff --git a/payloads/external/FILO/Kconfig.name b/payloads/external/FILO/Kconfig.name new file mode 100644 index 0000000..244077d --- /dev/null +++ b/payloads/external/FILO/Kconfig.name @@ -0,0 +1,8 @@ +config PAYLOAD_FILO + bool "FILO" + help + Select this option if you want to build a coreboot image + with a FILO payload. If you don't know what this is + about, just leave it enabled. + + See http://coreboot.org/Payloads for more information. diff --git a/payloads/external/GRUB2/Kconfig b/payloads/external/GRUB2/Kconfig new file mode 100644 index 0000000..008efa5 --- /dev/null +++ b/payloads/external/GRUB2/Kconfig @@ -0,0 +1,15 @@ +choice + prompt "GRUB2 version" + default GRUB2_MASTER + depends on PAYLOAD_GRUB2 + +config GRUB2_MASTER + bool "HEAD" + help + Newest GRUB2 version + +endchoice + +config PAYLOAD_FILE + depends on PAYLOAD_GRUB2 + default "payloads/external/GRUB2/grub2/build/default_payload.elf" diff --git a/payloads/external/GRUB2/Kconfig.name b/payloads/external/GRUB2/Kconfig.name new file mode 100644 index 0000000..c3a8ddf --- /dev/null +++ b/payloads/external/GRUB2/Kconfig.name @@ -0,0 +1,8 @@ +config PAYLOAD_GRUB2 + bool "GRUB2" + help + Select this option if you want to build a coreboot image + with a GRUB2 payload. If you don't know what this is + about, just leave it enabled. + + See http://coreboot.org/Payloads for more information. diff --git a/payloads/external/SeaBIOS/Kconfig b/payloads/external/SeaBIOS/Kconfig new file mode 100644 index 0000000..5df19f8 --- /dev/null +++ b/payloads/external/SeaBIOS/Kconfig @@ -0,0 +1,58 @@ +choice + prompt "SeaBIOS version" + default SEABIOS_STABLE + depends on PAYLOAD_SEABIOS + +config SEABIOS_STABLE + bool "1.7.5" + help + Stable SeaBIOS version +config SEABIOS_MASTER + bool "master" + help + Newest SeaBIOS version + +endchoice + +config SEABIOS_PS2_TIMEOUT + prompt "PS/2 keyboard controller initialization timeout (milliseconds)" if PAYLOAD_SEABIOS + default 0 + depends on EXPERT + int + help + Some PS/2 keyboard controllers don't respond to commands immediately + after powering on. This specifies how long SeaBIOS will wait for the + keyboard controller to become ready before giving up. + +config SEABIOS_THREAD_OPTIONROMS + prompt "Hardware init during option ROM execution" if PAYLOAD_SEABIOS + default n + bool + help + Allow hardware init to run in parallel with optionrom execution. + + This can reduce boot time, but can cause some timing + variations during option ROM code execution. It is not + known if all option ROMs will behave properly with this option. + +config SEABIOS_MALLOC_UPPERMEMORY + bool + default y + depends on PAYLOAD_SEABIOS + help + Use the "Upper Memory Block" area (0xc0000-0xf0000) for internal + "low memory" allocations. If this is not selected, the memory is + instead allocated from the "9-segment" (0x90000-0xa0000). + This is not typically needed, but may be required on some platforms + to allow USB and SATA buffers to be written correctly by the + hardware. In general, if this is desired, the option will be + set to 'N' by the chipset Kconfig. + +config PAYLOAD_FILE + depends on PAYLOAD_SEABIOS + default "payloads/external/SeaBIOS/seabios/out/bios.bin.elf" + +config PAYLOAD_VGABIOS_FILE + string + depends on PAYLOAD_SEABIOS && SEABIOS_VGA_COREBOOT + default "payloads/external/SeaBIOS/seabios/out/vgabios.bin" diff --git a/payloads/external/SeaBIOS/Kconfig.name b/payloads/external/SeaBIOS/Kconfig.name new file mode 100644 index 0000000..9ab235d --- /dev/null +++ b/payloads/external/SeaBIOS/Kconfig.name @@ -0,0 +1,9 @@ +config PAYLOAD_SEABIOS + bool "SeaBIOS" + depends on ARCH_X86 + help + Select this option if you want to build a coreboot image + with a SeaBIOS payload. If you don't know what this is + about, just leave it enabled. + + See http://coreboot.org/Payloads for more information. diff --git a/payloads/external/linux/Kconfig b/payloads/external/linux/Kconfig new file mode 100644 index 0000000..9956a23 --- /dev/null +++ b/payloads/external/linux/Kconfig @@ -0,0 +1,20 @@ +config PAYLOAD_FILE + string "Linux path and filename" + depends on PAYLOAD_LINUX + default "bzImage" + help + The path and filename of the bzImage kernel to use as payload. + +config LINUX_COMMAND_LINE + string "Linux command line" + depends on PAYLOAD_LINUX + default "" + help + A command line to add to the Linux kernel. + +config LINUX_INITRD + string "Linux initrd" + depends on PAYLOAD_LINUX + default "" + help + An initrd image to add to the Linux kernel. diff --git a/payloads/external/linux/Kconfig.name b/payloads/external/linux/Kconfig.name new file mode 100644 index 0000000..ae1d906 --- /dev/null +++ b/payloads/external/linux/Kconfig.name @@ -0,0 +1,9 @@ +config PAYLOAD_LINUX + bool "A Linux payload" + help + Select this option if you have a Linux bzImage which coreboot + should run as soon as the basic hardware initialization + is completed. + + You will be able to specify the location and file name of the + payload image later. diff --git a/payloads/external/tianocore/Kconfig b/payloads/external/tianocore/Kconfig new file mode 100644 index 0000000..e6d3a3b --- /dev/null +++ b/payloads/external/tianocore/Kconfig @@ -0,0 +1,6 @@ +config PAYLOAD_FILE + string "Tianocore firmware volume" + depends on PAYLOAD_TIANOCORE + default "COREBOOT.fd" + help + The result of a corebootPkg build diff --git a/payloads/external/tianocore/Kconfig.name b/payloads/external/tianocore/Kconfig.name new file mode 100644 index 0000000..d57f7e1 --- /dev/null +++ b/payloads/external/tianocore/Kconfig.name @@ -0,0 +1,8 @@ +config PAYLOAD_TIANOCORE + bool "Tiano Core" + help + Select this option if you want to build a coreboot image + with a Tiano Core payload. If you don't know what this is + about, just leave it enabled. + + See http://coreboot.org/Payloads for more information. diff --git a/src/Kconfig b/src/Kconfig index 94b3508..a2129ad 100644 --- a/src/Kconfig +++ b/src/Kconfig @@ -667,144 +667,11 @@ config PAYLOAD_ELF You will be able to specify the location and file name of the payload image later.
-config PAYLOAD_LINUX - bool "A Linux payload" - help - Select this option if you have a Linux bzImage which coreboot - should run as soon as the basic hardware initialization - is completed. - - You will be able to specify the location and file name of the - payload image later. - -config PAYLOAD_SEABIOS - bool "SeaBIOS" - depends on ARCH_X86 - help - Select this option if you want to build a coreboot image - with a SeaBIOS payload. If you don't know what this is - about, just leave it enabled. - - See http://coreboot.org/Payloads for more information. - -config PAYLOAD_FILO - bool "FILO" - help - Select this option if you want to build a coreboot image - with a FILO payload. If you don't know what this is - about, just leave it enabled. - - See http://coreboot.org/Payloads for more information. - -config PAYLOAD_GRUB2 - bool "GRUB2" - help - Select this option if you want to build a coreboot image - with a GRUB2 payload. If you don't know what this is - about, just leave it enabled. - - See http://coreboot.org/Payloads for more information. - -config PAYLOAD_TIANOCORE - bool "Tiano Core" - help - Select this option if you want to build a coreboot image - with a Tiano Core payload. If you don't know what this is - about, just leave it enabled. - - See http://coreboot.org/Payloads for more information. - -endchoice - -choice - prompt "SeaBIOS version" - default SEABIOS_STABLE - depends on PAYLOAD_SEABIOS - -config SEABIOS_STABLE - bool "1.7.5" - help - Stable SeaBIOS version -config SEABIOS_MASTER - bool "master" - help - Newest SeaBIOS version - -endchoice - -config SEABIOS_PS2_TIMEOUT - prompt "PS/2 keyboard controller initialization timeout (milliseconds)" if PAYLOAD_SEABIOS - default 0 - depends on EXPERT - int - help - Some PS/2 keyboard controllers don't respond to commands immediately - after powering on. This specifies how long SeaBIOS will wait for the - keyboard controller to become ready before giving up. - -config SEABIOS_THREAD_OPTIONROMS - prompt "Hardware init during option ROM execution" if PAYLOAD_SEABIOS - default n - bool - help - Allow hardware init to run in parallel with optionrom execution. - - This can reduce boot time, but can cause some timing - variations during option ROM code execution. It is not - known if all option ROMs will behave properly with this option. - -config SEABIOS_MALLOC_UPPERMEMORY - bool - default y - depends on PAYLOAD_SEABIOS - help - Use the "Upper Memory Block" area (0xc0000-0xf0000) for internal - "low memory" allocations. If this is not selected, the memory is - instead allocated from the "9-segment" (0x90000-0xa0000). - This is not typically needed, but may be required on some platforms - to allow USB and SATA buffers to be written correctly by the - hardware. In general, if this is desired, the option will be - set to 'N' by the chipset Kconfig. - -config SEABIOS_VGA_COREBOOT - prompt "Include generated option rom that implements legacy VGA BIOS compatibility" if PAYLOAD_SEABIOS - default n - depends on !VGA_BIOS && (MAINBOARD_DO_NATIVE_VGA_INIT || MAINBOARD_HAS_NATIVE_VGA_INIT_TEXTMODECFG) - bool - help - Coreboot can initialize the GPU of some mainboards. - - After initializing the GPU, the information about it can be passed to the payload. - Provide an option rom that implements this legacy VGA BIOS compatibility requirement. - -choice - prompt "GRUB2 version" - default GRUB2_MASTER - depends on PAYLOAD_GRUB2 - -config GRUB2_MASTER - bool "HEAD" - help - Newest GRUB2 version +source "payloads/external/*/Kconfig.name"
endchoice
-choice - prompt "FILO version" - default FILO_STABLE - depends on PAYLOAD_FILO - -config FILO_STABLE - bool "0.6.0" - help - Stable FILO version - -config FILO_MASTER - bool "HEAD" - help - Newest FILO version - -endchoice +source "payloads/external/*/Kconfig"
config PAYLOAD_FILE string "Payload path and filename" @@ -813,60 +680,15 @@ config PAYLOAD_FILE help The path and filename of the ELF executable file to use as payload.
-config PAYLOAD_FILE - string "Linux path and filename" - depends on PAYLOAD_LINUX - default "bzImage" - help - The path and filename of the bzImage kernel to use as payload. - -config PAYLOAD_FILE - depends on PAYLOAD_SEABIOS - default "payloads/external/SeaBIOS/seabios/out/bios.bin.elf" - -config PAYLOAD_VGABIOS_FILE - string - depends on PAYLOAD_SEABIOS && SEABIOS_VGA_COREBOOT - default "payloads/external/SeaBIOS/seabios/out/vgabios.bin" - -config PAYLOAD_FILE - depends on PAYLOAD_FILO - default "payloads/external/FILO/filo/build/filo.elf" - -config PAYLOAD_FILE - depends on PAYLOAD_GRUB2 - default "payloads/external/GRUB2/grub2/build/default_payload.elf" - -config PAYLOAD_FILE - string "Tianocore firmware volume" - depends on PAYLOAD_TIANOCORE - default "COREBOOT.fd" - help - The result of a corebootPkg build - # TODO: Defined if no payload? Breaks build? config COMPRESSED_PAYLOAD_LZMA bool "Use LZMA compression for payloads" default y - depends on PAYLOAD_ELF || PAYLOAD_SEABIOS || PAYLOAD_FILO || PAYLOAD_TIANOCORE || PAYLOAD_GRUB2 + depends on !PAYLOAD_NONE && !PAYLOAD_LINUX help In order to reduce the size payloads take up in the ROM chip coreboot can compress them using the LZMA algorithm.
-config LINUX_COMMAND_LINE - string "Linux command line" - depends on PAYLOAD_LINUX - default "" - help - A command line to add to the Linux kernel. - -config LINUX_INITRD - string "Linux initrd" - depends on PAYLOAD_LINUX - default "" - help - An initrd image to add to the Linux kernel. - endmenu
menu "Debugging"