From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergei Shtylyov Subject: Re: [PATCH 01/10] ide: add ->dma_clear method and remove ->dma_timeout one Date: Mon, 09 Mar 2009 01:17:21 +0300 Message-ID: <49B443F1.7050102@ru.mvista.com> References: <20090220171139.25429.26702.sendpatchset@localhost.localdomain> <20090220171147.25429.26218.sendpatchset@localhost.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from h155.mvista.com ([63.81.120.155]:5994 "EHLO imap.sh.mvista.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1753521AbZCHWR2 (ORCPT ); Sun, 8 Mar 2009 18:17:28 -0400 In-Reply-To: <20090220171147.25429.26218.sendpatchset@localhost.localdomain> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Bartlomiej Zolnierkiewicz Cc: linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org Bartlomiej Zolnierkiewicz wrote: > All custom ->dma_timeout implementations call the generic one thus it is > possible to have only an optional method for resetting DMA engine instead: > > * Add ->dma_clear method and convert hpt366, pdc202xx_old and sl82c105 > host drivers to use it. > > * Always use ide_dma_timeout() in ide_dma_timeout_retry() and remove > ->dma_timeout method. > > * Make ide_dma_timeout() static. > > There should be no functional changes caused by this patch. > > Signed-off-by: Bartlomiej Zolnierkiewicz > Acked-by: Sergei Shtylyov > Index: b/include/linux/ide.h > =================================================================== > --- a/include/linux/ide.h > +++ b/include/linux/ide.h > @@ -715,8 +715,9 @@ struct ide_dma_ops { > int (*dma_end)(struct ide_drive_s *); > int (*dma_test_irq)(struct ide_drive_s *); > void (*dma_lost_irq)(struct ide_drive_s *); > + /* below ones are optional */ > int (*dma_timer_expiry)(struct ide_drive_s *); > - void (*dma_timeout)(struct ide_drive_s *); > + void (*dma_clear)(struct ide_drive_s *); > /* > * The following method is optional and only required to be > Perhaps worth removing the words about method being optional now... > * implemented for the SFF-8038i compatible controllers. > MBR, Sergei