[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