York Yang (york.yang@intel.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/9976
-gerrit
commit 7608ec80ee94d290dc686425251c5acc6dbc9d8f Author: York Yang york.yang@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@intel.com --- src/drivers/intel/fsp1_0/fastboot_cache.c | 2 +- src/drivers/intel/fsp1_0/fsp_util.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/drivers/intel/fsp1_0/fastboot_cache.c b/src/drivers/intel/fsp1_0/fastboot_cache.c old mode 100644 new mode 100755 index bcf39ab..e8651bb --- a/src/drivers/intel/fsp1_0/fastboot_cache.c +++ b/src/drivers/intel/fsp1_0/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/fsp1_0/fsp_util.c b/src/drivers/intel/fsp1_0/fsp_util.c old mode 100644 new mode 100755 index a00cb81..ab67147 --- a/src/drivers/intel/fsp1_0/fsp_util.c +++ b/src/drivers/intel/fsp1_0/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 */