[coreboot-gerrit] New patch to review for coreboot: 274d78a driver/intel/fsp: Correct the fastboot data (MRC data) printing length

York Yang (york.yang@intel.com) gerrit at coreboot.org
Thu Apr 23 22:05:19 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 274d78a557d139bd8694a841ed30fa0e9030931d
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 is printed by hexdump32() in dword length. So the data
    length need to be divided by 4 when print.
    
    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..5565ae4
--- 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..0bf5408
--- 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