summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPatrick Georgi <patrick@georgi-clan.de>2012-03-11 19:42:33 +0100
committerPatrick Georgi <patrick@georgi-clan.de>2012-03-16 22:22:49 +0100
commit1a34165e37e8ab7ced6a639441f5df2478246976 (patch)
tree164e610d23c528f40183a3562034cecb52a4046e
parentfe9210f5d0ba6662ac47d3a9b962665d4d484fee (diff)
xchg is atomic with side-effects
clang doesn't know about the side effect, so we have to tell it that it's okay not to care about the result. Change-Id: Ib11890bff6779e36cf09c178d224695ea16a8ae8 Signed-off-by: Patrick Georgi <patrick@georgi-clan.de> Reviewed-on: http://review.coreboot.org/783 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
-rw-r--r--src/include/cpu/x86/lapic.h2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/include/cpu/x86/lapic.h b/src/include/cpu/x86/lapic.h
index 8b44a6cb66..68608edab1 100644
--- a/src/include/cpu/x86/lapic.h
+++ b/src/include/cpu/x86/lapic.h
@@ -108,7 +108,7 @@ static inline unsigned long __xchg(unsigned long x, volatile void * ptr, int siz
static inline void lapic_write_atomic(unsigned long reg, unsigned long v)
{
- xchg((volatile unsigned long *)(LAPIC_DEFAULT_BASE+reg), v);
+ (void)xchg((volatile unsigned long *)(LAPIC_DEFAULT_BASE+reg), v);
}