From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932522AbcAYMDp (ORCPT ); Mon, 25 Jan 2016 07:03:45 -0500 Received: from mga11.intel.com ([192.55.52.93]:51497 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932071AbcAYMDo (ORCPT ); Mon, 25 Jan 2016 07:03:44 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.22,344,1449561600"; d="scan'208";a="641035281" Date: Mon, 25 Jan 2016 17:37:35 +0530 From: Vinod Koul To: Mans Rullgard Cc: Viresh Kumar , Andy Shevchenko , linux-kernel@vger.kernel.org, dmaengine@vger.kernel.org Subject: Re: [PATCH 00/15] dmaengine: dw: various fixes and cleanups Message-ID: <20160125120735.GI11130@localhost> References: <1453663322-14474-1-git-send-email-mans@mansr.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1453663322-14474-1-git-send-email-mans@mansr.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, Jan 24, 2016 at 07:21:47PM +0000, Mans Rullgard wrote: > This patch series contains a number of mostly minor fixes and cleanups > for the DW DMA driver. A couple of them affect the DT binding so these > may need to be updated to maintain compatibility. The rest should be > relatively straight-forward. > > Andy's branch had two additional patches which I have ommitted here since > one, "dmaengine: dw: clear soft LLP flag in case of error", doesn't look > correct and the other, "dmaengine: dw: split dwc_dostart() helper to two", > depends on the first. > > Andy Shevchenko (11): > dmaengine: dw: rename masters to reflect actual topology > dmaengine: dw: substitute dma_read_byaddr by dma_readl_native > dmaengine: dw: revisit data_width property > dmaengine: dw: define counter variables as unsigned int > dmaengine: dw: keep entire platform data in struct dw_dma > dmaengine: dw: pass platform data via struct dw_dma_chip > dmaengine: dw: platform: use field-by-field initialization > dmaengine: dw: move dwc->paused to dwc->flags > dmaengine: dw: move dwc->initialized to dwc->flags > dmaengine: dw: move residue to a descriptor > dmaengine: dw: set cdesc to NULL when free cyclic transfers > > Mans Rullgard (4): > dmaengine: dw: fix byte order of hw descriptor fields > dmaengine: dw: clear LLP_[SD]_EN bits in last descriptor of a chain > dmaengine: dw: set src and dst master select according to xfer > direction > dmaengine: dw: set LMS field in descriptors > > Documentation/devicetree/bindings/dma/snps-dma.txt | 9 +- > arch/arc/boot/dts/abilis_tb10x.dtsi | 2 +- > arch/arm/boot/dts/spear13xx.dtsi | 4 +- > arch/avr32/mach-at32ap/at32ap700x.c | 16 +- > drivers/ata/sata_dwc_460ex.c | 6 +- > drivers/dma/dw/core.c | 295 ++++++++++----------- > drivers/dma/dw/pci.c | 3 +- > drivers/dma/dw/platform.c | 38 ++- > drivers/dma/dw/regs.h | 55 ++-- > drivers/spi/spi-pxa2xx-pci.c | 8 +- > drivers/tty/serial/8250/8250_pci.c | 8 +- > include/linux/dma/dw.h | 14 +- > include/linux/platform_data/dma-dw.h | 15 +- > sound/soc/intel/common/sst-firmware.c | 2 +- > 14 files changed, 241 insertions(+), 234 deletions(-) Your patch title are also misleading. Only dmaengine code should have tag "dmaengine: dw:" rest should have their own subsystem tags. This will help folks review their changes faster -- ~Vinod