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_ro.c | |
| parent | afe3848b79a7ff351e9fbf3a7c162d2de002279b (diff) | |
| download | olio-uboot-2014.01-ef4e8ce1beb5b93aedda5a4c1b90bfd989c6791e.tar.xz olio-uboot-2014.01-ef4e8ce1beb5b93aedda5a4c1b90bfd989c6791e.zip | |
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_ro.c')
| -rw-r--r-- | libfdt/fdt_ro.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/libfdt/fdt_ro.c b/libfdt/fdt_ro.c index 69af7bb84..8382afd38 100644 --- a/libfdt/fdt_ro.c +++ b/libfdt/fdt_ro.c @@ -412,10 +412,10 @@ int fdt_node_offset_by_phandle(const void *fdt, uint32_t phandle) &phandle, sizeof(phandle)); } -int _stringlist_contains(const void *strlist, int listlen, const char *str) +int _stringlist_contains(const char *strlist, int listlen, const char *str) { int len = strlen(str); - const void *p; + const char *p; while (listlen >= len) { if (memcmp(str, strlist, len+1) == 0) |