[coreboot-gerrit] Change in coreboot[master]: lib/spd_bin.c: Drop CAR_GLOBAL_MIGRATION support

[Patrick Georgi (Code Review)]

Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/37031 )

Change subject: lib/spd_bin.c: Drop CAR_GLOBAL_MIGRATION support
......................................................................

lib/spd_bin.c: Drop CAR_GLOBAL_MIGRATION support

Change-Id: I1c307e1d5532929de6d876ce9215515ab1cf4652
Signed-off-by: Arthur Heymans 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/37031
Reviewed-by: HAOUAS Elyes 
Reviewed-by: Kyösti Mälkki 
Tested-by: build bot (Jenkins) 
---
M src/include/spd_bin.h
M src/lib/spd_bin.c
2 files changed, 3 insertions(+), 6 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Kyösti Mälkki: Looks good to me, approved
  HAOUAS Elyes: Looks good to me, but someone else must approve

diff --git a/src/include/spd_bin.h b/src/include/spd_bin.h
index 3de60a9..f144b14 100644
--- a/src/include/spd_bin.h
+++ b/src/include/spd_bin.h
@@ -16,7 +16,6 @@
 #ifndef SPD_BIN_H
 #define SPD_BIN_H

-#include <arch/early_variables.h>
 #include <stdint.h>
 #include <commonlib/region.h>

diff --git a/src/lib/spd_bin.c b/src/lib/spd_bin.c
index 62e600b..71fe036 100644
--- a/src/lib/spd_bin.c
+++ b/src/lib/spd_bin.c
@@ -20,7 +20,7 @@
 #include <device/early_smbus.h>
 #include <device/dram/ddr3.h>

-static u8 spd_data[CONFIG_DIMM_MAX * CONFIG_DIMM_SPD_SIZE] CAR_GLOBAL;
+static u8 spd_data[CONFIG_DIMM_MAX * CONFIG_DIMM_SPD_SIZE];

void dump_spd_info(struct spd_block *blk)
 {
@@ -258,12 +258,10 @@
 void get_spd_smbus(struct spd_block *blk)
 {
    u8 i;
-	unsigned char *spd_data_ptr = car_get_var_ptr(&spd_data);
-
    for (i = 0 ; i < CONFIG_DIMM_MAX; i++) {
-		get_spd(spd_data_ptr + i * CONFIG_DIMM_SPD_SIZE,
+		get_spd(&spd_data[i * CONFIG_DIMM_SPD_SIZE],
    		blk->addr_map[i]);
-		blk->spd_array[i] = spd_data_ptr + i * CONFIG_DIMM_SPD_SIZE;
+		blk->spd_array[i] = &spd_data[i * CONFIG_DIMM_SPD_SIZE];
    }

update_spd_len(blk);