[coreboot-gerrit] Patch set updated for coreboot: FSP 1.0: Fix CAR issues - broken timestamps and console

[Ben Gardner]

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

-gerrit

commit e76c8e47c841486b2600d8100a90a7833a01a139
Author: Ben Gardner 
Date:   Mon Nov 23 20:47:59 2015 -0600

FSP 1.0: Fix CAR issues - broken timestamps and console

FSP 1.0 has a fixed-size temporary cache size and address and the entire
    cache is migrated in the FSP FspInitEntry() function.

Previous code expected the symbol _car_data_start to be the same as
    CONFIG_DCACHE_RAM_BASE and _car_data_end to be the same as
    _preram_cbmem_console.

FSP 1.0 is the only one that migrates _preram_cbmem_console.
    Others leave that where it is and extract the early console data in
    cbmemc_reinit(). Special handling is needed to handle that.

Commit dd6fa93d broke both assumptions and so broke the timestamp table
    and console.

The fix is to use CONFIG_DCACHE_RAM_BASE when calculating the offset and
    to use _preram_cbmem_console instead of _car_data_end for the console
    check.

Change-Id: I6db109269b3537f7cb1300357c483ff2a745ffa7
    Signed-off-by: Ben Gardner 
---
 src/cpu/x86/car.c | 22 ++++++++++++++--------
 1 file changed, 14 insertions(+), 8 deletions(-)

diff --git a/src/cpu/x86/car.c b/src/cpu/x86/car.c
index 36c5cf0..fda3f7d 100644
--- a/src/cpu/x86/car.c
+++ b/src/cpu/x86/car.c
@@ -18,6 +18,7 @@
 #include <console/console.h>
 #include <cbmem.h>
 #include <arch/early_variables.h>
+#include <symbols.h>

#if IS_ENABLED(CONFIG_PLATFORM_USES_FSP1_0)
 #include <drivers/intel/fsp1_0/fsp_util.h>
@@ -63,15 +64,16 @@ void *car_get_var_ptr(void *var)
 #if IS_ENABLED(CONFIG_PLATFORM_USES_FSP1_0)
    migrated_base=(char *)find_saved_temp_mem(
    		*(void **)CBMEM_FSP_HOB_PTR);
+	/* FSP 1.0 migrates the entire DCACHE RAM */
+	offset = (char *)var - (char *)CONFIG_DCACHE_RAM_BASE;
 #else
    migrated_base = cbmem_find(CBMEM_ID_CAR_GLOBALS);
+	offset = (char *)var - (char *)_car_start;
 #endif

if (migrated_base == NULL)
    	die( "CAR: Could not find migration base!\n");

-	offset = (char *)var - (char *)_car_start;
-
    return &migrated_base[offset];
 }

@@ -89,15 +91,19 @@ void *car_sync_var_ptr(void *var)
    if (mig_var == var)
    	return mig_var;

-	/* It's already pointing outside car.global_data. */
-	if (*mig_var < _car_start || *mig_var > _car_end)
+	/*
+	 * Migrate the cbmem console pointer for FSP 1.0 platforms. Otherwise,
+	 * keep console buffer in CAR until cbmemc_reinit() moves it.
+	 */
+	if (*mig_var == _preram_cbmem_console) {
+		if (IS_ENABLED(CONFIG_PLATFORM_USES_FSP1_0))
+			*mig_var += (char *)mig_var - (char *)var;
    	return mig_var;
+	}

-#if !IS_ENABLED(CONFIG_PLATFORM_USES_FSP1_0)
-	/* Keep console buffer in CAR until cbmemc_reinit() moves it. */
-	if (*mig_var == _car_end)
+	/* It's already pointing outside car.global_data. */
+	if (*mig_var < _car_start || *mig_var > _car_end)
    	return mig_var;
-#endif

/* Move the pointer by the same amount the variable storing it was
     * moved by.