Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1423328AbXBHUe7 (ORCPT ); Thu, 8 Feb 2007 15:34:59 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1423326AbXBHUe7 (ORCPT ); Thu, 8 Feb 2007 15:34:59 -0500 Received: from pentafluge.infradead.org ([213.146.154.40]:57085 "EHLO pentafluge.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1423317AbXBHUe5 (ORCPT ); Thu, 8 Feb 2007 15:34:57 -0500 Date: Thu, 8 Feb 2007 20:34:53 +0000 From: Christoph Hellwig To: Andrew Morton Cc: Christoph Hellwig , "Zhang, Yanmin" , LKML , linux-scsi@vger.kernel.org, linux-ide@vger.kernel.org, Jeff Garzik Subject: Re: [PATCH] ATA convert GSI to irq on ia64 Message-ID: <20070208203453.GA10122@infradead.org> Mail-Followup-To: Christoph Hellwig , Andrew Morton , "Zhang, Yanmin" , LKML , linux-scsi@vger.kernel.org, linux-ide@vger.kernel.org, Jeff Garzik 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 Content-Disposition: inline In-Reply-To: <20070208122539.d9c24ad9.akpm@linux-foundation.org> User-Agent: Mutt/1.4.2.2i X-SRS-Rewrite: SMTP reverse-path rewritten from by pentafluge.infradead.org See http://www.infradead.org/rpr.html Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1043 Lines: 24 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? - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/