summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSrinidhi N Kaushik <srinidhi.n.kaushik@intel.com>2020-11-25 02:21:57 -0800
committerFurquan Shaikh <furquan@google.com>2020-12-08 22:57:54 +0000
commit4eb489fb0f2d2ca3a1da2ad3ca0f608d503182ee (patch)
treecca4a76fb4fbd55f02ab5cc4afc539bb0aea900d
parent237afda813c87fa65cb2adf2df9e7368d0479034 (diff)
soc/intel/common/fast_spi: Add support for configuring MTRRs
This change enables caching for extended BIOS region. Currently, caching is enabled for the standard BIOS region upto a maximum of 16MiB using fast_spi_cache_bios_region, used the same function to add the support for caching for extended BIOS region as well. Changes include: 1. Add a new helper function fast_spi_cache_ext_bios_window() which calls fast_spi_ext_bios_cache_range() which calls fast_spi_get_ext_bios_window() to get details about the extended BIOS window from the boot media map and checks for allignment and set mtrr. 2. Make a call to fast_spi_cache_ext_bios_region() from fast_spi_cache_bios_region (). 3. Add new helper function fast_spi_cache_ext_bios_postcar() which does caching ext BIOS region in postcar similar to 1. 4. If the extended window is used, then it enables caching for this window similar to how it is done for the standard window. BUG=b:171534504 Signed-off-by: Srinidhi N Kaushik <srinidhi.n.kaushik@intel.com> Change-Id: I9711f110a35a167efe3a4c912cf46c63c0812779 Reviewed-on: https://review.coreboot.org/c/coreboot/+/47991 Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org> Reviewed-by: Furquan Shaikh <furquan@google.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
-rw-r--r--src/soc/intel/common/block/fast_spi/fast_spi.c51
-rw-r--r--src/soc/intel/common/block/include/intelblocks/fast_spi.h5
-rw-r--r--src/soc/intel/common/block/systemagent/memmap.c4
3 files changed, 60 insertions, 0 deletions
diff --git a/src/soc/intel/common/block/fast_spi/fast_spi.c b/src/soc/intel/common/block/fast_spi/fast_spi.c
index 24c667b845..d6e8f53ee4 100644
--- a/src/soc/intel/common/block/fast_spi/fast_spi.c
+++ b/src/soc/intel/common/block/fast_spi/fast_spi.c
@@ -1,5 +1,6 @@
/* SPDX-License-Identifier: GPL-2.0-only */
+#include <arch/romstage.h>
#include <device/mmio.h>
#include <assert.h>
#include <device/pci_def.h>
@@ -212,6 +213,53 @@ size_t fast_spi_get_bios_region(size_t *bios_size)
return bios_start;
}
+static bool fast_spi_ext_bios_cache_range(uintptr_t *base, size_t *size)
+{
+ uint32_t alignment;
+ if (!CONFIG(FAST_SPI_SUPPORTS_EXT_BIOS_WINDOW))
+ return false;
+
+ fast_spi_get_ext_bios_window(base, size);
+
+ /* Enable extended bios only if Size of Bios region is greater than 16MiB */
+ if (*size == 0 || *base == 0)
+ return false;
+
+ /* Round to power of two */
+ alignment = 1UL << (log2_ceil(*size));
+ *size = ALIGN_UP(*size, alignment);
+ *base = ALIGN_DOWN(*base, *size);
+
+ return true;
+}
+
+static void fast_spi_cache_ext_bios_window(void)
+{
+ size_t ext_bios_size;
+ uintptr_t ext_bios_base;
+ const int type = MTRR_TYPE_WRPROT;
+
+ if (!fast_spi_ext_bios_cache_range(&ext_bios_base, &ext_bios_size))
+ return;
+
+ int mtrr = get_free_var_mtrr();
+ if (mtrr == -1)
+ return;
+ set_var_mtrr(mtrr, ext_bios_base, ext_bios_size, type);
+}
+
+void fast_spi_cache_ext_bios_postcar(struct postcar_frame *pcf)
+{
+ size_t ext_bios_size;
+ uintptr_t ext_bios_base;
+ const int type = MTRR_TYPE_WRPROT;
+
+ if (!fast_spi_ext_bios_cache_range(&ext_bios_base, &ext_bios_size))
+ return;
+
+ postcar_frame_add_mtrr(pcf, ext_bios_base, ext_bios_size, type);
+}
+
void fast_spi_cache_bios_region(void)
{
size_t bios_size;
@@ -245,6 +293,9 @@ void fast_spi_cache_bios_region(void)
set_var_mtrr(mtrr, base, bios_size, type);
}
+
+ /* Check if caching is needed for extended bios region if supported */
+ fast_spi_cache_ext_bios_window();
}
/*
diff --git a/src/soc/intel/common/block/include/intelblocks/fast_spi.h b/src/soc/intel/common/block/include/intelblocks/fast_spi.h
index 81437b6e6a..e3ffa6a39a 100644
--- a/src/soc/intel/common/block/include/intelblocks/fast_spi.h
+++ b/src/soc/intel/common/block/include/intelblocks/fast_spi.h
@@ -83,5 +83,10 @@ void fast_spi_get_ext_bios_window(uintptr_t *base, size_t *size);
* SOC function to get SPI-DMI Destination Id
*/
uint32_t soc_get_spi_dmi_destination_id(void);
+/*
+ * Add MTRR for extended BIOS region(when supported) to postcar frame
+ */
+struct postcar_frame;
+void fast_spi_cache_ext_bios_postcar(struct postcar_frame *pcf);
#endif /* SOC_INTEL_COMMON_BLOCK_FAST_SPI_H */
diff --git a/src/soc/intel/common/block/systemagent/memmap.c b/src/soc/intel/common/block/systemagent/memmap.c
index 27870b0cf7..86ca4e1b84 100644
--- a/src/soc/intel/common/block/systemagent/memmap.c
+++ b/src/soc/intel/common/block/systemagent/memmap.c
@@ -5,6 +5,7 @@
#include <console/console.h>
#include <cpu/x86/mtrr.h>
#include <cpu/x86/smm.h>
+#include <intelblocks/fast_spi.h>
#include <intelblocks/systemagent.h>
#include <types.h>
@@ -71,4 +72,7 @@ void fill_postcar_frame(struct postcar_frame *pcf)
/* Cache the TSEG region */
postcar_enable_tseg_cache(pcf);
+
+ /* Cache the extended BIOS region if it is supported */
+ fast_spi_cache_ext_bios_postcar(pcf);
}