diff options
author | Gabe Black <gabeblack@google.com> | 2013-07-30 13:52:27 -0700 |
---|---|---|
committer | Patrick Georgi <patrick@georgi-clan.de> | 2013-12-21 10:48:57 +0100 |
commit | dd8f60363a007b43714fa8c0a6044fb6c7fb9866 (patch) | |
tree | fb2d71adffb9ccb0eb926948ee98ac626845f5d9 | |
parent | 713853a9c8fca493567548fe3cf0b8107b7fc138 (diff) | |
download | coreboot-dd8f60363a007b43714fa8c0a6044fb6c7fb9866.tar.xz |
arm: Don't use const pointers with the write functions
This functions are by definition changing the data pointed to by their
arguments, so they shouldn't by const.
Change-Id: Id29b3f76526aba463f8bb744f53101327f9c7bde
Signed-off-by: Gabe Black <gabeblack@google.com>
Reviewed-on: https://gerrit.chromium.org/gerrit/63777
Commit-Queue: Gabe Black <gabeblack@chromium.org>
Reviewed-by: Gabe Black <gabeblack@chromium.org>
Tested-by: Gabe Black <gabeblack@chromium.org>
Reviewed-on: http://review.coreboot.org/4400
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick@georgi-clan.de>
-rw-r--r-- | src/arch/armv7/include/arch/io.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/arch/armv7/include/arch/io.h b/src/arch/armv7/include/arch/io.h index be1da51c71..e044090e68 100644 --- a/src/arch/armv7/include/arch/io.h +++ b/src/arch/armv7/include/arch/io.h @@ -45,21 +45,21 @@ static inline uint32_t read32(const void *addr) return *(volatile uint32_t *)addr; } -static inline void write8(uint8_t val, const void *addr) +static inline void write8(uint8_t val, void *addr) { dmb(); *(volatile uint8_t *)addr = val; dmb(); } -static inline void write16(uint16_t val, const void *addr) +static inline void write16(uint16_t val, void *addr) { dmb(); *(volatile uint16_t *)addr = val; dmb(); } -static inline void write32(uint32_t val, const void *addr) +static inline void write32(uint32_t val, void *addr) { dmb(); *(volatile uint32_t *)addr = val; |