Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753137AbcKGNzQ (ORCPT ); Mon, 7 Nov 2016 08:55:16 -0500 Received: from mga01.intel.com ([192.55.52.88]:65499 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751296AbcKGNzM (ORCPT ); Mon, 7 Nov 2016 08:55:12 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.31,606,1473145200"; d="scan'208";a="621521" Message-ID: <1478526908.5295.67.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" , "vireshk@kernel.org" , "vinod.koul@intel.com" , "linux-snps-arc@lists.infradead.org" Date: Mon, 07 Nov 2016 15:55:08 +0200 In-Reply-To: <1478087707.2603.7.camel@synopsys.com> References: <1477670402-23943-1-git-send-email-Eugeniy.Paltsev@synopsys.com> <1478087707.2603.7.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: 1970 Lines: 51 On Wed, 2016-11-02 at 11:55 +0000, Eugeniy Paltsev wrote: > Hi Andy, > Could you possibly tell me your ideas about these changes? > Thanks. Thanks for an update, but, please, answer to all my comments to your patch v2. Either you are okay with them, then you didn't address few, or you are not okay, I didn't get why. Deffer newer version until we get an agreement on the implementation. > > On Fri, 2016-10-28 at 18:59 +0300, Eugeniy Paltsev wrote: > > This series is to address a proposal by Andy in these threads: > > http://www.spinics.net/lists/dmaengine/msg11506.html > > http://www.spinics.net/lists/dmaengine/msg11541.html > > Split platform data to actual hardware properties, and platform > > quirks. > > Now we able to use quirks and hardware properties separately from > > different sources (pdata, device tree or autoconfig registers) > > > > Changes for v3: > >  -   Split changes to separate patches. > >  -   Add "DW_DMA_IS_LLP_SUPPORTED" flag and get rid of "dwc->nollp" > >      as separate variable. > >  -   Make "dw_dma_slave" documentation comments more clear about > > quirks. > >      "is_memcpy" and "is_private" are quirks, "is_nollp" is regular  > >      pdata property. > > > > Eugeniy Paltsev (3): > >   dmaengine: DW DMAC: split pdata to hardware properties and > > platform > >     quirks > >   dmaengine: DW DMAC: convert to unified device property API > >   dmaengine: DW DMAC: move "nollp" to "dwc->flags" > > > >  drivers/dma/dw/core.c                | 34 +++++++++++++---------- > >  drivers/dma/dw/platform.c            | 53 +++++++++++++++++++---- > > --- > > ---------- > >  drivers/dma/dw/regs.h                |  2 +- > >  include/linux/platform_data/dma-dw.h |  5 ++++ > >  4 files changed, 54 insertions(+), 40 deletions(-) > > > > --  >  Paltsev Eugeniy -- Andy Shevchenko Intel Finland Oy