diff options
| author | Paul Menzel <paulepanter@users.sourceforge.net> | 2013-03-31 22:02:16 +0200 |
|---|---|---|
| committer | Anton Kochkov <anton.kochkov@gmail.com> | 2013-04-03 11:13:29 +0200 |
| commit | a8db717d4af799fabd26383e6a748de94318d280 (patch) | |
| tree | 4697a596c06b50cd478757316fb367c707caed15 | |
| parent | aa3f7ba36ebe3a933aa664f826382f60b31e86f1 (diff) | |
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@users.sourceforge.net>
Reviewed-on: http://review.coreboot.org/2994
Reviewed-by: Anton Kochkov <anton.kochkov@gmail.com>
Tested-by: build bot (Jenkins)
| -rw-r--r-- | util/inteltool/memory.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/util/inteltool/memory.c b/util/inteltool/memory.c index 72a2de477c..6847e5c547 100644 --- a/util/inteltool/memory.c +++ b/util/inteltool/memory.c @@ -197,7 +197,7 @@ int print_mchbar(struct pci_dev *nb, struct pci_access *pacc) for (i = 0; i < size; i++) { switch (mch_registers[i].size) { case 8: - printf("mchbase+0x%04x: 0x%016lx (%s)\n", + printf("mchbase+0x%04x: 0x%016llx (%s)\n", mch_registers[i].addr, *(uint64_t *)(mchbar+mch_registers[i].addr), mch_registers[i].name); |