New patch to review for coreboot: 6c4706c baytrail: Reserve memory between ASEG and 1MB and for ramoops

[Aaron Durbin]

Aaron Durbin ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/4932

-gerrit

commit 6c4706c6f52882bba01d8969225f8e27305f31eb
Author: Duncan Laurie 
Date:   Sun Nov 3 19:38:12 2013 -0800

baytrail: Reserve memory between ASEG and 1MB and for ramoops

Low system tables are in this region, and it is probably safer
    to keep ASEG reserved.

Also keep the region used by ramoops from being used by the OS
    and from being cleared by developer mode boots.

Lots more work needed to make the ACPI tables fully functional.

BUG=chrome-os-partner:23505
    BRANCH=rambi
    TEST=boot on rambi and see that the kernel finds RSDP and uses ACPI

Change-Id: I4f7064d3cff14a3ecf15b194a1f20c1fa9d5e134
    Signed-off-by: Duncan Laurie 
    Reviewed-on: https://chromium-review.googlesource.com/175554
    Reviewed-by: Aaron Durbin 
    Signed-off-by: Aaron Durbin 
---
 src/soc/intel/baytrail/northcluster.c | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/src/soc/intel/baytrail/northcluster.c b/src/soc/intel/baytrail/northcluster.c
index 714d532..eca122c 100644
--- a/src/soc/intel/baytrail/northcluster.c
+++ b/src/soc/intel/baytrail/northcluster.c
@@ -117,6 +117,20 @@ static void nc_read_resources(device_t dev)
    if (bmbound_hi > four_gig_kib)
    	ram_resource(dev, index++, four_gig_kib,
    	             bmbound_hi - four_gig_kib);
+
+	/* Reserve everything between A segment and 1MB:
+	 *
+	 * 0xa0000 - 0xbffff: legacy VGA
+	 * 0xc0000 - 0xfffff: RAM
+	 */
+	mmio_resource(dev, index++, (0xa0000 >> 10), (0xc0000 - 0xa0000) >> 10);
+	reserved_ram_resource(dev, index++, (0xc0000 >> 10),
+	                      (0x100000 - 0xc0000) >> 10);
+#if CONFIG_CHROMEOS_RAMOOPS
+	reserved_ram_resource(dev, index++,
+			CONFIG_CHROMEOS_RAMOOPS_RAM_START >> 10,
+			CONFIG_CHROMEOS_RAMOOPS_RAM_SIZE >> 10);
+#endif
 }

static struct device_operations nc_ops = {