From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ug-out-1314.google.com (ug-out-1314.google.com [66.249.92.170]) by ozlabs.org (Postfix) with ESMTP id A8D32DDE2D for ; Thu, 15 Feb 2007 06:42:51 +1100 (EST) Received: by ug-out-1314.google.com with SMTP id k3so280408ugf for ; Wed, 14 Feb 2007 11:42:49 -0800 (PST) From: Bartlomiej Zolnierkiewicz To: Sergei Shtylyov Subject: Re: [PATCH] (pata-2.6 fix queue) sl82c105: DMA support code cleanup Date: Wed, 14 Feb 2007 20:48:32 +0100 References: <200702140043.25244.sshtylyov@ru.mvista.com> In-Reply-To: <200702140043.25244.sshtylyov@ru.mvista.com> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Message-Id: <200702142048.32535.bzolnier@gmail.com> Cc: linux-ide@vger.kernel.org, linuxppc-dev@ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Tuesday 13 February 2007 22:43, Sergei Shtylyov wrote: > Fold the now equivalent code in the ide_dma_check() method into a mere call to > ide_use_dma(). Make config_for_dma() return non-zero if DMA mode has been set > and call it from the ide_dma_check() method instead of ide_dma_on(). > Also, defer writing the DMA timings to the chip registers until DMA is actually > turned on (and do not enable IORDY for DMA). Remove unnecessary code from the > init_hwif() method, improve its looks overall, and also rename the driver's > ide_dma_check() and dma_start() methods. > > Signed-off-by: Sergei Shtylyov > > --- > The patch has been actually tested at last. :-) > Note that now this driver can be joined to the ide-use-fast-pio patch. applied