[coreboot-gerrit] Patch set updated for coreboot: fda6c95 baytrail: Reserve memory between ASEG and 1MB and for ramoops

Aaron Durbin (adurbin@google.com) gerrit at coreboot.org
Tue Jan 28 05:32:26 CET 2014


Aaron Durbin (adurbin at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/4932

-gerrit

commit fda6c95b1cf1fe5b0e5870f831b3bb1da87cc4fe
Author: Duncan Laurie <dlaurie at chromium.org>
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 <dlaurie at chromium.org>
    Reviewed-on: https://chromium-review.googlesource.com/175554
    Reviewed-by: Aaron Durbin <adurbin at chromium.org>
    Signed-off-by: Aaron Durbin <adurbin at chromium.org>
---
 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 = {



More information about the coreboot-gerrit mailing list