commit 72240430cf26fd269e89a730f70eab582153d445
author Raul E Rangel <rrangel@chromium.org> Mon Jul 12 09:50:49 2021 -0600
committer Raul Rangel <rrangel@chromium.org> Wed Jul 14 17:53:42 2021 +0000
tree e390869ac02c7e153c7dfdf6f255016a3ba8bd20
parent a24472a7f88ecfea999e4962931e14f2b319b47a

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
Reviewed-on: https://review.coreboot.org/c/coreboot/+/56225
Reviewed-by: Karthik Ramasubramanian <kramasub@google.com>
Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>

src/soc/amd/common/block/apob/apob_cache.c

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(&region), region_sz(&region));