[coreboot-gerrit] Patch merged into coreboot/master: cb891de cbmem: parse_cbtable: Use length modifier `ll` `u64` argument
gerrit at coreboot.org
gerrit at coreboot.org
Sun Apr 14 02:06:22 CEST 2013
the following patch was just integrated into master:
commit cb891de07ffe605897010776fc1becc9589d3648
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date: Sat Apr 13 18:35:32 2013 +0200
cbmem: parse_cbtable: Use length modifier `ll` `u64` argument
Currently on a 32-bit system cbmem fails to build due to `-Werror`
and the following warning.
$ make
cc -O2 -Wall -Werror -iquote ../../src/include -iquote ../../src/src/arch/x86 -c -o cbmem.o cbmem.c
[…]
cbmem.c: In function ‘parse_cbtable’:
cbmem.c:135:2: error: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 2 has type ‘u64’ [-Werror=format]
cc1: all warnings being treated as errors
[…]
Using the length modifier `ll` instead of `l` gets rid of this
warning.
Change-Id: Ib2656e27594c7aaa687aa84bf07042933f840e46
Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
Reviewed-on: http://review.coreboot.org/3084
Tested-by: build bot (Jenkins)
Reviewed-by: Ronald G. Minnich <rminnich at gmail.com>
Build-Tested: build bot (Jenkins) at Sat Apr 13 19:48:39 2013, giving +1
Reviewed-By: Ronald G. Minnich <rminnich at gmail.com> at Sun Apr 14 02:06:21 2013, giving +2
See http://review.coreboot.org/3084 for details.
-gerrit
More information about the coreboot-gerrit
mailing list