diff options
author | Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at> | 2013-07-17 23:46:44 +0000 |
---|---|---|
committer | Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at> | 2013-07-17 23:46:44 +0000 |
commit | 305e0b999a7d452a845709d5558c17a31afe178c (patch) | |
tree | 3b65c3cd627ff55d683f26bf0819913a3e63c5d8 /flash.h | |
parent | 4a03865cd9b192e6e65b5dbc5a928af87322385d (diff) | |
download | flashrom-305e0b999a7d452a845709d5558c17a31afe178c.tar.gz flashrom-305e0b999a7d452a845709d5558c17a31afe178c.tar.bz2 flashrom-305e0b999a7d452a845709d5558c17a31afe178c.zip |
Uintptr_t-ify map_flash_region functions
Unsigned long is not the right type for manipulating pointer values.
Since C99 there are suitable unsigned and signed types available, namely
uintptr_t and intptr_t respectively.
Use them in functions assigned to programmers' map_flash_region fields and
their callers where applicable.
This patch also changes the display width of all associated address values in
physmap.c to 16/8 hex characters depending on the actual size by
introducing a macro PRIxPTR_WIDTH and exploiting printf's * field width
specifier.
Corresponding to flashrom svn r1701.
Signed-off-by: Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at>
Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>
Diffstat (limited to 'flash.h')
-rw-r--r-- | flash.h | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -43,10 +43,10 @@ /* TODO: check using code for correct usage of types */ typedef uintptr_t chipaddr; +#define PRIxPTR_WIDTH ((int)(sizeof(uintptr_t)*2)) int register_shutdown(int (*function) (void *data), void *data); -void *programmer_map_flash_region(const char *descr, unsigned long phys_addr, - size_t len); +void *programmer_map_flash_region(const char *descr, uintptr_t phys_addr, size_t len); void programmer_unmap_flash_region(void *virt_addr, size_t len); void programmer_delay(int usecs); |