[PATCH v3 7/7] crypto: omap_sham: Remove usage of private DMA API

Mark A. Greer mgreer at animalcreek.com
Fri Nov 16 17:40:19 EST 2012


On Wed, Nov 14, 2012 at 09:47:55AM -0700, Mark A. Greer wrote:
> On Tue, Nov 13, 2012 at 11:47:57PM -0800, Kasatkin, Dmitry wrote:
> > On Fri, Nov 9, 2012 at 9:17 AM, Mark A. Greer <mgreer at animalcreek.com> 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 <mgreer at animalcreek.com> wrote:
> > >> > From: "Mark A. Greer" <mgreer at animalcreek.com>
> > >> >
> > >> > Remove usage of the private OMAP DMA API.
> > >> > The dmaengine API will be used instead.
> > >> >
> > >> > CC: Russell King <rmk+kernel at arm.linux.org.uk>
> > >> > CC: Dmitry Kasatkin <dmitry.kasatkin at intel.com>
> > >> > Signed-off-by: Mark A. Greer <mgreer at animalcreek.com>
> > >> > ---
> > >> >  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?
> 
> I'm going to respin the patch the I posted in the email thread above
> to only set bursting for non-cyclic DMAs.

Just sent.  Should be in your inbox shortly.

Mark
--



More information about the linux-arm-kernel mailing list