diff for duplicates of <1493738176.25985.23.camel@synopsys.com> diff --git a/a/1.txt b/N1/1.txt index 26ae0d3..51cba4e 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,7 +1,7 @@ Hi Vinod, -On Mon, 2017-05-01@11:21 +0530, Vinod Koul wrote: -> On Fri, Apr 28, 2017@04:37:46PM +0300, Eugeniy Paltsev wrote: +On Mon, 2017-05-01 at 11:21 +0530, Vinod Koul wrote: +> On Fri, Apr 28, 2017 at 04:37:46PM +0300, Eugeniy Paltsev wrote: > > In the current implementation dma_get_slave_caps is used to check > > state of descriptor_reuse option. But dma_get_slave_caps includes > > check if the channel supports slave transactions. @@ -32,10 +32,10 @@ And, of course, we can pass DMA_CTRL_REUSE flag to device_prep_dma_sg or device_prep_dma_memcpy directly without checks. > > -> > Signed-off-by: Eugeniy Paltsev <Eugeniy.Paltsev at synopsys.com> +> > Signed-off-by: Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com> > > --- -> > ?include/linux/dmaengine.h | 6 +----- -> > ?1 file changed, 1 insertion(+), 5 deletions(-) +> > include/linux/dmaengine.h | 6 +----- +> > 1 file changed, 1 insertion(+), 5 deletions(-) > > > > diff --git a/include/linux/dmaengine.h b/include/linux/dmaengine.h > > index 5336808..92cf8b0 100644 @@ -43,22 +43,22 @@ or device_prep_dma_memcpy directly without checks. > > +++ b/include/linux/dmaengine.h > > @@ -1376,11 +1376,7 @@ static inline int dma_get_slave_caps(struct > > dma_chan *chan, -> > ? -> > ?static inline int dmaengine_desc_set_reuse(struct +> > +> > static inline int dmaengine_desc_set_reuse(struct > > dma_async_tx_descriptor *tx) -> > ?{ +> > { > > - struct dma_slave_caps caps; > > - > > - dma_get_slave_caps(tx->chan, &caps); > > - > > - if (caps.descriptor_reuse) { > > + if (tx->chan->device->descriptor_reuse) { -> > ? tx->flags |= DMA_CTRL_REUSE; -> > ? return 0; -> > ? } else { -> > --? +> > tx->flags |= DMA_CTRL_REUSE; +> > return 0; +> > } else { +> > -- > > 2.9.3 > > > > -- -- -?Eugeniy Paltsev + Eugeniy Paltsev diff --git a/a/content_digest b/N1/content_digest index a6e027e..13ef2a0 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,15 +1,22 @@ "ref\020170428133746.1855-1-Eugeniy.Paltsev@synopsys.com\0" "ref\020170501055125.GN6263@localhost\0" - "From\0Eugeniy.Paltsev@synopsys.com (Eugeniy Paltsev)\0" - "Subject\0[PATCH] Allow to use DMA_CTRL_REUSE flag for all channel types\0" + "From\0Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com>\0" + "Subject\0Re: [PATCH] Allow to use DMA_CTRL_REUSE flag for all channel types\0" "Date\0Tue, 2 May 2017 15:16:18 +0000\0" - "To\0linux-snps-arc@lists.infradead.org\0" + "To\0vinod.koul@intel.com <vinod.koul@intel.com>\0" + "Cc\0dmaengine@vger.kernel.org <dmaengine@vger.kernel.org>" + linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org> + andriy.shevchenko@linux.intel.com <andriy.shevchenko@linux.intel.com> + dan.j.williams@intel.com <dan.j.williams@intel.com> + Alexey.Brodkin@synopsys.com <Alexey.Brodkin@synopsys.com> + Eugeniy.Paltsev@synopsys.com <Eugeniy.Paltsev@synopsys.com> + " linux-snps-arc@lists.infradead.org <linux-snps-arc@lists.infradead.org>\0" "\00:1\0" "b\0" "Hi Vinod,\n" "\n" - "On Mon, 2017-05-01@11:21 +0530, Vinod Koul wrote:\n" - "> On Fri, Apr 28, 2017@04:37:46PM +0300, Eugeniy Paltsev wrote:\n" + "On Mon, 2017-05-01 at 11:21 +0530, Vinod Koul wrote:\n" + "> On Fri, Apr 28, 2017 at 04:37:46PM +0300, Eugeniy Paltsev wrote:\n" "> > In the current implementation dma_get_slave_caps is used to check\n" "> > state of descriptor_reuse option. But dma_get_slave_caps includes\n" "> > check if the channel supports slave transactions.\n" @@ -40,10 +47,10 @@ "or device_prep_dma_memcpy directly without checks.\n" "\n" "> > \n" - "> > Signed-off-by: Eugeniy Paltsev <Eugeniy.Paltsev at synopsys.com>\n" + "> > Signed-off-by: Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com>\n" "> > ---\n" - "> > ?include/linux/dmaengine.h | 6 +-----\n" - "> > ?1 file changed, 1 insertion(+), 5 deletions(-)\n" + "> > \302\240include/linux/dmaengine.h | 6 +-----\n" + "> > \302\2401 file changed, 1 insertion(+), 5 deletions(-)\n" "> > \n" "> > diff --git a/include/linux/dmaengine.h b/include/linux/dmaengine.h\n" "> > index 5336808..92cf8b0 100644\n" @@ -51,24 +58,24 @@ "> > +++ b/include/linux/dmaengine.h\n" "> > @@ -1376,11 +1376,7 @@ static inline int dma_get_slave_caps(struct\n" "> > dma_chan *chan,\n" - "> > ?\n" - "> > ?static inline int dmaengine_desc_set_reuse(struct\n" + "> > \302\240\n" + "> > \302\240static inline int dmaengine_desc_set_reuse(struct\n" "> > dma_async_tx_descriptor *tx)\n" - "> > ?{\n" + "> > \302\240{\n" "> > -\tstruct dma_slave_caps caps;\n" "> > -\n" "> > -\tdma_get_slave_caps(tx->chan, &caps);\n" "> > -\n" "> > -\tif (caps.descriptor_reuse) {\n" "> > +\tif (tx->chan->device->descriptor_reuse) {\n" - "> > ?\t\ttx->flags |= DMA_CTRL_REUSE;\n" - "> > ?\t\treturn 0;\n" - "> > ?\t} else {\n" - "> > --?\n" + "> > \302\240\t\ttx->flags |= DMA_CTRL_REUSE;\n" + "> > \302\240\t\treturn 0;\n" + "> > \302\240\t} else {\n" + "> > --\302\240\n" "> > 2.9.3\n" "> > \n" "> > --\n" "-- \n" - ?Eugeniy Paltsev + "\302\240Eugeniy Paltsev" -23cddbd5b71759cbaa8dd846c155babe03077618e344a901d6648613f23bad25 +4cb21672ef0910e73bea2b294efc060740bbb8388075ca23c10069731017c794
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.