[coreboot-gerrit] New patch to review for coreboot: 145ee96 util/cbmem: Fix format string in cbmem.c

Nico Huber (nico.huber@secunet.com) gerrit at coreboot.org
Sun May 26 18:22:03 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

-gerrit

commit 145ee965dc7c49315126493afb5f57ccdd4940ae
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 Funktion »parse_cbtable«:
    cbmem.c:135:2: Fehler: Format »%llx« erwartet Argumenttyp »long long unsigned int«, aber Argument 2 hat Typ »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