From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934216AbcKPPTF (ORCPT ); Wed, 16 Nov 2016 10:19:05 -0500 Received: from mga07.intel.com ([134.134.136.100]:36563 "EHLO mga07.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752532AbcKPPTD (ORCPT ); Wed, 16 Nov 2016 10:19:03 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.31,500,1473145200"; d="scan'208";a="1060228714" Message-ID: <1479309538.24056.49.camel@linux.intel.com> Subject: Re: [PATCH 0/4] DW DMAC: update device tree From: Andy Shevchenko To: Eugeniy Paltsev , devicetree@vger.kernel.org Cc: robh+dt@kernel.org, mark.rutland@arm.com, linux-kernel@vger.kernel.org, vireshk@kernel.org, dan.j.williams@intel.com, vinod.koul@intel.com, dmaengine@vger.kernel.org, linux-snps-arc@lists.infradead.org Date: Wed, 16 Nov 2016 17:18:58 +0200 In-Reply-To: <1479304615-11360-1-git-send-email-Eugeniy.Paltsev@synopsys.com> References: <1479304615-11360-1-git-send-email-Eugeniy.Paltsev@synopsys.com> Organization: Intel Finland Oy Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.22.2-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 On Wed, 2016-11-16 at 16:56 +0300, Eugeniy Paltsev wrote: > It wasn't possible to enable some features like > memory-to-memory transfers or multi block transfers via DT. > It is fixed by these patches. > >  * Rename is_private to is-private as ordered by DT policy. >  (just for cleanup) The change leaves the support for the  >  old format. > >  * Add is-memcpu property, so it is possible to >  enable memory-to-memory transfers support via DT. > >  * Add hw-llp property, so it is possible to enable >  hardware multi block transfers support via DT. > >  * Update DW DMAC device tree documentation. I have few comments I posted. Besides that don't forget about current users of the DT properties you standardized (by naming). Better you convert them at the same time. Older DT (blobs) are being still supported. Otherwise looks okay after you address all my comments and maybe others will do some. The DT people ACK is a must before this goes somewhere. Also, please keep Cc list as small as possible. For example I'm not sure Viresh has time to look at them, but he might keep an eye on the dmaengine mailing list. Same about Dan. > > Eugeniy Paltsev (4): >   DW DMAC: rename is_private property as ordered by DT policy >   DW DMAC: add is-memcpu property to device tree >   DW DMAC: add hw-llp property to device tree >   Update device tree Synopsys DW DMAC documentation > >  Documentation/devicetree/bindings/dma/snps-dma.txt | 10 ++++++++-- >  drivers/dma/dw/core.c                              |  2 +- >  drivers/dma/dw/platform.c                          | 10 ++++++++++ >  include/linux/platform_data/dma-dw.h               |  4 ++-- >  4 files changed, 21 insertions(+), 5 deletions(-) > -- Andy Shevchenko Intel Finland Oy