Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752170AbcKHNgq (ORCPT ); Tue, 8 Nov 2016 08:36:46 -0500 Received: from mga03.intel.com ([134.134.136.65]:49086 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751980AbcKHNgk (ORCPT ); Tue, 8 Nov 2016 08:36:40 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.31,462,1473145200"; d="scan'208";a="1185302" Message-ID: <1478612190.5295.92.camel@linux.intel.com> Subject: Re: [PATCH v3 0/3] dmaengine: DW DMAC: split pdata to hardware properties From: Andy Shevchenko To: Eugeniy Paltsev Cc: "dan.j.williams@intel.com" , "linux-kernel@vger.kernel.org" , "dmaengine@vger.kernel.org" , "vinod.koul@intel.com" , "vireshk@kernel.org" , "linux-snps-arc@lists.infradead.org" Date: Tue, 08 Nov 2016 15:36:30 +0200 In-Reply-To: <1478607771.2603.31.camel@synopsys.com> References: <1477670402-23943-1-git-send-email-Eugeniy.Paltsev@synopsys.com> <1478087707.2603.7.camel@synopsys.com> <1478526908.5295.67.camel@linux.intel.com> <1478607771.2603.31.camel@synopsys.com> Organization: Intel Finland Oy Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.22.1-1 Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1731 Lines: 59 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 > > > device > > > tree > > > + * node (if exists) or from hardware autoconfig registers. > > > > Can you somehow be more clear that all listed quirks will be copied > > from > > platform data. > > See comment below. > > >   > > >   > > >    * @is_nollp: The device channels does not support multi block > > > transfers. > > >    * @chan_allocation_order: Allocate channels starting from 0 or > > > 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; > > >   > > > + bool only_quirks_used; > > > > 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 > 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") > > So, in PATCH v3 series "is_nollp" used as regular pdata field. I still would consider is_nollp as a quirk since nothing prevents to override the hardware value (see Intel Quark case). -- Andy Shevchenko Intel Finland Oy