[coreboot-gerrit] New patch to review for coreboot: 12437c0 Hide PLATFORM_USES_FSP1_1.

Vladimir Serbinenko (phcoder@gmail.com) gerrit at coreboot.org
Sat May 30 22:59:20 CEST 2015


Vladimir Serbinenko (phcoder at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10378

-gerrit

commit 12437c04f8218e2d08ffb4bf2d20cab04e6bff27
Author: Vladimir Serbinenko <phcoder at gmail.com>
Date:   Sat May 30 22:30:13 2015 +0200

    Hide PLATFORM_USES_FSP1_1.
    
    This should be an internal selectable variable rather than user-visible config.
    Moreover the description is misleading.
    
    This is a typical case of an option "Should it work?" where there is only one
    right answer yet we still ask it.
    
    Change-Id: Idc0ce2e1b9f89eddd034966cc877483d994ce0eb
    Signed-off-by: Vladimir Serbinenko <phcoder at gmail.com>
---
 src/drivers/intel/fsp1_1/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/drivers/intel/fsp1_1/Kconfig b/src/drivers/intel/fsp1_1/Kconfig
index a6d34ee..f722e65 100644
--- a/src/drivers/intel/fsp1_1/Kconfig
+++ b/src/drivers/intel/fsp1_1/Kconfig
@@ -20,7 +20,7 @@
 comment "Intel FSP"
 
 config PLATFORM_USES_FSP1_1
-	bool "Use Intel Firmware Support Package"
+	bool
 	help
 	  Does the code require the Intel Firmware Support Package?
 



More information about the coreboot-gerrit mailing list