[coreboot-gerrit] New patch to review for coreboot: e876b16 Enable CAR migration on Exynos 5250 and 5420

[Stefan Reinauer (stefan.reinauer@coreboot.org)]

Stefan Reinauer (stefan.reinauer at coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/4394

-gerrit

commit e876b16cafe58f97594e9da763f013ae10a2daee
Author: Stefan Reinauer <reinauer at chromium.org>
Date:   Tue Jul 16 11:03:27 2013 -0700

    Enable CAR migration on Exynos 5250 and 5420
    
    Despite calling romstage memory CAR in this case, the variables actually
    do live in SRAM on the Exynos CPUs. However, in order to share as much
    generic code as possible, we're using the same infrastructure here.
    
    Change-Id: I85173c37099a25f3e55980e88120401826cdf29c
    Signed-off-by: Stefan Reinauer <reinauer at google.com>
    Reviewed-on: https://gerrit.chromium.org/gerrit/62188
    Reviewed-by: David Hendricks <dhendrix at chromium.org>
    Commit-Queue: Stefan Reinauer <reinauer at chromium.org>
    Tested-by: Stefan Reinauer <reinauer at chromium.org>
---
 src/arch/armv7/romstage.ld | 20 +++++++++++++++++++-
 1 file changed, 19 insertions(+), 1 deletion(-)

diff --git a/src/arch/armv7/romstage.ld b/src/arch/armv7/romstage.ld
index 0555fc4..146ba77 100644
--- a/src/arch/armv7/romstage.ld
+++ b/src/arch/armv7/romstage.ld
@@ -52,10 +52,28 @@ SECTIONS
 		*(.rodata);
 		*(.machine_param);
 		*(.data);
-		. = ALIGN(8);
+		. = ALIGN(16);
+		_car_migrate_start = .;
+		*(.car.migrate);
+		_car_migrate_end = .;
+		. = ALIGN(16);
 		_erom = .;
 	}
 
+	.car.data . (NOLOAD) : {
+		_car_data_start = .;
+		*(.car.global_data);
+		/* The cbmem_console section comes last to take advantage of
+		 * a zero-sized array to hold the memconsole contents that
+		 * grows to a bound of CONFIG_CONSOLE_CAR_BUFFER_SIZE. However,
+		 * collisions within the cache-as-ram region cannot be
+		 * statically checked because the cache-as-ram region usage is
+		 * cpu/chipset dependent. */
+		*(.car.cbmem_console);
+		_car_data_end = .;
+	}
+
+
 	__image_copy_end = .;
 
 	/* bss does not contain data, it is just a space that should be zero