From 34473ea6c9ee63de46b04b46cc47ef4aa5bae2b7 Mon Sep 17 00:00:00 2001
From: Johnny Lin <johnny_lin@wiwynn.com>
Date: Wed, 18 Mar 2020 10:23:26 +0800
Subject: [PATCH] soc/intel/xeon_sp: Modify FSP-T code caching parameters

Use CACHE_ROM_BASE and CACHE_ROM_SIZE for code caching
parameters.

Tested on OCP Tioga Pass.

Change-Id: Ibba133d9f8fdfbdfae9a0e8e698356a3ca9ba424
Signed-off-by: Johnny Lin <johnny_lin@wiwynn.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/39625
Reviewed-by: Nico Huber <nico.h@gmx.de>
Reviewed-by: Andrey Petrov <anpetrov@fb.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
---
 src/soc/intel/xeon_sp/bootblock/bootblock.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/soc/intel/xeon_sp/bootblock/bootblock.c b/src/soc/intel/xeon_sp/bootblock/bootblock.c
index 6b2c48809d..dc88adc308 100644
--- a/src/soc/intel/xeon_sp/bootblock/bootblock.c
+++ b/src/soc/intel/xeon_sp/bootblock/bootblock.c
@@ -19,6 +19,7 @@
 #include <intelblocks/fast_spi.h>
 #include <soc/iomap.h>
 #include <console/console.h>
+#include <cpu/x86/mtrr.h>
 
 const FSPT_UPD temp_ram_init_params = {
 	.FspUpdHeader = {
@@ -29,8 +30,8 @@ const FSPT_UPD temp_ram_init_params = {
 	.FsptCoreUpd = {
 		.MicrocodeRegionBase = (UINT32)CONFIG_CPU_MICROCODE_CBFS_LOC,
 		.MicrocodeRegionLength = (UINT32)CONFIG_CPU_MICROCODE_CBFS_LEN,
-		.CodeRegionBase = (uint32_t)(0x100000000ULL - CONFIG_ROM_SIZE),
-		.CodeRegionLength = (UINT32)CONFIG_ROM_SIZE,
+		.CodeRegionBase = (UINT32)CACHE_ROM_BASE,
+		.CodeRegionLength = (UINT32)CACHE_ROM_SIZE,
 		.Reserved1 = {0},
 	},
 	.FsptConfig = {