On 20 November 2012 00:00, Michael Cree <mcree@xxxxxxxxxxxx> wrote: > On Mon, Nov 19, 2012 at 04:41:39PM +0530, Sachin Kamat wrote: >> asm/xchg.h was included twice. >> >> Signed-off-by: Sachin Kamat <sachin.kamat@xxxxxxxxxx> >> --- >> arch/alpha/include/asm/cmpxchg.h | 1 - >> 1 files changed, 0 insertions(+), 1 deletions(-) >> >> diff --git a/arch/alpha/include/asm/cmpxchg.h b/arch/alpha/include/asm/cmpxchg.h >> index 429e8cd..8df1674 100644 >> --- a/arch/alpha/include/asm/cmpxchg.h >> +++ b/arch/alpha/include/asm/cmpxchg.h >> @@ -40,7 +40,6 @@ >> #undef ____cmpxchg >> #define ____xchg(type, args...) __xchg ##type(args) >> #define ____cmpxchg(type, args...) __cmpxchg ##type(args) >> -#include <asm/xchg.h> > > asm/xchg.h is intended to be included twice. Your patch will break > compilation. I did not realise it. I thought it was an unconditional include. Sorry for the noise. > > Cheers > Michael. > -- With warm regards, Sachin -- To unsubscribe from this list: send the line "unsubscribe linux-alpha" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html