[coreboot-gerrit] Patch set updated for coreboot: 55274a0 driver/intel/fsp: Correct the fastboot data (MRC data) printing length

York Yang (york.yang@intel.com) gerrit at coreboot.org
Fri Apr 24 00:31:17 CEST 2015


York Yang (york.yang at intel.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/9976

-gerrit

commit 55274a0425a8a46573a6073759bb93d43d1999a9
Author: York Yang <york.yang at intel.com>
Date:   Thu Apr 23 13:00:20 2015 -0700

    driver/intel/fsp: Correct the fastboot data (MRC data) printing length
    
    Fastboot data in Intel FSP project is printed by hexdump32() in dword
    length. So the data length needs to be divided by 4 when printing it.
    
    Change-Id: I959d538bd6e60282882dd138045cc730b4bd8159
    Signed-off-by: York Yang <york.yang at intel.com>
---
 src/drivers/intel/fsp/fastboot_cache.c | 2 +-
 src/drivers/intel/fsp/fsp_util.c       | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/drivers/intel/fsp/fastboot_cache.c b/src/drivers/intel/fsp/fastboot_cache.c
old mode 100644
new mode 100755
index bcf39ab..e8651bb
--- a/src/drivers/intel/fsp/fastboot_cache.c
+++ b/src/drivers/intel/fsp/fastboot_cache.c
@@ -230,7 +230,7 @@ void * find_and_set_fastboot_cache(void)
 	}
 	printk(BIOS_DEBUG, "FSP MRC cache present at %x.\n", (u32)mrc_cache);
 	printk(BIOS_SPEW, "Saved MRC data:\n");
-	hexdump32(BIOS_SPEW, (void *)mrc_cache->mrc_data, mrc_cache->mrc_data_size);
+	hexdump32(BIOS_SPEW, (void *)mrc_cache->mrc_data, (mrc_cache->mrc_data_size) / 4);
 	return (void *) mrc_cache->mrc_data;
 }
 
diff --git a/src/drivers/intel/fsp/fsp_util.c b/src/drivers/intel/fsp/fsp_util.c
old mode 100644
new mode 100755
index a00cb81..ab67147
--- a/src/drivers/intel/fsp/fsp_util.c
+++ b/src/drivers/intel/fsp/fsp_util.c
@@ -292,7 +292,7 @@ int save_mrc_data(void *hob_start)
 			mrc_data->mrc_data_size);
 
 	printk(BIOS_SPEW, "Fast boot data (includes align and checksum):\n");
-	hexdump32(BIOS_SPEW, (void *)mrc_data->mrc_data, output_len);
+	hexdump32(BIOS_SPEW, (void *)mrc_data->mrc_data, output_len / 4);
 	return (1);
 }
 #endif /* CONFIG_ENABLE_MRC_CACHE */



More information about the coreboot-gerrit mailing list