Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751514Ab2HBEwa (ORCPT ); Thu, 2 Aug 2012 00:52:30 -0400 Received: from co1ehsobe002.messaging.microsoft.com ([216.32.180.185]:20043 "EHLO co1outboundpool.messaging.microsoft.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751269Ab2HBEw1 convert rfc822-to-8bit (ORCPT ); Thu, 2 Aug 2012 00:52:27 -0400 X-Forefront-Antispam-Report: CIP:70.37.183.190;KIP:(null);UIP:(null);IPV:NLI;H:mail.freescale.net;RD:none;EFVD:NLI X-SpamScore: -7 X-BigFish: VS-7(zz98dI9371I542M1432Izz1202hzz8275ch8275bh8275dhz2dh2a8h668h839h8e2h8e3h944hd25hf0ah107ahbe9i) From: Liu Qiang-B32616 To: "Ira W. Snyder" CC: "linux-crypto@vger.kernel.org" , "linuxppc-dev@lists.ozlabs.org" , "linux-kernel@vger.kernel.org" , "dan.j.williams@gmail.com" , Vinod Koul , "herbert@gondor.apana.org.au" , Dan Williams , "davem@davemloft.net" Subject: RE: [PATCH v5 2/6] fsl-dma: remove attribute DMA_INTERRUPT of dmaengine Thread-Topic: [PATCH v5 2/6] fsl-dma: remove attribute DMA_INTERRUPT of dmaengine Thread-Index: AQHNb8X9jQey50gUhkqw6GQS1GFpjZdFe7SAgAB5k7A= Date: Thu, 2 Aug 2012 04:52:23 +0000 Message-ID: References: <1343810948-25343-1-git-send-email-qiang.liu@freescale.com> <20120801163553.GC11359@ovro.caltech.edu> In-Reply-To: <20120801163553.GC11359@ovro.caltech.edu> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.192.208.94] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 X-OriginatorOrg: freescale.com Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3872 Lines: 109 > -----Original Message----- > From: Ira W. Snyder [mailto:iws@ovro.caltech.edu] > Sent: Thursday, August 02, 2012 12:36 AM > To: Liu Qiang-B32616 > Cc: linux-crypto@vger.kernel.org; linuxppc-dev@lists.ozlabs.org; linux- > kernel@vger.kernel.org; dan.j.williams@gmail.com; Vinod Koul; > herbert@gondor.hengli.com.au; Dan Williams; davem@davemloft.net > Subject: Re: [PATCH v5 2/6] fsl-dma: remove attribute DMA_INTERRUPT of > dmaengine > > On Wed, Aug 01, 2012 at 04:49:08PM +0800, qiang.liu@freescale.com wrote: > > From: Qiang Liu > > > > Delete attribute DMA_INTERRUPT because fsl-dma doesn't support this > > function, exception will be thrown if talitos is used to offload xor at > the same time. > > > > I have no problem with this patch. > > However, it ***WILL BREAK*** both drivers in drivers/misc/carma. Please > add my patch 7/7 titled "[PATCH 7/7] carma: remove unnecessary > DMA_INTERRUPT capability" to your series. I suggest placing it > immediately after this patch in your series. > > The carma drivers use the fsldma driver exclusively. Fine, thanks. > > > Cc: Dan Williams > > Cc: Vinod Koul > > Cc: Li Yang > > Signed-off-by: Qiang Liu > > Acked-by: Ira W. Snyder > > --- > > drivers/dma/fsldma.c | 31 ------------------------------- > > 1 files changed, 0 insertions(+), 31 deletions(-) > > > > diff --git a/drivers/dma/fsldma.c b/drivers/dma/fsldma.c index > > 8f84761..4f2f212 100644 > > --- a/drivers/dma/fsldma.c > > +++ b/drivers/dma/fsldma.c > > @@ -543,35 +543,6 @@ static void fsl_dma_free_chan_resources(struct > > dma_chan *dchan) } > > > > static struct dma_async_tx_descriptor * > > -fsl_dma_prep_interrupt(struct dma_chan *dchan, unsigned long flags) > > -{ > > - struct fsldma_chan *chan; > > - struct fsl_desc_sw *new; > > - > > - if (!dchan) > > - return NULL; > > - > > - chan = to_fsl_chan(dchan); > > - > > - new = fsl_dma_alloc_descriptor(chan); > > - if (!new) { > > - chan_err(chan, "%s\n", msg_ld_oom); > > - return NULL; > > - } > > - > > - new->async_tx.cookie = -EBUSY; > > - new->async_tx.flags = flags; > > - > > - /* Insert the link descriptor to the LD ring */ > > - list_add_tail(&new->node, &new->tx_list); > > - > > - /* Set End-of-link to the last link descriptor of new list */ > > - set_ld_eol(chan, new); > > - > > - return &new->async_tx; > > -} > > - > > -static struct dma_async_tx_descriptor * fsl_dma_prep_memcpy(struct > > dma_chan *dchan, > > dma_addr_t dma_dst, dma_addr_t dma_src, > > size_t len, unsigned long flags) > > @@ -1352,12 +1323,10 @@ static int __devinit fsldma_of_probe(struct > platform_device *op) > > fdev->irq = irq_of_parse_and_map(op->dev.of_node, 0); > > > > dma_cap_set(DMA_MEMCPY, fdev->common.cap_mask); > > - dma_cap_set(DMA_INTERRUPT, fdev->common.cap_mask); > > dma_cap_set(DMA_SG, fdev->common.cap_mask); > > dma_cap_set(DMA_SLAVE, fdev->common.cap_mask); > > fdev->common.device_alloc_chan_resources = > fsl_dma_alloc_chan_resources; > > fdev->common.device_free_chan_resources = > fsl_dma_free_chan_resources; > > - fdev->common.device_prep_dma_interrupt = fsl_dma_prep_interrupt; > > fdev->common.device_prep_dma_memcpy = fsl_dma_prep_memcpy; > > fdev->common.device_prep_dma_sg = fsl_dma_prep_sg; > > fdev->common.device_tx_status = fsl_tx_status; > > -- > > 1.7.5.1 > > > > > > _______________________________________________ > > Linuxppc-dev mailing list > > Linuxppc-dev@lists.ozlabs.org > > https://lists.ozlabs.org/listinfo/linuxppc-dev -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/