diff options
| author | David Gibson <david@gibson.dropbear.id.au> | 2008-07-07 10:10:48 +1000 |
|---|---|---|
| committer | Jon Loeliger <jdl@jdl.com> | 2008-07-14 12:36:08 -0500 |
| commit | 36786db6154533b67d736b414ef63b4457009326 (patch) | |
| tree | 73ddad751d647db11333552261873dc6293eec42 /libfdt/libfdt_internal.h | |
| parent | 76e0622b687d795bb1379cf183c6ce8613e14658 (diff) | |
| download | dtc-36786db6154533b67d736b414ef63b4457009326.zip dtc-36786db6154533b67d736b414ef63b4457009326.tar.gz dtc-36786db6154533b67d736b414ef63b4457009326.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.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Diffstat (limited to 'libfdt/libfdt_internal.h')
| -rw-r--r-- | libfdt/libfdt_internal.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/libfdt/libfdt_internal.h b/libfdt/libfdt_internal.h index f72e70d..2ba30db 100644 --- a/libfdt/libfdt_internal.h +++ b/libfdt/libfdt_internal.h @@ -72,7 +72,7 @@ int _fdt_node_end_offset(void *fdt, int nodeoffset); static inline const void *_fdt_offset_ptr(const void *fdt, int offset) { - return fdt + fdt_off_dt_struct(fdt) + offset; + return (const char *)fdt + fdt_off_dt_struct(fdt) + offset; } static inline void *_fdt_offset_ptr_w(void *fdt, int offset) @@ -82,8 +82,8 @@ static inline void *_fdt_offset_ptr_w(void *fdt, int offset) static inline const struct fdt_reserve_entry *_fdt_mem_rsv(const void *fdt, int n) { - const struct fdt_reserve_entry *rsv_table = - fdt + fdt_off_mem_rsvmap(fdt); + const struct fdt_reserve_entry *rsv_table = (struct fdt_reserve_entry *) + ((const char *)fdt + fdt_off_mem_rsvmap(fdt)); return rsv_table + n; } |