Patrick Georgi ([email protected]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/783
-gerrit commit 044489fc6f8e15eaafa4172186ec6a6fb79b4f90 Author: Patrick Georgi <[email protected]> Date: Sun Mar 11 19:42:33 2012 +0100 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 <[email protected]> --- src/include/cpu/x86/lapic.h | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/include/cpu/x86/lapic.h b/src/include/cpu/x86/lapic.h index 8b44a6c..68608ed 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); } -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
