From mboxrd@z Thu Jan 1 00:00:00 1970 From: Richard Zhao Subject: Re: [PATCH 3/9] spi: tegra114: move to generic dma DT binding Date: Tue, 30 Jul 2013 11:15:36 +0800 Message-ID: <20130730031536.GK15855@rizhao-lap> References: <1374639002-16753-1-git-send-email-rizhao@nvidia.com> <1374639002-16753-4-git-send-email-rizhao@nvidia.com> <51F2CF53.30002@wwwdotorg.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Return-path: In-Reply-To: <51F2CF53.30002@wwwdotorg.org> Content-Disposition: inline Sender: linux-serial-owner@vger.kernel.org To: Stephen Warren Cc: "devicetree@vger.kernel.org" , Mark Brown , "linux-tegra@vger.kernel.org" , Ian Campbell , "linux-arm-kernel@lists.infradead.org" , Mark Rutland , "linux-serial@vger.kernel.org" , Pawel Moll , "rob.herring@calxeda.com" , "vinod.koul@intel.com" , "djbw@fb.com" List-Id: linux-tegra@vger.kernel.org On Sat, Jul 27, 2013 at 03:34:43AM +0800, Stephen Warren wrote: > (Stripping CC list a bit) > > On 07/23/2013 10:09 PM, Richard Zhao wrote: > > - driver: remove use of nvidia,dma-request-selector > > use dma_request_slave_channel to request channel > > - if dmas/dma-names are missing, it still supports cpu based transfer > > - update binding doc and specify dmas/dma-names properties as optional > > I'll simply repeat the comments I already made downstream: > > Most commit messages aren't written as bullet points. Alright, I'll change it. (Personally I like bullet points). > This might be > better written as: > > ---------- > Convert the Tegra114 SPI driver to support the standard DMA device-tree > bindings. > > Note that DMA mode is optional; if the DT properties required to support > DMA are missing, then the driver will fall back to pure CPU operation. > ---------- > > > diff --git a/Documentation/devicetree/bindings/spi/nvidia,tegra114-spi.txt b/Documentation/devicetree/bindings/spi/nvidia,tegra114-spi.txt > > > +Optional properties: > > +- dmas : The Tegra DMA controller's phandle and request selector for > > + this SPI controller. > > +- dma-names : Should be "rx-tx". > > This should reference dma.txt which defines the DMA DT bindings. How about: > > ---------- > Optional properties: > - dmas, dma-names : The DMA channel to use for DMA-mode operation. See > dma.txt for the definition of these properties. A single entry named > "rx-tx" must be specified. > ---------- Ok. but how about replace "DMA channel" with "DMA request"? Thanks Richard