[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