diff for duplicates of <1493143941.24567.196.camel@linux.intel.com> diff --git a/a/1.txt b/N1/1.txt index 14a32b7..c8d0393 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,11 +1,11 @@ -On Tue, 2017-04-25@15:16 +0000, Eugeniy Paltsev wrote: -> On Mon, 2017-04-24@19:56 +0300, Andy Shevchenko wrote: -> > On Mon, 2017-04-24@15:55 +0000, Eugeniy Paltsev wrote: +On Tue, 2017-04-25 at 15:16 +0000, Eugeniy Paltsev wrote: +> On Mon, 2017-04-24 at 19:56 +0300, Andy Shevchenko wrote: +> > On Mon, 2017-04-24 at 15:55 +0000, Eugeniy Paltsev wrote: > > > Hi, -> > > On Fri, 2017-04-21@18:13 +0300, Andy Shevchenko wrote: -> > > > On Fri, 2017-04-21@14:29 +0000, Eugeniy Paltsev wrote: -> > > > > On Tue, 2017-04-18@15:31 +0300, Andy Shevchenko wrote: -> > > > > > On Fri, 2017-04-07@17:04 +0300, Eugeniy Paltsev wrote: +> > > On Fri, 2017-04-21 at 18:13 +0300, Andy Shevchenko wrote: +> > > > On Fri, 2017-04-21 at 14:29 +0000, Eugeniy Paltsev wrote: +> > > > > On Tue, 2017-04-18 at 15:31 +0300, Andy Shevchenko wrote: +> > > > > > On Fri, 2017-04-07 at 17:04 +0300, Eugeniy Paltsev wrote: > > > This IP can be (ans is) configured with small block size. > > > (note, that I am not saying about runtime HW configuration) @@ -62,7 +62,7 @@ UART case where buffer is small enough and latency like above matters). > If we want to use same descriptors several times we just can use > DMA_CTRL_REUSE option - the descriptors will be created one time and -> re-splitting will be ?ompleted only one time. +> re-splitting will be сompleted only one time. There are two type of descriptors: SW and HW. That flag about SW descriptor, so, it in most cases has nothing to do with the actual entry @@ -100,14 +100,14 @@ transfer and in one place where the sg-list is created. > > See below. > > > > > I checked hsu/hsu.c dma driver implementation: -> > > ???vdma descriptor is deleted from desc_issued list when transfer -> > > ???starts. When descriptor marked as error descriptor -> > > ???vchan_cookie_complete isn't called for this descriptor. And +> > > vdma descriptor is deleted from desc_issued list when transfer +> > > starts. When descriptor marked as error descriptor +> > > vchan_cookie_complete isn't called for this descriptor. And > > > this -> > > ???descriptor isn't placed in any list. So error descriptors +> > > descriptor isn't placed in any list. So error descriptors > > > *never* -> > > ???will be freed. -> > > ???I don't actually like this approach. +> > > will be freed. +> > > I don't actually like this approach. > > > > Descriptor is active until terminate_all() is called or new > > descriptor @@ -161,5 +161,5 @@ Also I would reconsider clock (un)preparation in those hooks. Why do you need that part? It's resource consuming. -- -Andy Shevchenko <andriy.shevchenko at linux.intel.com> +Andy Shevchenko <andriy.shevchenko@linux.intel.com> Intel Finland Oy diff --git a/a/content_digest b/N1/content_digest index 19514b4..22f1457 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -6,20 +6,28 @@ "ref\01493049305.25985.4.camel@synopsys.com\0" "ref\01493052970.24567.168.camel@linux.intel.com\0" "ref\01493133369.25985.10.camel@synopsys.com\0" - "From\0andriy.shevchenko@linux.intel.com (Andy Shevchenko)\0" - "Subject\0[PATCH v2 2/2] dmaengine: Add DW AXI DMAC driver\0" + "From\0Andy Shevchenko <andriy.shevchenko@linux.intel.com>\0" + "Subject\0Re: [PATCH v2 2/2] dmaengine: Add DW AXI DMAC driver\0" "Date\0Tue, 25 Apr 2017 21:12:21 +0300\0" - "To\0linux-snps-arc@lists.infradead.org\0" + "To\0Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com>\0" + "Cc\0vinod.koul@intel.com <vinod.koul@intel.com>" + linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org> + robh+dt@kernel.org <robh+dt@kernel.org> + Alexey.Brodkin@synopsys.com <Alexey.Brodkin@synopsys.com> + devicetree@vger.kernel.org <devicetree@vger.kernel.org> + linux-snps-arc@lists.infradead.org <linux-snps-arc@lists.infradead.org> + dan.j.williams@intel.com <dan.j.williams@intel.com> + " dmaengine@vger.kernel.org <dmaengine@vger.kernel.org>\0" "\00:1\0" "b\0" - "On Tue, 2017-04-25@15:16 +0000, Eugeniy Paltsev wrote:\n" - "> On Mon, 2017-04-24@19:56 +0300, Andy Shevchenko wrote:\n" - "> > On Mon, 2017-04-24@15:55 +0000, Eugeniy Paltsev wrote:\n" + "On Tue, 2017-04-25 at 15:16 +0000, Eugeniy Paltsev wrote:\n" + "> On Mon, 2017-04-24 at 19:56 +0300, Andy Shevchenko wrote:\n" + "> > On Mon, 2017-04-24 at 15:55 +0000, Eugeniy Paltsev wrote:\n" "> > > Hi,\n" - "> > > On Fri, 2017-04-21@18:13 +0300, Andy Shevchenko wrote:\n" - "> > > > On Fri, 2017-04-21@14:29 +0000, Eugeniy Paltsev wrote:\n" - "> > > > > On Tue, 2017-04-18@15:31 +0300, Andy Shevchenko wrote:\n" - "> > > > > > On Fri, 2017-04-07@17:04 +0300, Eugeniy Paltsev wrote:\n" + "> > > On Fri, 2017-04-21 at 18:13 +0300, Andy Shevchenko wrote:\n" + "> > > > On Fri, 2017-04-21 at 14:29 +0000, Eugeniy Paltsev wrote:\n" + "> > > > > On Tue, 2017-04-18 at 15:31 +0300, Andy Shevchenko wrote:\n" + "> > > > > > On Fri, 2017-04-07 at 17:04 +0300, Eugeniy Paltsev wrote:\n" "\n" "> > > This IP can be (ans is) configured with small block size.\n" "> > > (note, that I am not saying about runtime HW configuration)\n" @@ -76,7 +84,7 @@ "\n" "> If we want to use same descriptors several times we just can use\n" "> DMA_CTRL_REUSE option - the descriptors will be created one time and\n" - "> re-splitting will be ?ompleted only one time.\n" + "> re-splitting will be \321\201ompleted only one time.\n" "\n" "There are two type of descriptors: SW and HW. That flag about SW\n" "descriptor, so, it in most cases has nothing to do with the actual entry\n" @@ -114,14 +122,14 @@ "> > See below.\n" "> > \n" "> > > I checked hsu/hsu.c dma driver implementation:\n" - "> > > ???vdma descriptor is deleted from desc_issued list when transfer\n" - "> > > ???starts. When descriptor marked as error descriptor\n" - "> > > ???vchan_cookie_complete isn't called for this descriptor. And\n" + "> > > \302\240\302\240\302\240vdma descriptor is deleted from desc_issued list when transfer\n" + "> > > \302\240\302\240\302\240starts. When descriptor marked as error descriptor\n" + "> > > \302\240\302\240\302\240vchan_cookie_complete isn't called for this descriptor. And\n" "> > > this\n" - "> > > ???descriptor isn't placed in any list. So error descriptors\n" + "> > > \302\240\302\240\302\240descriptor isn't placed in any list. So error descriptors\n" "> > > *never*\n" - "> > > ???will be freed.\n" - "> > > ???I don't actually like this approach.\n" + "> > > \302\240\302\240\302\240will be freed.\n" + "> > > \302\240\302\240\302\240I don't actually like this approach.\n" "> > \n" "> > Descriptor is active until terminate_all() is called or new\n" "> > descriptor\n" @@ -175,7 +183,7 @@ "need that part? It's resource consuming.\n" "\n" "-- \n" - "Andy Shevchenko <andriy.shevchenko at linux.intel.com>\n" + "Andy Shevchenko <andriy.shevchenko@linux.intel.com>\n" Intel Finland Oy -c5887346e69cdd85e5aca0cb18f2feb13e2caabea741a5cbba44dffe49d74335 +30de22e7d264cef2322cd59e6cea6c9fda10abc94af2de8dd448b8674b275fd9
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.