[coreboot-gerrit] New patch to review for coreboot: drivers/intel/fsp2_0: Print address of FspMemoryInit entry point

Andrey Petrov (andrey.petrov@intel.com) gerrit at coreboot.org
Sun Jan 24 03:33:45 CET 2016


Andrey Petrov (andrey.petrov at intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/13370

-gerrit

commit a043f99772e777991da986133763ebc8bcde12cb
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