Re: [PATCH v5 0/6] dma: Add Xilinx ZynqMP DPDMA driver

From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Vinod Koul <vkoul@kernel.org>
Cc: dmaengine@vger.kernel.org, Michal Simek <michal.simek@xilinx.com>,
	Hyun Kwon <hyun.kwon@xilinx.com>,
	Tejas Upadhyay <tejasu@xilinx.com>,
	Satish Kumar Nagireddy <SATISHNA@xilinx.com>,
	Peter Ujfalusi <peter.ujfalusi@ti.com>,
	Greg KH <gregkh@linuxfoundation.org>
Subject: Re: [PATCH v5 0/6] dma: Add Xilinx ZynqMP DPDMA driver
Date: Tue, 30 Jun 2020 02:39:54 +0300	[thread overview]
Message-ID: <20200629233954.GH21452@pendragon.ideasonboard.com> (raw)
In-Reply-To: <20200629095647.GI2599@vkoul-mobl>

Hi Vinod,

On Mon, Jun 29, 2020 at 03:26:47PM +0530, Vinod Koul wrote:
> On 29-06-20, 12:30, Laurent Pinchart wrote:
> > Hi Vinod,
> > 
> > CC'ing Greg.
> > 
> > The first version of this patch series I've posted dates back from early
> > November last year. Could you please help getting it merged ?
> 
> Sorry for the delay in reviewing this series, this one is on my review
> list for this week, so I should get to this  very soon.

Thank you.

> > On Thu, May 28, 2020 at 05:52:22AM +0300, Laurent Pinchart wrote:
> > > Hello,
> > > 
> > > This patch series adds a new driver for the DPDMA engine found in the
> > > Xilinx ZynqMP.
> > > 
> > > The previous version can be found at [1]. All review comments have been
> > > taken into account. The only change is the addition of the
> > > DMA_PREP_LOAD_EOT transaction flag (and the corresponding DMA_LOAD_EOT
> > > capability bit), as requested during the review of v4. Please see the
> > > discussions from v4 for the rationale.
> > > 
> > > The driver has been successfully tested with the ZynqMP DisplayPort
> > > subsystem DRM driver.
> > > 
> > > [1] https://lore.kernel.org/dmaengine/20200513165943.25120-1-laurent.pinchart@ideasonboard.com/
> > > 
> > > Hyun Kwon (1):
> > >   dmaengine: xilinx: dpdma: Add the Xilinx DisplayPort DMA engine driver
> > > 
> > > Laurent Pinchart (5):
> > >   dt: bindings: dma: xilinx: dpdma: DT bindings for Xilinx DPDMA
> > >   dmaengine: virt-dma: Use lockdep to check locking requirements
> > >   dmaengine: Add support for repeating transactions
> > >   dmaengine: xilinx: dpdma: Add debugfs support
> > >   arm64: dts: zynqmp: Add DPDMA node
> > > 
> > >  .../dma/xilinx/xlnx,zynqmp-dpdma.yaml         |   68 +
> > >  MAINTAINERS                                   |    9 +
> > >  .../arm64/boot/dts/xilinx/zynqmp-clk-ccf.dtsi |    4 +
> > >  arch/arm64/boot/dts/xilinx/zynqmp.dtsi        |   10 +
> > >  drivers/dma/Kconfig                           |   10 +
> > >  drivers/dma/virt-dma.c                        |    2 +
> > >  drivers/dma/virt-dma.h                        |   10 +
> > >  drivers/dma/xilinx/Makefile                   |    1 +
> > >  drivers/dma/xilinx/xilinx_dpdma.c             | 1782 +++++++++++++++++
> > >  include/dt-bindings/dma/xlnx-zynqmp-dpdma.h   |   16 +
> > >  include/linux/dmaengine.h                     |   19 +
> > >  11 files changed, 1931 insertions(+)
> > >  create mode 100644 Documentation/devicetree/bindings/dma/xilinx/xlnx,zynqmp-dpdma.yaml
> > >  create mode 100644 drivers/dma/xilinx/xilinx_dpdma.c
> > >  create mode 100644 include/dt-bindings/dma/xlnx-zynqmp-dpdma.h

-- 
Regards,

Laurent Pinchart