[coreboot-gerrit] Patch set updated for coreboot: drivers/intel/fsp2_0: Print address of FspMemoryInit entry point

Alexandru Gagniuc (mr.nuke.me@gmail.com) gerrit at coreboot.org
Mon Jan 25 18:02:27 CET 2016


Alexandru Gagniuc (mr.nuke.me at gmail.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/13370

-gerrit

commit 5e163c89da174167edac6a090bc8e62c4b504e21
Author: Alexandru Gagniuc <alexandrux.gagniuc at intel.com>
Date:   Tue Dec 1 16:41:02 2015 -0800

    drivers/intel/fsp2_0: Print address of FspMemoryInit entry point
    
    Through an oversight, the address of fsp_memory_init was printed,
    which is a coreboot function. the intention is to print the address
    of the fsp_raminit pointer, which is the entry point into FSP memory
    init.
    
    Change-Id: I95904279fc148319033281d45b681ad427e03130
    Signed-off-by: Alexandru Gagniuc <alexandrux.gagniuc at intel.com>
---
 src/drivers/intel/fsp2_0/memory_init.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/drivers/intel/fsp2_0/memory_init.c b/src/drivers/intel/fsp2_0/memory_init.c
index 4ac720d..e9d6e42 100644
--- a/src/drivers/intel/fsp2_0/memory_init.c
+++ b/src/drivers/intel/fsp2_0/memory_init.c
@@ -88,7 +88,7 @@ static enum fsp_status do_fsp_memory_init(void **hob_list_ptr,
 
 	/* Call FspMemoryInit */
 	fsp_raminit = (void *)(hdr->image_base + hdr->memory_init_entry_offset);
-	printk(BIOS_DEBUG, "Calling FspMemoryInit: 0x%p\n", fsp_memory_init);
+	printk(BIOS_DEBUG, "Calling FspMemoryInit: 0x%p\n", fsp_raminit);
 	printk(BIOS_SPEW, "\t%p: nvs_buffer\n", raminit_params.nvs_buffer);
 	printk(BIOS_SPEW, "\t%p: rt_buffer\n", raminit_params.rt_buffer);
 	printk(BIOS_SPEW, "\t%p: hob_list\n", raminit_params.hob_list);



More information about the coreboot-gerrit mailing list