A lot of calls don't operate on bytes but on words or on structured data. So instead of a pointer to uint8_t, a void pointer is the better choice.
This allows removing many type casts. (Some very early implementations of memcpy used char pointers which were replaced by void pointers for the same reason). v2: Change a type cast from (uint8_t *) to (void *) to improve readability as suggested by Aurelien Jarno. This type cast is needed to remove the const attribute, not to change the data type to (uint8_t *). Cc: Aurelien Jarno <aurel...@aurel32.net> Signed-off-by: Stefan Weil <w...@mail.berlios.de> --- cpu-common.h | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/cpu-common.h b/cpu-common.h index ef4e8da..96c02ae 100644 --- a/cpu-common.h +++ b/cpu-common.h @@ -68,14 +68,14 @@ void cpu_unregister_io_memory(int table_address); void cpu_physical_memory_rw(target_phys_addr_t addr, uint8_t *buf, int len, int is_write); static inline void cpu_physical_memory_read(target_phys_addr_t addr, - uint8_t *buf, int len) + void *buf, int len) { cpu_physical_memory_rw(addr, buf, len, 0); } static inline void cpu_physical_memory_write(target_phys_addr_t addr, - const uint8_t *buf, int len) + const void *buf, int len) { - cpu_physical_memory_rw(addr, (uint8_t *)buf, len, 1); + cpu_physical_memory_rw(addr, (void *)buf, len, 1); } void *cpu_physical_memory_map(target_phys_addr_t addr, target_phys_addr_t *plen, -- 1.7.2.5