On Sat, Aug 3, 2013 at 3:37 AM, Michael Schmitz <schmitzmic@xxxxxxxxx> wrote:
static inline bool arch_irqs_disabled_flags(unsigned long flags) { + #if defined(MACH_ATARI_ONLY) || defined(MACH_AMIGA_ONLY)
The above is not needed: - If MACH_ATARI_ONLY, MACH_IS_ATARI is hardcoded to 1, so the compiler will optimize-away the else part. In that case, "ALLOWINT | 0x200" == "ALLOWINT". - If MACH_AMIGA_ONLY, MACH_IS_ATARI is hardcoded to 0, and the compiler will optimize-away the other part.
return (flags & ~ALLOWINT) != 0; + #else + if (MACH_IS_ATARI) { + /* Ignore HSYNC = ipl 2 on Atari */ + return (flags & ~(ALLOWINT | 0x200)) != 0; + } else + return (flags & ~ALLOWINT) != 0; + #endif } (or whatever compiler flag we have to indicate we're not building a multi-platform kernel)
Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds -- To unsubscribe from this list: send the line "unsubscribe linux-m68k" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html