[S] Change in coreboot[master]: payloads: Make PAYLOAD_NONE a bool outside of the choice

[Martin L Roth (Code Review)]

Attention is currently required from: Martin Roth.

Hello Martin Roth,

I'd like you to do a code review.
Please visit

https://review.coreboot.org/c/coreboot/+/68595

to review the following change.

Change subject: payloads: Make PAYLOAD_NONE a bool outside of the choice
......................................................................

payloads: Make PAYLOAD_NONE a bool outside of the choice

Instead of having the config option PAYLOADS_NONE inside the choice of
payloads, make that a separate choice that enables the payload menu.

If the no_payload option is selected, this hides the other options
and keeps them out of the saved config file and config.h

Signed-off-by: Martin Roth 
Change-Id: I932c65630261a5b39809abf4dfbda5bf932c6684
---
M payloads/Kconfig
1 file changed, 26 insertions(+), 9 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/95/68595/1

diff --git a/payloads/Kconfig b/payloads/Kconfig
index 2e627d7..09e8e8e 100644
--- a/payloads/Kconfig
+++ b/payloads/Kconfig
@@ -1,15 +1,8 @@
 menu "Payload"

-config NO_DEFAULT_PAYLOAD
-	bool
-
-choice
-	prompt "Add a payload"
-	default PAYLOAD_NONE if NO_DEFAULT_PAYLOAD || !ARCH_X86
-	default PAYLOAD_SEABIOS if ARCH_X86
-
 config PAYLOAD_NONE
-	bool "None"
+	bool "Don't add a payload"
+	default n
    help
      Select this option if you want to create an "empty" coreboot
      ROM image for a certain mainboard, i.e. a coreboot ROM image
@@ -18,6 +11,11 @@
      For such an image to be useful, you have to use 'cbfstool'
      to add a payload to the ROM image later.

+if !PAYLOAD_NONE
+choice
+	prompt "Payload to add"
+	default PAYLOAD_SEABIOS if ARCH_X86
+
 config PAYLOAD_ELF
    bool "An ELF executable payload"
    help
@@ -178,4 +176,7 @@
 source "payloads/external/*/Kconfig.secondary"

endmenu # "Secondary Payloads"
+
+endif # !PAYLOAD_NONE
+
 endmenu