[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAMuHMdXJEeooAjdQo+JhUWJcrGzKZb-jQPPhVggy3ocQyPe3Lw@mail.gmail.com>
Date: Tue, 13 Sep 2011 13:18:06 +0200
From: Geert Uytterhoeven <geert@...ux-m68k.org>
To: Finn Thain <fthain@...egraphics.com.au>
Cc: linux-m68k@...ts.linux-m68k.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 00/28 v6] m68k: Convert to genirq
On Tue, Sep 13, 2011 at 13:11, Finn Thain <fthain@...egraphics.com.au> wrote:
> When I was testing your v5 patch series, I had to patch macmace.c to
> resolve some compiler errors --
>
> --- linux-m68k.orig/drivers/net/macmace.c 2011-09-13 18:33:55.000000000 +1000
> +++ linux-m68k/drivers/net/macmace.c 2011-09-13 18:34:05.000000000 +1000
> @@ -31,8 +31,8 @@
> #include <linux/dma-mapping.h>
> #include <linux/platform_device.h>
> #include <linux/gfp.h>
> +#include <linux/interrupt.h>
> #include <asm/io.h>
> -#include <asm/irq.h>
> #include <asm/macints.h>
> #include <asm/mac_psc.h>
> #include <asm/page.h>
>
> Would you please check whether this is still needed after the v6 series?
Will do. Don't think I ever saw that problem, though.
Did it show up with genirq disabled or enabled?
> (I'd check myself, but I don't have the source tree on which your emailed
> patches were based...)
They're based on a v3.1-rc5 version of the m68k tree. I think it's
identical to the
current m68k master with v3.1-rc5 merged into.
> BTW, you'll probably want to patch this comment in
> include/linux/interrupt.h --
>
> /*
> * Special function to avoid ifdeffery in kernel/irq/devres.c which
> * gets magically built by GENERIC_HARDIRQS=n architectures (sparc,
> * m68k).
Thanks, will do.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@...ux-m68k.org
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-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists