[coreboot-gerrit] Patch set updated for coreboot: Fix leaking CONFIG_VGA=y

Kyösti Mälkki (kyosti.malkki@gmail.com) gerrit at coreboot.org
Thu May 26 14:54:36 CEST 2016


Kyösti Mälkki (kyosti.malkki at gmail.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/14823

-gerrit

commit 7a10a31b39a9be8d23a12ac91f90b125e5c5b987
Author: Kyösti Mälkki <kyosti.malkki at gmail.com>
Date:   Mon May 16 14:49:57 2016 +0300

    Fix leaking CONFIG_VGA=y
    
    Items under DEVICE_SPECIFIC_OPTIONS got selected without
    the driver being selected.
    
    Change-Id: I1797fa6175620a9291873559a6308eaea85a090e
    Signed-off-by: Kyösti Mälkki <kyosti.malkki at gmail.com>
---
 src/drivers/aspeed/common/Kconfig         | 4 ++--
 src/drivers/xgi/common/Kconfig            | 4 ++--
 src/northbridge/intel/sandybridge/Kconfig | 1 +
 3 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/drivers/aspeed/common/Kconfig b/src/drivers/aspeed/common/Kconfig
index 0f7056b..a879217 100644
--- a/src/drivers/aspeed/common/Kconfig
+++ b/src/drivers/aspeed/common/Kconfig
@@ -1,10 +1,10 @@
 config DRIVERS_ASPEED_AST_COMMON
 	bool
 
-if !MAINBOARD_DO_NATIVE_VGA_INIT
+if DRIVERS_ASPEED_AST_COMMON
 
 config DEVICE_SPECIFIC_OPTIONS # dummy
 	def_bool y
 	select VGA
 
-endif # MAINBOARD_DO_NATIVE_VGA_INIT
+endif # DRIVERS_ASPEED_AST_COMMON
diff --git a/src/drivers/xgi/common/Kconfig b/src/drivers/xgi/common/Kconfig
index e0ed9d6..44a0908 100644
--- a/src/drivers/xgi/common/Kconfig
+++ b/src/drivers/xgi/common/Kconfig
@@ -1,10 +1,10 @@
 config DRIVERS_XGI_Z79_COMMON
 	bool
 
-if !MAINBOARD_DO_NATIVE_VGA_INIT
+if DRIVERS_XGI_Z79_COMMON
 
 config DEVICE_SPECIFIC_OPTIONS # dummy
 	def_bool y
 	select VGA
 
-endif # MAINBOARD_DO_NATIVE_VGA_INIT
+endif # DRIVERS_XGI_Z79_COMMON
diff --git a/src/northbridge/intel/sandybridge/Kconfig b/src/northbridge/intel/sandybridge/Kconfig
index 64f4a14..0cfdf2b 100644
--- a/src/northbridge/intel/sandybridge/Kconfig
+++ b/src/northbridge/intel/sandybridge/Kconfig
@@ -55,6 +55,7 @@ config CACHE_MRC_SIZE_KB
 
 config SANDYBRIDGE_IVYBRIDGE_LVDS
 	def_bool n
+	select VGA
 	select MAINBOARD_HAS_NATIVE_VGA_INIT
 	select MAINBOARD_HAS_NATIVE_VGA_INIT_TEXTMODECFG
 



More information about the coreboot-gerrit mailing list