From mboxrd@z Thu Jan 1 00:00:00 1970 From: Russell King Subject: Re: 2.6.32: Promise UDMA33 card refuses to work in UDMA mode Date: Thu, 24 Dec 2009 21:54:51 +0000 Message-ID: <20091224215451.GA2476@flint.arm.linux.org.uk> References: <20091224181300.GA4654@flint.arm.linux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from caramon.arm.linux.org.uk ([78.32.30.218]:54806 "EHLO caramon.arm.linux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754750AbZLXVzA (ORCPT ); Thu, 24 Dec 2009 16:55:00 -0500 Received: from flint.arm.linux.org.uk ([2002:4e20:1eda:1:201:2ff:fe14:8fad]) by caramon.arm.linux.org.uk with esmtpsa (TLSv1:AES256-SHA:256) (Exim 4.69) (envelope-from ) id 1NNve3-0007Ag-RU for linux-ide@vger.kernel.org; Thu, 24 Dec 2009 21:54:56 +0000 Received: from rmk by flint.arm.linux.org.uk with local (Exim 4.69) (envelope-from ) id 1NNve1-0000lw-1v for linux-ide@vger.kernel.org; Thu, 24 Dec 2009 21:54:53 +0000 Content-Disposition: inline In-Reply-To: <20091224181300.GA4654@flint.arm.linux.org.uk> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: linux-ide@vger.kernel.org Some additions due to further testing: On Thu, Dec 24, 2009 at 06:13:00PM +0000, Russell King wrote: > I tried upgrading my main machine from an old 2.6.23 kernel to 2.6.32. > In doing so, I switched from IDE to ATA support. IDE on 2.6.32 results in "irq 24: nobody cared" for the IDE interrupt. Booting back to 2.6.23 results in everything working normally again. > Could this be another case where the kernel should always need to write > out the full task file to the drive? I don't see any form of hardware > control on the PDC20247 (which is the UDMA add-on to the PDC20246 chip) > to control its mode, other than it snooping the taskfile writes. I tried commenting out the caching of the drive control register: diff --git a/drivers/ata/libata-sff.c b/drivers/ata/libata-sff.c index bbbb1fa..ddd275a 100644 --- a/drivers/ata/libata-sff.c +++ b/drivers/ata/libata-sff.c @@ -574,12 +574,12 @@ void ata_sff_tf_load(struct ata_port *ap, const struct ata_taskfile *tf) struct ata_ioports *ioaddr = &ap->ioaddr; unsigned int is_addr = tf->flags & ATA_TFLAG_ISADDR; - if (tf->ctl != ap->last_ctl) { +// if (tf->ctl != ap->last_ctl) { if (ioaddr->ctl_addr) iowrite8(tf->ctl, ioaddr->ctl_addr); ap->last_ctl = tf->ctl; ata_wait_idle(ap); - } +// } if (is_addr && (tf->flags & ATA_TFLAG_LBA48)) { WARN_ON_ONCE(!ioaddr->ctl_addr); This resulted in the same failure, but without the 'timeout' for the second drive. So there seems to be a separate problem in that when the Promise host gets reset, the host state isn't properly updated on all channels, resulting in the drive control register not being written for the 'other' channel. -- Russell King Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/ maintainer of: