New patch to review for coreboot: src/Kconfig: Move bootblock behavior to arch/x86 as TODO suggested

[Martin Roth]

Martin Roth ([e-mail]) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17909

-gerrit

commit a96501521c3d5b9453438af0154f991881e3fef5
Author: Martin Roth 
Date:   Thu Dec 15 16:04:55 2016 -0700

src/Kconfig: Move bootblock behavior to arch/x86 as TODO suggested

The four options are only used in X86:
    - BOOTBLOCK_SIMPLE
    - BOOTBLOCK_NORMAL
    - BOOTBLOCK_SOURCE
    - SKIP_MAX_REBOOT_CNT_CLEAR

Move them all into src/arch/x86/Kconfig - this puts them in the chipset
    menu instead of general setup.

Verified that this makes no significant changes to any config file.

Change-Id: I2798ef67a8c6aed5afac34322be15fdf0c794059
    Signed-off-by: Martin Roth 
---
 src/Kconfig          | 27 ---------------------------
 src/arch/x86/Kconfig | 26 ++++++++++++++++++++++++++
 2 files changed, 26 insertions(+), 27 deletions(-)

diff --git a/src/Kconfig b/src/Kconfig
index 84ea565..8337264 100644
--- a/src/Kconfig
+++ b/src/Kconfig
@@ -214,33 +214,6 @@ config CACHE_RELOCATED_RAMSTAGE_OUTSIDE_CBMEM
     The relocated ramstage is saved in an area specified by the
     by the board and/or chipset.

-# TODO: This doesn't belong here, move to src/arch/x86/Kconfig
-choice
-	prompt "Bootblock behaviour"
-	default BOOTBLOCK_SIMPLE
-
-config BOOTBLOCK_SIMPLE
-	bool "Always load fallback"
-
-config BOOTBLOCK_NORMAL
-	bool "Switch to normal if CMOS says so"
-
-endchoice
-
-config BOOTBLOCK_SOURCE
-	string
-	default "bootblock_simple.c" if BOOTBLOCK_SIMPLE
-	default "bootblock_normal.c" if BOOTBLOCK_NORMAL
-
-config SKIP_MAX_REBOOT_CNT_CLEAR
-	bool "Do not clear reboot count after successful boot"
-	depends on BOOTBLOCK_NORMAL
-	help
-	  Do not clear the reboot count immediately after successful boot.
-	  Set to allow the payload to control normal/fallback image recovery.
-	  Note that it is the responsibility of the payload to reset the
-	  normal boot bit to 1 after each successsful boot.
-
 config UPDATE_IMAGE
    bool "Update existing coreboot.rom image"
    help
diff --git a/src/arch/x86/Kconfig b/src/arch/x86/Kconfig
index cd5f8e5..2ae5873 100644
--- a/src/arch/x86/Kconfig
+++ b/src/arch/x86/Kconfig
@@ -208,3 +208,29 @@ config ROMSTAGE_DEBUG_SPINLOOP
    help
      Add a spin (JMP .) in assembly_entry.S during early romstage to wait
      for a JTAG debugger to break into the execution sequence.
+
+choice
+	prompt "Bootblock behaviour"
+	default BOOTBLOCK_SIMPLE
+
+config BOOTBLOCK_SIMPLE
+	bool "Always load fallback"
+
+config BOOTBLOCK_NORMAL
+	bool "Switch to normal if CMOS says so"
+
+endchoice
+
+config BOOTBLOCK_SOURCE
+	string
+	default "bootblock_simple.c" if BOOTBLOCK_SIMPLE
+	default "bootblock_normal.c" if BOOTBLOCK_NORMAL
+
+config SKIP_MAX_REBOOT_CNT_CLEAR
+	bool "Do not clear reboot count after successful boot"
+	depends on BOOTBLOCK_NORMAL
+	help
+	  Do not clear the reboot count immediately after successful boot.
+	  Set to allow the payload to control normal/fallback image recovery.
+	  Note that it is the responsibility of the payload to reset the
+	  normal boot bit to 1 after each successsful boot.