From: "Kasatkin, Dmitry" Subject: Re: [PATCH v3 7/7] crypto: omap_sham: Remove usage of private DMA API Date: Tue, 13 Nov 2012 23:47:57 -0800 Message-ID: References: <1352257033-32495-1-git-send-email-mgreer@animalcreek.com> <1352257033-32495-8-git-send-email-mgreer@animalcreek.com> <20121109171745.GB13898@animalcreek.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Cc: linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-crypto@vger.kernel.org, herbert@gondor.apana.org.au, davem@davemloft.net, rmk+kernel@arm.linux.org.uk, paul@pwsan.com, khilman@deeprootsystems.com To: "Mark A. Greer" Return-path: Received: from mga12.intel.com ([143.182.124.36]:58744 "EHLO azsmga102.ch.intel.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S932666Ab2KNHr7 (ORCPT ); Wed, 14 Nov 2012 02:47:59 -0500 Received: by mail-qc0-f198.google.com with SMTP id e13so241616qcs.1 for ; Tue, 13 Nov 2012 23:47:57 -0800 (PST) In-Reply-To: <20121109171745.GB13898@animalcreek.com> Sender: linux-crypto-owner@vger.kernel.org List-ID: On Fri, Nov 9, 2012 at 9:17 AM, Mark A. Greer wrote: > On Fri, Nov 09, 2012 at 06:28:16PM +0200, Kasatkin, Dmitry wrote: >> On Wed, Nov 7, 2012 at 4:57 AM, Mark A. Greer wrote: >> > From: "Mark A. Greer" >> > >> > Remove usage of the private OMAP DMA API. >> > The dmaengine API will be used instead. >> > >> > CC: Russell King >> > CC: Dmitry Kasatkin >> > Signed-off-by: Mark A. Greer >> > --- >> > drivers/crypto/omap-sham.c | 117 --------------------------------------------- >> > 1 file changed, 117 deletions(-) >> > >> > diff --git a/drivers/crypto/omap-sham.c b/drivers/crypto/omap-sham.c >> > index b57277c..ebb5255 100644 >> > --- a/drivers/crypto/omap-sham.c >> > +++ b/drivers/crypto/omap-sham.c > >> > @@ -807,18 +762,6 @@ static int omap_sham_handle_queue(struct omap_sham_dev *dd, >> > if (err) >> > goto err1; >> > >> > -#ifdef OMAP_SHAM_DMA_PRIVATE >> > - omap_set_dma_dest_params(dd->dma_lch, 0, >> > - OMAP_DMA_AMODE_CONSTANT, >> > - dd->phys_base + SHA_REG_DIN(0), 0, 16); >> > - >> > - omap_set_dma_dest_burst_mode(dd->dma_lch, >> > - OMAP_DMA_DATA_BURST_16); >> > - >> > - omap_set_dma_src_burst_mode(dd->dma_lch, >> > - OMAP_DMA_DATA_BURST_4); >> >> Burst mode significantly improves performance. >> How do you configure burst mode with new API? > > This is (or should be) taken care of by the dmaengine infrastructure. > I've noted that there's an issue and there is a discussion about it > here: > > http://www.spinics.net/lists/linux-omap/msg79855.html > > We probably need to extend the dmaengine API to allow API-users to > request specific tweaks/optimizations/whatever but that's MHO. > Hello, I am in favor of new APIs. The only my concern is that it performs worse.. Is it possible to keep burst mode setting there? - Dmitry > Mark > --