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_wip.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_wip.c')
-rw-r--r-- | libfdt/fdt_wip.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libfdt/fdt_wip.c b/libfdt/fdt_wip.c index 24e1724..b336113 100644 --- a/libfdt/fdt_wip.c +++ b/libfdt/fdt_wip.c @@ -80,7 +80,7 @@ static void nop_region(void *start, int len) { uint32_t *p; - for (p = start; (void *)p < (start + len); p++) + for (p = start; (char *)p < ((char *)start + len); p++) *p = cpu_to_fdt32(FDT_NOP); } |