From 16a70a48c68f9027b0900e15fc9e3bc14391ad44 Mon Sep 17 00:00:00 2001 From: Arthur Heymans Date: Fri, 22 Sep 2017 12:22:24 +0200 Subject: nb/intel/x4x: Change memory layout to improve MTRR This change also makes sure that the sum the uma regions (TSEG, GSM, GSM) is 4MiB aligned. This is needed to avoid cbmem_top floating between 2 usable ram region, since cbmem_top is aligned 4MiB down to easy MTRR setup for ramstage. At least tianocore requires this and fails to boot without it. Better MTRR are achieved by making the memory 'hole' till 4GiB exactly 2Gib. This code mimics how it is done in nb/intel/gm45 and achieves similar results. TSEG is enabled and set to 8M since this makes it easier to reuse the common smm setup / parallel mp code and makes it possible to cache the ramstage in there like how it's done on newer targets. TESTED on Intel DG43GT. Change-Id: I1b5ea04d9b7d5494a30aa7156d8c17170e77b8ad Signed-off-by: Arthur Heymans Reviewed-on: https://review.coreboot.org/21634 Tested-by: build bot (Jenkins) Reviewed-by: Nico Huber --- src/mainboard/gigabyte/ga-g41m-es2l/cmos.layout | 1 - src/mainboard/intel/dg43gt/cmos.layout | 1 - 2 files changed, 2 deletions(-) (limited to 'src/mainboard') diff --git a/src/mainboard/gigabyte/ga-g41m-es2l/cmos.layout b/src/mainboard/gigabyte/ga-g41m-es2l/cmos.layout index 15cd29b106..4da0e96cbf 100644 --- a/src/mainboard/gigabyte/ga-g41m-es2l/cmos.layout +++ b/src/mainboard/gigabyte/ga-g41m-es2l/cmos.layout @@ -92,7 +92,6 @@ enumerations 7 0 Disable 7 1 Enable 7 2 Keep -11 0 1M 11 1 4M 11 2 8M 11 3 16M diff --git a/src/mainboard/intel/dg43gt/cmos.layout b/src/mainboard/intel/dg43gt/cmos.layout index 1175cafdc2..abeff71f8b 100644 --- a/src/mainboard/intel/dg43gt/cmos.layout +++ b/src/mainboard/intel/dg43gt/cmos.layout @@ -92,7 +92,6 @@ enumerations 7 2 Keep 10 0 AHCI 10 1 Compatible -11 0 1M 11 1 4M 11 2 8M 11 3 16M -- cgit v1.2.3