From mboxrd@z Thu Jan 1 00:00:00 1970 From: Robert Hancock Subject: Re: sata_nv + ADMA + Samsung disk problem Date: Mon, 07 Jan 2008 18:10:29 -0600 Message-ID: <4782BF75.9040803@shaw.ca> References: <20070808120804.GB5257@boogie.lpds.sztaki.hu> <20080101164416.GA29574@boogie.lpds.sztaki.hu> <477B0429.7040909@gmail.com> <477B0CFD.1030603@shaw.ca> <477BDEA5.8040701@garzik.org> <477C2A99.9010208@shaw.ca> <477C61D3.30009@rtr.ca> <477C6A85.9020607@shaw.ca> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from idcmail-mo1so.shaw.ca ([24.71.223.10]:13875 "EHLO pd4mo1so.prod.shaw.ca" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754334AbYAHALq (ORCPT ); Mon, 7 Jan 2008 19:11:46 -0500 In-reply-to: Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Gabor Gombas Cc: Allen Martin , Mark Lord , Jeff Garzik , Tejun Heo , linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org, Kuan Luo , Peer Chen Allen Martin wrote: > >>> Dunno about the NVidia version. >> Theirs works rather differently - the GO bit is there, but there's >> another append register which is used to tell the controller >> that a new >> tag has been added to the CPB list. >> >> The only thing we currently use the GO bit for is to switch >> between ADMA >> and port register mode. Could be there's something we need to >> do there, >> though, who knows.. >> > > You shouldn't ever need to touch GO other than the ADMA / legacy mode > switch as you say. > > The NVIDIA ADMA hw is not based on the Pacific Digital core. Gabor, I just noticed you said that it worked OK in 2.6.20, yet 2.6.22 fails. 2.6.20 had ADMA support as well, so I wonder what change started causing the problem. Would it be possible for you to do a git bisect (or at least try 2.6.21 to try and narrow it down)?