From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Hellwig Subject: Re: [PATCH] ATA convert GSI to irq on ia64 Date: Thu, 8 Feb 2007 20:34:53 +0000 Message-ID: <20070208203453.GA10122@infradead.org> References: <1170920422.15989.354.camel@ymzhang> <20070208194204.GC2354@infradead.org> <20070208122539.d9c24ad9.akpm@linux-foundation.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20070208122539.d9c24ad9.akpm@linux-foundation.org> Sender: linux-ide-owner@vger.kernel.org To: Andrew Morton Cc: Christoph Hellwig , "Zhang, Yanmin" , LKML , linux-scsi@vger.kernel.org, linux-ide@vger.kernel.org, Jeff Garzik List-Id: linux-scsi@vger.kernel.org On Thu, Feb 08, 2007 at 12:25:39PM -0800, Andrew Morton wrote: > On Thu, 8 Feb 2007 19:42:04 +0000 > Christoph Hellwig wrote: > > > On Thu, Feb 08, 2007 at 03:40:22PM +0800, Zhang, Yanmin wrote: > > > If an ATA drive uses legacy mode, ata driver will choose 14 and 15 as the > > > fixed irq number. On ia64 platform, such numbers are GSI and should be converted > > > to irq vector. > > > > > > Below patch against kernel 2.6.20 fixes it. > > > > These direct arch ifdefs in core aren't acceptable. > > For 2.6.21 that's true. But this is a suitable fix for 2.6.20.x, no? Just putting in an #ifndef and allow to to set it up in asm/ata.h is almost as trivial and a lot nice conceptually. Then again what happened to the idea of putting fixes into mainline before -stable?