Patch set updated for coreboot: drivers/intel/fsp2_0: Monitor FSP setting of MTRRs

[Lee Leahy]

Lee Leahy ([e-mail]) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/15849

-gerrit

commit 5cf6f630026338f8756d5c7dade7826b4df08e33
Author: Lee Leahy 
Date:   Sun Jul 24 08:26:06 2016 -0700

drivers/intel/fsp2_0: Monitor FSP setting of MTRRs

Display the MTRR values in the following locations:
    * Before the call to FspMemoryInit to document coreboot settings
    * After the call to FspMemoryInit
    * After the call to FspSiliconInit
    * After the call to FspNotify

TEST=Build and run on Galileo Gen2

Change-Id: I8942ef4ca4677501a5c38abaff1c3489eebea53c
    Signed-off-by: Lee Leahy 
---
 src/drivers/intel/fsp2_0/debug.c | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/src/drivers/intel/fsp2_0/debug.c b/src/drivers/intel/fsp2_0/debug.c
index 02e9487..6e28939 100644
--- a/src/drivers/intel/fsp2_0/debug.c
+++ b/src/drivers/intel/fsp2_0/debug.c
@@ -11,6 +11,7 @@

#include <console/console.h>
 #include <fsp/util.h>
+#include <soc/intel/common/util.h>

/*-----------
  * MemoryInit
@@ -20,6 +21,10 @@ void fsp_debug_before_memory_init(fsp_memory_init_fn memory_init,
    const struct FSPM_UPD *fspm_old_upd,
    const struct FSPM_UPD *fspm_new_upd, void **hob_list_ptr)
 {
+	/* Display the MTRRs */
+	if (IS_ENABLED(CONFIG_DISPLAY_MTRRS))
+		soc_display_mtrrs();
+
    /* Display the call entry point and paramters */
    if (IS_ENABLED(CONFIG_DISPLAY_FSP_CALLS_AND_STATUS)) {
    	printk(BIOS_DEBUG, "Calling FspMemoryInit: 0x%p\n",
@@ -34,6 +39,10 @@ void fsp_debug_memory_init(enum fsp_status status,
 {
    if (IS_ENABLED(CONFIG_DISPLAY_FSP_CALLS_AND_STATUS))
    	printk(BIOS_DEBUG, "FspMemoryInit returned 0x%08x\n", status);
+
+	/* Display the MTRRs */
+	if (IS_ENABLED(CONFIG_DISPLAY_MTRRS))
+		soc_display_mtrrs();
 }

/*-----------
@@ -55,6 +64,10 @@ void fsp_debug_silicon_init(enum fsp_status status)
 {
    if (IS_ENABLED(CONFIG_DISPLAY_FSP_CALLS_AND_STATUS))
    	printk(BIOS_SPEW, "FspSiliconInit returned 0x%08x\n", status);
+
+	/* Display the MTRRs */
+	if (IS_ENABLED(CONFIG_DISPLAY_MTRRS))
+		soc_display_mtrrs();
 }

/*-----------
@@ -77,4 +90,8 @@ void fsp_debug_notify(enum fsp_status status)
 {
    if (IS_ENABLED(CONFIG_DISPLAY_FSP_CALLS_AND_STATUS))
    	printk(BIOS_SPEW, "FspNotify returned 0x%08x\n", status);
+
+	/* Display the MTRRs */
+	if (IS_ENABLED(CONFIG_DISPLAY_MTRRS))
+		soc_display_mtrrs();
 }