diff options
| author | Paul Menzel <paulepanter@users.sourceforge.net> | 2013-04-14 13:00:22 +0200 |
|---|---|---|
| committer | Ronald G. Minnich <rminnich@gmail.com> | 2013-04-14 23:00:35 +0200 |
| commit | 15a1fd1db9dd93004f808badcb15ca635177def6 (patch) | |
| tree | 9391ec7d6668e35364e034c0d720e3fdf0f22354 /util/inteltool | |
| parent | 0b3128679688b76f8fc92a770615270707a80afd (diff) | |
| download | coreboot-15a1fd1db9dd93004f808badcb15ca635177def6.tar.xz | |
inteltool: Use portable type `uint64_t` instead of `u64`
In [1] Idwer Vollering noted, that the type `u64` is not portable so
on his FreeBSD system, the following warning is shown.
$ clang -O2 -Wall -W -I/usr/local/include -c -o amb.o amb.c
amb.c:441:22: error: use of undeclared identifier 'u64'
ambconfig_phys = ((u64)pci_read_long(dev16, 0x4c) << 32) |
The type `uint64_t` seems to be defined also on FreeBSD, so using this
fixes the warning.
Note, this warning is not reproducable with Debian Sid/unstable for
example. I have no idea why though.
[1] http://review.coreboot.org/#/c/3015/
Change-Id: Ic22f4371114b68ae8221d84a01fef6888d43f365
Signed-off-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-on: http://review.coreboot.org/3086
Tested-by: build bot (Jenkins)
Reviewed-by: Ronald G. Minnich <rminnich@gmail.com>
Diffstat (limited to 'util/inteltool')
| -rw-r--r-- | util/inteltool/amb.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/util/inteltool/amb.c b/util/inteltool/amb.c index a1761ee501..13397dbf5e 100644 --- a/util/inteltool/amb.c +++ b/util/inteltool/amb.c @@ -438,7 +438,7 @@ int print_ambs(struct pci_dev *dev, struct pci_access *pacc) return 1; } - ambconfig_phys = ((u64)pci_read_long(dev16, 0x4c) << 32) | + ambconfig_phys = ((uint64_t)pci_read_long(dev16, 0x4c) << 32) | pci_read_long(dev16, 0x48); max_channel = pci_read_byte(dev16, 0x56)/max_branch; |