From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Milburn Subject: Re: [PATCH #sx4] sata_sx4: fixup interrupt handling Date: Wed, 13 May 2009 18:39:38 -0500 Message-ID: <4A0B5A3A.4050705@redhat.com> References: <20090513230221.GB5863@dhcp-210.hsv.redhat.com> <4A0B572B.7040302@garzik.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from mx2.redhat.com ([66.187.237.31]:42913 "EHLO mx2.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755258AbZEMXl1 (ORCPT ); Wed, 13 May 2009 19:41:27 -0400 In-Reply-To: <4A0B572B.7040302@garzik.org> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Jeff Garzik Cc: mikpe@it.uu.se, tj@kernel.org, linux-ide@vger.kernel.org Jeff Garzik wrote: > David Milburn wrote: >> Issuing ATA_CMD_SET_FEATURES (0xef) times out because >> pdc20621_interrupt ignores command completion since >> ATA_TFLAG_POLLING flag is set. >> >> This has already been fixed for sata_promise: >> >> commit 51b94d2a5a90d4800e74d7348bcde098a28f4fb3 >> Author: Tejun Heo >> Date: Fri Jun 8 13:46:55 2007 -0700 >> >> sata_promise: use TF interface for polling NODATA commands >> >> Also, this patch includes Mikael's original patches: >> >> http://marc.info/?l=linux-ide&m=121135828227724&w=2 >> http://marc.info/?l=linux-ide&m=121144512109826&w=2 >> >> Signed-off-by: Mikael Pettersson >> Signed-off-by: David Milburn >> --- >> drivers/ata/sata_sx4.c | 22 +++++++++------------- >> 1 files changed, 9 insertions(+), 13 deletions(-) > > Good spotting... > > Is this on top of #sx4 or upstream? > Jeff, this is on top of #sx4 branch. Thanks, David > Jeff > > >