[coreboot-gerrit] Patch merged into coreboot/master: a8db717 inteltool: Use `ll` instead of `l` as the length modifier for `uint64_t`
gerrit at coreboot.org
gerrit at coreboot.org
Wed Apr 3 11:13:33 CEST 2013
the following patch was just integrated into master:
commit a8db717d4af799fabd26383e6a748de94318d280
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date: Sun Mar 31 22:02:16 2013 +0200
inteltool: Use `ll` instead of `l` as the length modifier for `uint64_t`
When buidling inteltool with GCC, the following warning is printed.
$ make
[…]
gcc -O2 -g -Wall -W -c -o memory.o memory.c
memory.c: In function ‘print_mchbar’:
memory.c:287:7: warning: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 3 has type ‘uint64_t’ [-Wformat]
[…]
This was introduced in commit »inteltool: Add support for H65 Express
chipset« (c7fc4422) [1].
Address this warning, by using `%llx` instead of `%lx`.
[1] http://review.coreboot.org/1258
Change-Id: I4f714edce7e8b405e1a7a417d02fa498322c88a8
Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
Reviewed-on: http://review.coreboot.org/2994
Reviewed-by: Anton Kochkov <anton.kochkov at gmail.com>
Tested-by: build bot (Jenkins)
Build-Tested: build bot (Jenkins) at Tue Apr 2 01:22:51 2013, giving +1
Reviewed-By: Anton Kochkov <anton.kochkov at gmail.com> at Sun Mar 31 22:38:33 2013, giving +2
See http://review.coreboot.org/2994 for details.
-gerrit
More information about the coreboot-gerrit
mailing list