Raul Rangel has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/56225 )
Change subject: soc/amd/common/block/apob: Fix incorrect printf format ......................................................................
soc/amd/common/block/apob: Fix incorrect printf format
The %p format specifier already prints out 0x, so remove the 0x from the string. I also updated the other format specifiers to use the %# syntax to print out the 0x.
BUG=b:179699789 TEST=see correct format.
Signed-off-by: Raul E Rangel rrangel@chromium.org Change-Id: I5b00d2c06687e549f69486eb5e18f7bed560b2ec --- M src/soc/amd/common/block/apob/apob_cache.c 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/25/56225/1
diff --git a/src/soc/amd/common/block/apob/apob_cache.c b/src/soc/amd/common/block/apob/apob_cache.c index 238ae7a..34d5472 100644 --- a/src/soc/amd/common/block/apob/apob_cache.c +++ b/src/soc/amd/common/block/apob/apob_cache.c @@ -131,7 +131,7 @@ return; }
- printk(BIOS_SPEW, "Copy APOB from RAM 0x%p/0x%x to flash 0x%zx/0x%zx\n", + printk(BIOS_SPEW, "Copy APOB from RAM %p/%#x to flash %#zx/%#zx\n", apob_src_ram, apob_src_ram->size, region_offset(®ion), region_sz(®ion));