diff for duplicates of <1478612190.5295.92.camel@linux.intel.com> diff --git a/a/1.txt b/N1/1.txt index f6c96ac..265c77e 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,6 +1,6 @@ -On Tue, 2016-11-08@12:22 +0000, Eugeniy Paltsev wrote: -> On Mon, 2016-11-07@15:55 +0200, Andy Shevchenko wrote: -? +On Tue, 2016-11-08 at 12:22 +0000, Eugeniy Paltsev wrote: +> On Mon, 2016-11-07 at 15:55 +0200, Andy Shevchenko wrote: + > > > + * @only_quirks_used: Only read quirks (like "is_private" or > > > "is_memcpy") from > > > + * platform data structure. Read other parameters from @@ -12,39 +12,39 @@ On Tue, 2016-11-08@12:22 +0000, Eugeniy Paltsev wrote: > > from > > platform data. > -> See?comment below. +> See comment below. > -> > ? -> > > ? -> > > ???* @is_nollp: The device channels does not support multi block +> > +> > > +> > > * @is_nollp: The device channels does not support multi block > > > transfers. -> > > ???* @chan_allocation_order: Allocate channels starting from 0 or +> > > * @chan_allocation_order: Allocate channels starting from 0 or > > > 7 -> > > ???* @chan_priority: Set channel priority increasing from 0 to 7 +> > > * @chan_priority: Set channel priority increasing from 0 to 7 > > > or > > > 7 > > > to 0. > > > @@ -52,6 +55,7 @@ struct dw_dma_platform_data { -> > > ?? unsigned int nr_channels; -> > > ?? bool is_private; -> > > ?? bool is_memcpy; -> > > ? +> > > unsigned int nr_channels; +> > > bool is_private; +> > > bool is_memcpy; +> > > > > > + bool only_quirks_used; > > -> > Perhaps add if at the end of quirk list and name just? -> > ? -> > > ? -> > > ?? bool is_nollp; +> > Perhaps add if at the end of quirk list and name just +> > +> > > +> > > bool is_nollp; > > > > ...here -> > ? +> > > > bool use_quirks; What do think about shorten name? > > I don't treat "is_nollp" as quirks like "is_private" or "is_memcpy". -> It is like general pdata field: we can easily?read it from autoconfig +> It is like general pdata field: we can easily read it from autoconfig > registers (and we don't have any problem with that) in case of > pdata/device-tree absence (as opposed to quirks like "is_private" or > "is_memcpy") @@ -55,5 +55,5 @@ I still would consider is_nollp as a quirk since nothing prevents to override the hardware value (see Intel Quark case). -- -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 ea7b71c..6152aa7 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -2,15 +2,21 @@ "ref\01478087707.2603.7.camel@synopsys.com\0" "ref\01478526908.5295.67.camel@linux.intel.com\0" "ref\01478607771.2603.31.camel@synopsys.com\0" - "From\0andriy.shevchenko@linux.intel.com (Andy Shevchenko)\0" - "Subject\0[PATCH v3 0/3] dmaengine: DW DMAC: split pdata to hardware properties\0" + "From\0Andy Shevchenko <andriy.shevchenko@linux.intel.com>\0" + "Subject\0Re: [PATCH v3 0/3] dmaengine: DW DMAC: split pdata to hardware properties\0" "Date\0Tue, 08 Nov 2016 15:36:30 +0200\0" - "To\0linux-snps-arc@lists.infradead.org\0" + "To\0Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com>\0" + "Cc\0dan.j.williams@intel.com <dan.j.williams@intel.com>" + linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org> + dmaengine@vger.kernel.org <dmaengine@vger.kernel.org> + vinod.koul@intel.com <vinod.koul@intel.com> + vireshk@kernel.org <vireshk@kernel.org> + " linux-snps-arc@lists.infradead.org <linux-snps-arc@lists.infradead.org>\0" "\00:1\0" "b\0" - "On Tue, 2016-11-08@12:22 +0000, Eugeniy Paltsev wrote:\n" - "> On Mon, 2016-11-07@15:55 +0200, Andy Shevchenko wrote:\n" - "?\n" + "On Tue, 2016-11-08 at 12:22 +0000, Eugeniy Paltsev wrote:\n" + "> On Mon, 2016-11-07 at 15:55 +0200, Andy Shevchenko wrote:\n" + "\302\240\n" "> > > + * @only_quirks_used: Only read quirks (like \"is_private\" or\n" "> > > \"is_memcpy\") from\n" "> > > + *\tplatform data structure. Read other parameters from\n" @@ -22,39 +28,39 @@ "> > from\n" "> > platform data.\n" "> \n" - "> See?comment below.\n" + "> See\302\240comment below.\n" "> \n" - "> > ?\n" - "> > > ?\n" - "> > > ???* @is_nollp: The device channels does not support multi block\n" + "> > \302\240\n" + "> > > \302\240\n" + "> > > \302\240\302\240\302\240* @is_nollp: The device channels does not support multi block\n" "> > > transfers.\n" - "> > > ???* @chan_allocation_order: Allocate channels starting from 0 or\n" + "> > > \302\240\302\240\302\240* @chan_allocation_order: Allocate channels starting from 0 or\n" "> > > 7\n" - "> > > ???* @chan_priority: Set channel priority increasing from 0 to 7\n" + "> > > \302\240\302\240\302\240* @chan_priority: Set channel priority increasing from 0 to 7\n" "> > > or\n" "> > > 7\n" "> > > to 0.\n" "> > > @@ -52,6 +55,7 @@ struct dw_dma_platform_data {\n" - "> > > ??\tunsigned int\tnr_channels;\n" - "> > > ??\tbool\t\tis_private;\n" - "> > > ??\tbool\t\tis_memcpy;\n" - "> > > ?\n" + "> > > \302\240\302\240\tunsigned int\tnr_channels;\n" + "> > > \302\240\302\240\tbool\t\tis_private;\n" + "> > > \302\240\302\240\tbool\t\tis_memcpy;\n" + "> > > \302\240\n" "> > > +\tbool\t\tonly_quirks_used;\n" "> > \n" - "> > Perhaps add if at the end of quirk list and name just?\n" - "> > ?\n" - "> > > ?\n" - "> > > ??\tbool\t\tis_nollp;\n" + "> > Perhaps add if at the end of quirk list and name just\302\240\n" + "> > \302\240\n" + "> > > \302\240\n" + "> > > \302\240\302\240\tbool\t\tis_nollp;\n" "> > \n" "> > ...here\n" - "> > ?\n" + "> > \302\240\n" "> > bool use_quirks;\n" "\n" "What do think about shorten name?\n" "\n" "> \n" "> I don't treat \"is_nollp\" as quirks like \"is_private\" or \"is_memcpy\".\n" - "> It is like general pdata field: we can easily?read it from autoconfig\n" + "> It is like general pdata field: we can easily\302\240read it from autoconfig\n" "> registers (and we don't have any problem with that) in case of\n" "> pdata/device-tree absence (as opposed to quirks like \"is_private\" or\n" "> \"is_memcpy\")\n" @@ -65,7 +71,7 @@ "override the hardware value (see Intel Quark case).\n" "\n" "-- \n" - "Andy Shevchenko <andriy.shevchenko at linux.intel.com>\n" + "Andy Shevchenko <andriy.shevchenko@linux.intel.com>\n" Intel Finland Oy -5f8105238434ae859400d01c91b91f0082b3d540f61e3123c0959fb03fd29531 +5358740389b0e2b165c23b83d663c9d01403fd19d369707a936967f13f586664
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.