On Sat, Jun 22, 2013 at 06:54:14AM +0100, Chen Gang wrote: > > If 'COMPAT' not defined, aarch32_break_handler() cannot pass compiling, > so need add '#ifdef' for it just like the header file has done. > > The related make: > > make ARCH=arm64 randconfig > make ARCH=arm64 menuconfig > make ARCH=arm64 V=1 EXTRA_CFLAGS=-W > > The related error: > > arch/arm64/kernel/debug-monitors.c:249:5: error: redefinition of ‘aarch32_break_handler’ > In file included from arch/arm64/kernel/debug-monitors.c:29:0: > /root/linux-next/arch/arm64/include/asm/debug-monitors.h:89:12: note: previous definition of ‘aarch32_break_handler’ was here Actually, can we just kill the static definition in debug-monitors.h instead? I don't see anything in the full-blown aardch32_break_handler that won't play nicely if !COMPAT. Will -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html