Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751528AbaJOOKi (ORCPT ); Wed, 15 Oct 2014 10:10:38 -0400 Received: from mga02.intel.com ([134.134.136.20]:28876 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751191AbaJOOKf (ORCPT ); Wed, 15 Oct 2014 10:10:35 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.04,724,1406617200"; d="scan'208";a="589440586" Date: Wed, 15 Oct 2014 19:02:48 +0530 From: Vinod Koul To: Mark Rutland Cc: Srikanth Thokala , "dan.j.williams@intel.com" , "michals@xilinx.com" , "grant.likely@linaro.org" , "robh+dt@kernel.org" , "linux-kernel@vger.kernel.org" , "dmaengine@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "devicetree@vger.kernel.org" , "anirudh@xilinx.com" , "svemula@xilinx.com" , "appanad@xilinx.com" Subject: Re: [PATCH v4] dma: Add Xilinx AXI Direct Memory Access Engine driver support Message-ID: <20141015133248.GQ1638@intel.com> References: <1413374436-14410-1-git-send-email-sthokal@xilinx.com> <20141015124527.GA20034@leverpostej> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20141015124527.GA20034@leverpostej> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Oct 15, 2014 at 01:45:28PM +0100, Mark Rutland wrote: > Hi, > > On Wed, Oct 15, 2014 at 01:00:36PM +0100, Srikanth Thokala wrote: > > This is the driver for the AXI Direct Memory Access (AXI DMA) > > core, which is a soft Xilinx IP core that provides high- > > bandwidth direct memory access between memory and AXI4-Stream > > type target peripherals. > > > > This module works on Zynq (ARM Based SoC) and Microblaze platforms. > > > > Signed-off-by: Srikanth Thokala > > --- > > Changes in v4: > > - Add direction field to VDMA descriptor structure and removed from > > channel structure to avoid duplication. > > - Check for DMA idle condition before changing the configuration. > > - Residue is being calculated in complete_descriptor() and is reported > > to slave driver. > > > > Changes in v3: > > - Rebased on 3.16-rc7 > > > > Changes in v2: > > - Simplified the logic to set SOP and APP words in prep_slave_sg(). > > - Corrected function description comments to match the return type. > > - Fixed some minor comments as suggested by Andy, Thanks. > > This patch has come without the necessary device tree binding document, > and I was not able to find an existing binding document upstream (I > searched for "xlnx,axi-dma-1.00.a" and "axi-dma"). which tree did you grep. This is present in slave-dmaengine.git commit ddc643630f5deb1995d191719086b64873c67a44 Author: Srikanth Thokala Date: Mon Jul 28 17:47:48 2014 +0530 dma: Add Xilinx AXI DMA DT Binding Documentation Device-tree binding documentation of Xilinx DMA Engine Signed-off-by: Srikanth Thokala Acked-by: Arnd Bergmann Signed-off-by: Vinod Koul -- ~Vinod > > The driver expects a non-trivial set of properties, so a binding > document is essential for users. Additionally, checkpatch.pl should > scream regarding the undocumented comaptible string. > > Please put together a binding document. > > Thanks, > Mark. -- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/