- To: Dave Martin <dave.martin@xxxxxxxxxx>
- Subject: Re: [PATCH] ata: Don't use NO_IRQ in pata_of_platform driver
- From: Nicolas Pitre <nicolas.pitre@xxxxxxxxxx>
- Date: Mon, 5 Dec 2011 12:40:16 -0500 (EST)
- Cc: Russell King - ARM Linux <linux@xxxxxxxxxxxxxxxx>, Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>, Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>, Anton Vorontsov <cbouatmailru@xxxxxxxxx>, Alan Cox <alan@xxxxxxxxxxxxxxxxxxx>, Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx>, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, devicetree-discuss@xxxxxxxxxxxxxxxx, LKML <linux-kernel@xxxxxxxxxxxxxxx>, linux-ide@xxxxxxxxxxxxxxx, Randy Dunlap <rdunlap@xxxxxxxxxxxx>, linux-next@xxxxxxxxxxxxxxx, Ingo Molnar <mingo@xxxxxxx>, Jeff Garzik <jgarzik@xxxxxxxxxx>, Pawel Moll <pawel.moll@xxxxxxx>, linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
- In-reply-to: <20111205161157.GA27550@localhost.localdomain>
- List-id: <linux-ide.vger.kernel.org>
- User-agent: Alpine 2.02 (LFD 1266 2009-07-14)
On Mon, 5 Dec 2011, Dave Martin wrote:
> On Sat, Dec 03, 2011 at 10:12:53AM +1100, Benjamin Herrenschmidt wrote:
> > On Fri, 2011-12-02 at 11:28 -0800, Linus Torvalds wrote:
> > > Don't *change* NO_IRQ to zero (that whole #define is broken - leave it
> > > around as a marker of brokenness), just start removing it from all the
> > > ARM drivers that use the OF infrastructure. Which is presumably not
> > > all that many yet.
> > >
> > > So whenever you find breakage, the fix now is to just remove NO_IRQ
> > > tests, and replace them with "!irq".
> >
>
> Russell, do you know whether it would make sense to set a timeline for
> removing NO_IRQ from ARM platforms and migrating to 0 for the no-interrupt
> case? I'm assuming that this mainly involves migrating existing hard-wired
> code that deals with interrupt numbers to use irq domains.
How many drivers do use IRQ #0 to start with? We might discover that in
practice there is only a very few cases where this is an issue if 0
would mean no IRQ.
Nicolas
--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
[Linux Filesystems]
[Linux SCSI]
[Linux RAID]
[Git]
[Kernel Newbies]
[Linux Newbie]
[Share Photos]
[Security]
[Netfilter]
[Bugtraq]
[Photo]
[Yosemite]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Samba]
[Video 4 Linux]
[Device Mapper]
[Linux Resources]