erebion/coreboot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

ARM: Don't use const pointers with the write functions.

Browse source 
This functions are by definition changing the data pointed to by their
arguments, so they shouldn't by const.

BUG=chrome-os-partner:19420
TEST=Built for snow.
BRANCH=None

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>
This commit is contained in:
Gabe Black 2013-07-30 13:52:27 -07:00 • committed by ChromeBot
commit 1e18d98b9a
1 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
src/arch/armv7/include/arch/io.h
View file

@ -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;