erebion/coreboot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Cast addresses and offsets to long long to work on all platforms.

Browse source 
Right now, these offsets are not declared with the correct printf
specifiers. Fix that.

BUG=none
TEST=launch trybots on all platforms

Change-Id: Ic6ac0afe047ab7b61b5753cc5bab261d1a860ba6
Reviewed-on: https://gerrit.chromium.org/gerrit/48321
Reviewed-by: David James <davidjames@chromium.org>
Tested-by: David James <davidjames@chromium.org>
This commit is contained in:
David James 2013-04-16 17:11:32 -07:00
commit 6ba8deb56d
1 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
util/cbmem/cbmem.c
View file

@ -24,6 +24,7 @@
#include <getopt.h>
#include <errno.h>
#include <fcntl.h>
#include <inttypes.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <sys/mman.h>
@ -84,7 +85,7 @@ static void *map_memory(u64 physical)
/* Mapped memory must be aligned to page size */
p = physical & ~(page - 1);
debug("Mapping 1MB of physical memory at 0x%zx.\n", p);
debug("Mapping 1MB of physical memory at 0x%llx.\n", (unsigned long long) p);
v = mmap(NULL, MAP_BYTES, PROT_READ, MAP_SHARED, fd, p);
@ -132,7 +133,7 @@ static int parse_cbtable(u64 address)
int i, found = 0;
void *buf;
debug("Looking for coreboot table at %lx\n", address);
debug("Looking for coreboot table at %" PRIu64 "\n", address);
buf = map_memory(address);
/* look at every 16 bytes within 4K of the base */