Łukasz Dobrowolski (lukasz@dobrowolski.io) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17118
-gerrit
commit ef5754f5dc3be36c830b39a7ced4f0b3bd0a2c18 Author: Łukasz Dobrowolski lukasz@dobrowolski.io Date: Tue Oct 25 00:29:37 2016 +0200
vendorcode/amd/f14: Fix ignored argument in IDS_HDT_CONSOLE
String format required two arguments however those were packaged in ( , ) so the left one was ignored.
Change-Id: I59698319d5ff4215f296356147b4e22229cc9245 Signed-off-by: Łukasz Dobrowolski lukasz@dobrowolski.io --- src/vendorcode/amd/agesa/f14/Proc/Common/S3SaveState.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/vendorcode/amd/agesa/f14/Proc/Common/S3SaveState.c b/src/vendorcode/amd/agesa/f14/Proc/Common/S3SaveState.c index b7aaf22..62af8cc 100644 --- a/src/vendorcode/amd/agesa/f14/Proc/Common/S3SaveState.c +++ b/src/vendorcode/amd/agesa/f14/Proc/Common/S3SaveState.c @@ -641,7 +641,9 @@ S3SaveDebugPrintHexArray ( break; case AccessWidth64: case AccessS3SaveWidth64: - IDS_HDT_CONSOLE (S3_TRACE, "0x%08x%08x", ((UINT32*) ((UINT64*)Array + Index)[1], ((UINT32*) ((UINT64*)Array + Index))[0])); + IDS_HDT_CONSOLE(S3_TRACE, "0x%08x%08x", + (UINT32 *) ((UINT64 *)Array + Index)[1], + ((UINT32 *) ((UINT64 *)Array + Index))[0]); break; default: IDS_ERROR_TRAP;