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, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]
Archive: 
http://lists.debian.org/camuhmdxiwukj7o29dsw5vywaxfqur4n5ox-q_baizvhmm98...@mail.gmail.com

Reply via email to