summaryrefslogtreecommitdiff
path: root/payloads/libpayload/include/arm
diff options
context:
space:
mode:
authorJulius Werner <jwerner@chromium.org>2014-07-17 10:43:15 -0700
committerMarc Jones <marc.jones@se-eng.com>2015-01-12 05:56:01 +0100
commit8a1d11f7973f3b4d01f2d9e7c57a4a0c5e7c3959 (patch)
tree60ded0483a4bc67619918463d78d366a96ac2483 /payloads/libpayload/include/arm
parent1c5cdad09efa7d6bd196ad852dbeb5f740ac66e3 (diff)
downloadcoreboot-8a1d11f7973f3b4d01f2d9e7c57a4a0c5e7c3959.tar.xz
libpayload: Expand setbits_le32() and fix readl() const-ness
setbits_le32() is not really arch-specific... the arch-specific part of accessing memory is wrapped by readl() and writel(), and the endianness can be accounted for with the right macros. Generalize the definitions, add a be32 version and move them to endian.h so that all platforms can use them. Also include endian.h from libpayload.h so we won't update any payload's old use of the macros (endianness is something useful enough to always have avalable anyway, and shouldn't clash with other things). This also fixes a bug where these macros would only be available if libpayload-config.h had been independently included before. Also fix a bug with readl() macros on all archs where they refused to work on const pointers (which they should). CQ-DEPEND=CL:208712 BUG=None TEST=Stuff still compiles. Built and booted on Storm. Original-Change-Id: I01a7fbadbb5d740675657d95c1e969027562ba8c Original-Signed-off-by: Julius Werner <jwerner@chromium.org> Original-Reviewed-on: https://chromium-review.googlesource.com/208713 Original-Reviewed-by: Vadim Bendebury <vbendeb@chromium.org> Original-Reviewed-by: David Hendricks <dhendrix@chromium.org> (cherry picked from commit 951f8a6d77bc21bd793bf4f228a0965ade586f00) Signed-off-by: Marc Jones <marc.jones@se-eng.com> Change-Id: I51c25f01b200b91abbe32c879905349bb05dc9c8 Reviewed-on: http://review.coreboot.org/8129 Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org> Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net> Tested-by: build bot (Jenkins)
Diffstat (limited to 'payloads/libpayload/include/arm')
-rw-r--r--payloads/libpayload/include/arm/arch/io.h24
1 files changed, 6 insertions, 18 deletions
diff --git a/payloads/libpayload/include/arm/arch/io.h b/payloads/libpayload/include/arm/arch/io.h
index 5cb66dcd69..4b0a687c7b 100644
--- a/payloads/libpayload/include/arm/arch/io.h
+++ b/payloads/libpayload/include/arm/arch/io.h
@@ -34,22 +34,22 @@
#include <stdint.h>
#include <arch/cache.h>
-static inline uint8_t readb(volatile void *_a)
+static inline uint8_t readb(volatile const void *_a)
{
dmb();
- return *(volatile uint8_t *)_a;
+ return *(volatile const uint8_t *)_a;
}
-static inline uint16_t readw(volatile void *_a)
+static inline uint16_t readw(volatile const void *_a)
{
dmb();
- return *(volatile uint16_t *)_a;
+ return *(volatile const uint16_t *)_a;
}
-static inline uint32_t readl(volatile void *_a)
+static inline uint32_t readl(volatile const void *_a)
{
dmb();
- return *(volatile uint32_t *)_a;
+ return *(volatile const uint32_t *)_a;
}
static inline void writeb(uint8_t _v, volatile void *_a)
@@ -73,17 +73,5 @@ static inline void writel(uint32_t _v, volatile void *_a)
dmb();
}
-/*
- * Handy bit manipulation macros, BE version will have to be added when/if
- * needed.
- */
-
-#ifdef CONFIG_LP_LITTLE_ENDIAN
-#define clrsetbits_le32(addr, clear, set) writel((readl(addr) & ~(clear)) |\
- (set), (addr))
-#define setbits_le32(addr, set) writel(readl(addr) | (set), (addr))
-#define clrbits_le32(addr, clear) writel(readl(addr) & ~(clear), (addr))
-#endif
-
#endif