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.175]) by ozlabs.org (Postfix) with ESMTP id CD88FDDEA3 for ; Thu, 5 Apr 2007 05:36:41 +1000 (EST) Received: by ug-out-1314.google.com with SMTP id k3so750360ugf for ; Wed, 04 Apr 2007 12:36:39 -0700 (PDT) From: Bartlomiej Zolnierkiewicz To: Sergei Shtylyov Subject: Re: [PATCH pata-2.6] sl82c105: DMA support code cleanup (take 4) Date: Wed, 4 Apr 2007 21:38:02 +0200 References: <200702140043.25244.sshtylyov@ru.mvista.com> <200702231913.55522.sshtylyov@ru.mvista.com> <200703282250.02679.sshtylyov@ru.mvista.com> In-Reply-To: <200703282250.02679.sshtylyov@ru.mvista.com> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Message-Id: <200704042138.02233.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 Wednesday 28 March 2007, 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(). > Defer writing the DMA timings to the chip registers until DMA is really turned > on (and do not enable IORDY for DMA). > Remove unneeded code from the init_hwif() method, improve its overall looks. > Rename the dma_start(), ide_dma_check(), and ide_dma_lostirq() methods, and > also use more proper hwif->dma_command, fix printk() and comment in the latter > one as well. While at it, cleanup style in several places. > > --- > This patch has also been actually tested at last. :-) > > Signed-off-by: Sergei Shtylyov > Signed-off-by: Bartlomiej Zolnierkiewicz I moved Signed-off-by-s before "---" to prevent them from being eaten by git. applied