[coreboot-gerrit] Change in coreboot[master]: nb/intel/x4x/Kconfig: Don't fix CBFS_SIZE on i82801gx southb...

Arthur Heymans (Code Review) gerrit at coreboot.org
Thu Apr 6 21:43:44 CEST 2017


Arthur Heymans has uploaded a new change for review. ( https://review.coreboot.org/19181 )

Change subject: nb/intel/x4x/Kconfig: Don't fix CBFS_SIZE on i82801gx southbridge
......................................................................

nb/intel/x4x/Kconfig: Don't fix CBFS_SIZE on i82801gx southbridge

The x4x northbridge can be paired with either an ICH7 (in the case of
g41) or an ICH10 (all other cases: g45, q45, p45, ...). Only ICH10
sometimes occurs with a descriptor, gbe and an ME region.

ICH7 is always descriptorless so it makes no sense to fix CBFS to
accommodate for those other objects.

Change-Id: I4a01dfdbce1807e44932a3ac812110382332abd8
Signed-off-by: Arthur Heymans <arthur at aheymans.xyz>
---
M src/northbridge/intel/x4x/Kconfig
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/81/19181/1

diff --git a/src/northbridge/intel/x4x/Kconfig b/src/northbridge/intel/x4x/Kconfig
index cbbd9de..716a015 100644
--- a/src/northbridge/intel/x4x/Kconfig
+++ b/src/northbridge/intel/x4x/Kconfig
@@ -28,7 +28,7 @@
 
 config CBFS_SIZE
 	hex
-	default 0x100000
+	default 0x100000 if !SOUTHBRIDGE_INTEL_I82801GX
 
 config BOOTBLOCK_NORTHBRIDGE_INIT
 	string

-- 
To view, visit https://review.coreboot.org/19181
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4a01dfdbce1807e44932a3ac812110382332abd8
Gerrit-PatchSet: 1
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Arthur Heymans <arthur at aheymans.xyz>



More information about the coreboot-gerrit mailing list