Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751515AbdF1Grt (ORCPT ); Wed, 28 Jun 2017 02:47:49 -0400 Received: from mail-oi0-f42.google.com ([209.85.218.42]:34332 "EHLO mail-oi0-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751453AbdF1Grk (ORCPT ); Wed, 28 Jun 2017 02:47:40 -0400 Date: Wed, 28 Jun 2017 01:47:38 -0500 From: Andy Gross To: Rob Herring Cc: Varadarajan Narayanan , broonie@kernel.org, mark.rutland@arm.com, david.brown@linaro.org, linux-spi@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org, Abhishek Sahu Subject: Re: [PATCH v3 15/15] spi: qup: support for qup v1 dma Message-ID: <20170628064738.GD5203@hector.attlocal.net> References: <1497949857-1852-1-git-send-email-varada@codeaurora.org> <1497949857-1852-16-git-send-email-varada@codeaurora.org> <20170623214923.ratchkoay73ytlv3@rob-hp-laptop> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170623214923.ratchkoay73ytlv3@rob-hp-laptop> User-Agent: Mutt/1.5.23.1 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1837 Lines: 40 On Fri, Jun 23, 2017 at 04:49:23PM -0500, Rob Herring wrote: > On Tue, Jun 20, 2017 at 02:40:57PM +0530, Varadarajan Narayanan wrote: > > Currently the QUP Version v1 does not work with DMA so added > > the support for the same. > > > > 1. It uses ADM DMA which requires TX and RX CRCI > > 2. DMA channel initialization need to be done after setting > > block size for having valid values in maxburst > > 3. QUP mode should be DMOV instead of BAM. > > > > Signed-off-by: Abhishek Sahu > > Signed-off-by: Varadarajan Narayanan > > --- > > .../devicetree/bindings/spi/qcom,spi-qup.txt | 6 ++++ > > drivers/spi/spi-qup.c | 35 +++++++++++++++++----- > > 2 files changed, 34 insertions(+), 7 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/spi/qcom,spi-qup.txt b/Documentation/devicetree/bindings/spi/qcom,spi-qup.txt > > index 5c09077..e754181 100644 > > --- a/Documentation/devicetree/bindings/spi/qcom,spi-qup.txt > > +++ b/Documentation/devicetree/bindings/spi/qcom,spi-qup.txt > > @@ -38,6 +38,12 @@ Optional properties: > > - dma-names: Names for the dma channels, if present. There must be at > > least one channel named "tx" for transmit and named "rx" for > > receive. > > +- qcom,tx-crci: Identificator for Client Rate Control Interface (CRCI) to be > > Identificator is not a word. > > This sounds like something that should be a cell in the dmas property. Yes, the CRCI should be part of the dma cells for the ADM. That would make a channel + crci work as a virtual channel that is backed by a hardware channel. The only thing that has to be dealt with is the protocol difference between the BAM and ADM dma blocks, which is what the v1 compatible tells us. Regards, Andy