From 742a0e911ca1682a06856731c69479aaf9c77ab4 Mon Sep 17 00:00:00 2001 From: Arthur Heymans Date: Mon, 29 Jan 2018 16:34:46 +0100 Subject: nb/intel/sandybridge: Always use the same MMCONF_BASE_ADDRESS 'Optimizing' MMCONF_BASE_ADDRESS for the native codepath prevents the use of fallback/normal with both the native raminit and the mrc.bin. Using the same MMCONF_BASE_ADDRESS as the mrc.bin codepath means that 128MB less is available to devices using the native raminit. Most devices reserve 2048M for non memory resources below 4G, which in most cases is more than adequate. Devices with only 1024M (and that don't already use the mrc.bin) are: * lenovo/x220 * lenovo/x230 * lenovo/x131e * lenovo/x1_carbon_gen1 Those could fail to allocate PCI resources, but on at least x220 with a somewhat default configuration (USB3 expresscard, Wireless PCIe card) it still boots fine, so one should not expect many problems from this change. Change-Id: I1d0648fe36c88bd9279ac19e5c710055327599fd Signed-off-by: Arthur Heymans Reviewed-on: https://review.coreboot.org/23490 Tested-by: build bot (Jenkins) Reviewed-by: Nico Huber --- src/northbridge/intel/sandybridge/Kconfig | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/northbridge/intel/sandybridge/Kconfig b/src/northbridge/intel/sandybridge/Kconfig index 36cf940350..fce1a49092 100644 --- a/src/northbridge/intel/sandybridge/Kconfig +++ b/src/northbridge/intel/sandybridge/Kconfig @@ -94,11 +94,9 @@ config BOOTBLOCK_NORTHBRIDGE_INIT config MMCONF_BASE_ADDRESS hex - default 0xf8000000 if USE_NATIVE_RAMINIT default 0xf0000000 help - We can optimize the native case but the MRC blob requires it - to be at 0xf0000000. + The MRC blob requires it to be at 0xf0000000. if USE_NATIVE_RAMINIT -- cgit v1.2.3