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 /include/libfdt_env.h | |
| parent | afe3848b79a7ff351e9fbf3a7c162d2de002279b (diff) | |
| download | talos-obmc-uboot-ef4e8ce1beb5b93aedda5a4c1b90bfd989c6791e.tar.gz talos-obmc-uboot-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 'include/libfdt_env.h')
| -rw-r--r-- | include/libfdt_env.h | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/include/libfdt_env.h b/include/libfdt_env.h index a7fd2f8d9a..671c3a8981 100644 --- a/include/libfdt_env.h +++ b/include/libfdt_env.h @@ -38,4 +38,16 @@ extern struct fdt_header *working_fdt; /* Pointer to the working fdt */ #define fdt64_to_cpu(x) __be64_to_cpu(x) #define cpu_to_fdt64(x) __cpu_to_be64(x) +/* + * Types for `void *' pointers. + * + * Note: libfdt uses this definition from /usr/include/stdint.h. + * Define it here rather than pulling in all of stdint.h. + */ +#if __WORDSIZE == 64 +typedef unsigned long int uintptr_t; +#else +typedef unsigned int uintptr_t; +#endif + #endif /* _LIBFDT_ENV_H */ |
