From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id ; Sat, 22 Mar 2003 17:04:45 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id ; Sat, 22 Mar 2003 17:04:45 -0500 Received: from pc2-cwma1-4-cust86.swan.cable.ntl.com ([213.105.254.86]:31900 "EHLO irongate.swansea.linux.org.uk") by vger.kernel.org with ESMTP id ; Sat, 22 Mar 2003 17:04:44 -0500 Subject: Re: [PATCH] Re: 2.5.65-ac2 -- hda/ide trouble on ICH4 From: Alan Cox To: Bartlomiej Zolnierkiewicz Cc: Dominik Brodowski , Linux Kernel Mailing List In-Reply-To: References: Content-Type: text/plain Content-Transfer-Encoding: 7bit Organization: Message-Id: <1048375677.9219.42.camel@irongate.swansea.linux.org.uk> Mime-Version: 1.0 X-Mailer: Ximian Evolution 1.2.2 (1.2.2-5) Date: 22 Mar 2003 23:27:58 +0000 Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Sat, 2003-03-22 at 22:03, Bartlomiej Zolnierkiewicz wrote: > Previously callers called it with masked_irq=0 and disabling/enabling > hwif->irq code wasn't executed, now ide_do_request() is called with > masked_irq=IDE_NO_IRQ=-1 so this code is executed for sure. You are right - I botched the simplification of that. The logic is actually cleaner than I did with a bit more thought - IDE_NO_IRQ can go away and we should be using hwif->irq as the argument.