diff options
| author | David Gibson <david@gibson.dropbear.id.au> | 2008-07-07 10:10:48 +1000 |
|---|---|---|
| committer | Gerald Van Baren <vanbaren@cideas.com> | 2008-08-24 22:20:49 -0400 |
| commit | ef4e8ce1beb5b93aedda5a4c1b90bfd989c6791e (patch) | |
| tree | 4f975651b587a12ae9914bd2b9e558db88aea808 /libfdt/fdt_sw.c | |
| parent | afe3848b79a7ff351e9fbf3a7c162d2de002279b (diff) | |
| download | u-boot-imx-ef4e8ce1beb5b93aedda5a4c1b90bfd989c6791e.zip u-boot-imx-ef4e8ce1beb5b93aedda5a4c1b90bfd989c6791e.tar.gz u-boot-imx-ef4e8ce1beb5b93aedda5a4c1b90bfd989c6791e.tar.bz2 | |
dtc: Enable and fix -Wpointer-arith warnings
This patch turns on the -Wpointer-arith option in the dtc Makefile,
and fixes the resulting warnings due to using (void *) in pointer
arithmetic. While convenient, pointer arithmetic on void * is not
portable, so it's better that we avoid it, particularly in libfdt.
Also add necessary definition of uintptr_t needed by David Gibson's
changeset "dtc: Enable and fix -Wpointer-arith warnings" (the definition
comes from stdint.h, which u-boot doesn't have). -- gvb
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Signed-off-by: Gerald Van Baren <vanbaren@cideas.com>
Diffstat (limited to 'libfdt/fdt_sw.c')
| -rw-r--r-- | libfdt/fdt_sw.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libfdt/fdt_sw.c b/libfdt/fdt_sw.c index df09876..92f8f0b 100644 --- a/libfdt/fdt_sw.c +++ b/libfdt/fdt_sw.c @@ -121,7 +121,7 @@ int fdt_add_reservemap_entry(void *fdt, uint64_t addr, uint64_t size) if ((offset + sizeof(*re)) > fdt_totalsize(fdt)) return -FDT_ERR_NOSPACE; - re = (struct fdt_reserve_entry *)(fdt + offset); + re = (struct fdt_reserve_entry *)((char *)fdt + offset); re->address = cpu_to_fdt64(addr); re->size = cpu_to_fdt64(size); |