Re: [PATCH 00/28 v6] m68k: Convert to genirq

[Geert Uytterhoeven]

On Tue, Sep 13, 2011 at 13:11, Finn Thain <fthain@xxxxxxxxxxxxxxxxxxx> 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@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-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/