Re: AVR32 architecture patch against Linux 2.6.18-rc1 available

[Haavard Skinnemoen]

On Thu, 06 Jul 2006 13:30:25 +0200
Thomas Gleixner <tglx@xxxxxxxxxxxxx> wrote:

> On Thu, 2006-07-06 at 12:03 +0200, Haavard Skinnemoen wrote: 
> > > Looks pretty sane from a quick scan.
> > > 
> > > - request_irq() can use GFP_KERNEL?
> > 
> > Probably, but the genirq implementation also uses GFP_ATOMIC.
> 
> Is there a good reason, why AVR32 needs its own interrupt handling
> implementation ?

No, not really. At least not after the genirq stuff went in. I used to
be a bit concerned about the generic irq code being too heavyweight,
but I think handle_simple_irq() might be just what we need for
chip-internal interrupt handling.

> >From a short glance there's nothing which can not be handled by the
> generic code. Also there are a couple of things missing -e.g.
> recursive enable/disable_irq() handling. 

You're probably right. I'll see if I can get it converted to genirq
one of the next days.

HÃvard
-
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/