[coreboot-gerrit] Patch set updated for coreboot: drivers/intel/fsp2_0: Monitor FSP setting of MTRRs

[Lee Leahy (leroy.p.leahy@intel.com)]

Lee Leahy (leroy.p.leahy at intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/15849

-gerrit

commit 613aadb651191920792e5fa866611888b8a7a4fb
Author: Lee Leahy <leroy.p.leahy at intel.com>
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 <leroy.p.leahy at intel.com>
---
 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 31fba20..68bf115 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))
 		return;
@@ -33,6 +38,10 @@ void fsp_debug_after_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();
 }
 
 /*-----------
@@ -54,6 +63,10 @@ void fsp_debug_after_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();
 }
 
 /*-----------
@@ -76,4 +89,8 @@ void fsp_debug_after_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();
 }