[coreboot-gerrit] Change in coreboot[master]: payloads: Only show supported payloads in menuconfig

Philipp Deppenwiese (Code Review) gerrit at coreboot.org
Tue Oct 9 13:29:58 CEST 2018


Philipp Deppenwiese has submitted this change and it was merged. ( https://review.coreboot.org/28774 )

Change subject: payloads: Only show supported payloads in menuconfig
......................................................................

payloads: Only show supported payloads in menuconfig

Some payloads don't depend on their supported platforms, which makes
them visible in menuconfig even if they are not supported (yet).

Change-Id: Ia1ec0f0ca0b6521d0c024500c67326337f3d744f
Signed-off-by: Philipp Hug <philipp at hug.cx>
Reviewed-on: https://review.coreboot.org/28774
Tested-by: build bot (Jenkins) <no-reply at coreboot.org>
Reviewed-by: Martin Roth <martinroth at google.com>
Reviewed-by: Philipp Deppenwiese <zaolin.daisuki at gmail.com>
---
M payloads/Kconfig
M payloads/external/FILO/Kconfig.name
M payloads/external/linux/Kconfig.name
M payloads/external/tianocore/Kconfig.name
4 files changed, 5 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Martin Roth: Looks good to me, approved
  Philipp Deppenwiese: Looks good to me, approved



diff --git a/payloads/Kconfig b/payloads/Kconfig
index 8e05df3..c7a7ba6 100644
--- a/payloads/Kconfig
+++ b/payloads/Kconfig
@@ -30,6 +30,7 @@
 
 config PAYLOAD_FIT
 	bool "A FIT payload"
+	depends on ARCH_ARM64
 	select PAYLOAD_FIT_SUPPORT
 	help
 	  Select this option if you have a payload image (a FIT file) which
@@ -41,6 +42,7 @@
 
 config PAYLOAD_BAYOU
 	bool "Bayou"
+	depends on ARCH_X86
 	help
 	  Select this option if you want to set bayou as your primary
 	  payload.
diff --git a/payloads/external/FILO/Kconfig.name b/payloads/external/FILO/Kconfig.name
index ca94343..28a41dc 100644
--- a/payloads/external/FILO/Kconfig.name
+++ b/payloads/external/FILO/Kconfig.name
@@ -1,5 +1,6 @@
 config PAYLOAD_FILO
 	bool "FILO"
+	depends on ARCH_X86
 	help
 	  Select this option if you want to build a coreboot image
 	  with a FILO payload. If you don't know what this is
diff --git a/payloads/external/linux/Kconfig.name b/payloads/external/linux/Kconfig.name
index ae1d906..63621d8 100644
--- a/payloads/external/linux/Kconfig.name
+++ b/payloads/external/linux/Kconfig.name
@@ -1,5 +1,6 @@
 config PAYLOAD_LINUX
 	bool "A Linux payload"
+	depends on ARCH_X86 || ARCH_ARM
 	help
 	  Select this option if you have a Linux bzImage which coreboot
 	  should run as soon as the basic hardware initialization
diff --git a/payloads/external/tianocore/Kconfig.name b/payloads/external/tianocore/Kconfig.name
index e57925f..a56d248 100644
--- a/payloads/external/tianocore/Kconfig.name
+++ b/payloads/external/tianocore/Kconfig.name
@@ -1,5 +1,6 @@
 config PAYLOAD_TIANOCORE
 	bool "Tianocore coreboot payload package"
+	depends on ARCH_X86
 	help
 	  Select this option if you want to build a coreboot image
 	  with a Tianocore payload. If you don't know what this is

-- 
To view, visit https://review.coreboot.org/28774
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia1ec0f0ca0b6521d0c024500c67326337f3d744f
Gerrit-Change-Number: 28774
Gerrit-PatchSet: 4
Gerrit-Owner: Philipp Hug <philipp at hug.cx>
Gerrit-Reviewer: Angel Pons <th3fanbus at gmail.com>
Gerrit-Reviewer: Jonathan Neuschäfer <j.neuschaefer at gmx.net>
Gerrit-Reviewer: Martin Roth <martinroth at google.com>
Gerrit-Reviewer: Patrick Rudolph <siro at das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter at users.sourceforge.net>
Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki at gmail.com>
Gerrit-Reviewer: Philipp Hug <philipp at hug.cx>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer at coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply at coreboot.org>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20181009/9ace8614/attachment.html>


More information about the coreboot-gerrit mailing list