[coreboot-gerrit] Patch set updated for coreboot: cpu/x86: Kconfig cleanups

Stefan Reinauer (stefan.reinauer@coreboot.org) gerrit at coreboot.org
Wed Feb 24 23:07:23 CET 2016


Stefan Reinauer (stefan.reinauer at coreboot.org) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/10012

-gerrit

commit c101160eceeb171353fcc2a2b340fbd2f5893b56
Author: Stefan Reinauer <reinauer at chromium.org>
Date:   Mon Apr 27 13:40:16 2015 -0700

    cpu/x86: Kconfig cleanups
    
    Sort some Kconfig options
    
    Change-Id: I25ea327ed151e18ccb5d13626d44925d2a253d08
    Signed-off-by: Stefan Reinauer <stefan.reinauer at coreboot.org>
---
 src/cpu/x86/Kconfig | 23 ++++++++++++++---------
 1 file changed, 14 insertions(+), 9 deletions(-)

diff --git a/src/cpu/x86/Kconfig b/src/cpu/x86/Kconfig
index 94225a3..d8f338d 100644
--- a/src/cpu/x86/Kconfig
+++ b/src/cpu/x86/Kconfig
@@ -1,7 +1,16 @@
+# TODO These two options look too similar
 config PARALLEL_CPU_INIT
 	bool
 	default n
 
+config PARALLEL_MP
+	def_bool n
+	help
+	 This option uses common MP infrastructure for bringing up APs
+	 in parallel. It additionally provides a more flexible mechanism
+	 for sequencing the steps of bringing up the APs.
+
+
 config UDELAY_IO
 	bool
 	default y if !UDELAY_LAPIC && !UDELAY_TSC && !UDELAY_TIMER2
@@ -39,11 +48,14 @@ config TSC_MONOTONIC_TIMER
 	help
 	  Expose monotonic time using the TSC.
 
-config UDELAY_TIMER2
+# This option is used in code but never selected.
+config TSC_CALIBRATE_WITH_IO
 	bool
+	depends on UDELAY_TSC
 	default n
 
-config TSC_CALIBRATE_WITH_IO
+# This option is used in code but never selected.
+config UDELAY_TIMER2
 	bool
 	default n
 
@@ -121,13 +133,6 @@ config PLATFORM_USES_FSP1_0
 	  Selected for Intel processors/platform combinations that use the
 	  Intel Firmware Support Package (FSP) 1.0 for initialization.
 
-config PARALLEL_MP
-	def_bool n
-	help
-	 This option uses common MP infrastructure for bringing up APs
-	 in parallel. It additionally provides a more flexible mechanism
-	 for sequencing the steps of bringing up the APs.
-
 config BACKUP_DEFAULT_SMM_REGION
 	def_bool n
 	help



More information about the coreboot-gerrit mailing list