On Sat, Aug 3, 2013 at 3:37 AM, Michael Schmitz <[email protected]> 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 -- [email protected]
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 [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
