[coreboot-gerrit] Patch set updated for coreboot: bf8a4aa util/cbmem: Fix format string in cbmem.c

Nico Huber (nico.huber@secunet.com) gerrit at coreboot.org
Mon Jun 3 21:37:23 CEST 2013

Nico Huber (nico.huber at secunet.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3301


commit bf8a4aa59118a14720ecacc475e0a54ce048f79a
Author: Nico Huber <nico.huber at secunet.com>
Date:   Sun May 26 18:17:54 2013 +0200

    util/cbmem: Fix format string in cbmem.c
    Use PRIx64 to print a u64 instead of "llx". Fixes the following error:
    cbmem.c: In function 'parse_cbtable':
    cbmem.c:135:2: error: format '%llx' expects argument of type 'long long unsigned int', but argument 2 has type 'u64' [-Werror=format=]
    Change-Id: Ibc2bf8597cb86db5b2e71fba77ec837a08c5e3d4
    Signed-off-by: Nico Huber <nico.huber at secunet.com>
 util/cbmem/cbmem.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/util/cbmem/cbmem.c b/util/cbmem/cbmem.c
index 1ff9a08..74e0fd4 100644
--- a/util/cbmem/cbmem.c
+++ b/util/cbmem/cbmem.c
@@ -17,6 +17,7 @@
  * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
+#include <inttypes.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
@@ -132,7 +133,7 @@ static int parse_cbtable(u64 address)
 	int i, found = 0;
 	void *buf;
-	debug("Looking for coreboot table at %llx\n", address);
+	debug("Looking for coreboot table at %" PRIx64 "\n", address);
 	buf = map_memory(address);
 	/* look at every 16 bytes within 4K of the base */

More information about the coreboot-gerrit mailing list