Re: [PATCH v6 09/10] spi: omap2-mcspi: add generic DMA request support to the DT binding

2013-01-30 Thread Arnd Bergmann
On Wednesday 30 January 2013, Matt Porter wrote:
 +Optional properties:
 +- dmas: List of DMA controller phandle and DMA request ordered
 +   pairs. One tx and one rx pair is required for each chip
 +   select.

The binding looks ok, but the wording is slightly incorrect here:
strictly speaking, it's not a pair of controller phandle and request
line number, but a DMA descriptor as specified in bindings/dma/dma.txt
with a format specific to the dma engine being used. That can
require more than just a single integer request number.

Arnd

--
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_jan
___
spi-devel-general mailing list
spi-devel-general@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/spi-devel-general


Re: [PATCH v6 09/10] spi: omap2-mcspi: add generic DMA request support to the DT binding

2013-01-30 Thread Matt Porter
On Wed, Jan 30, 2013 at 09:24:00AM +, Arnd Bergmann wrote:
 On Wednesday 30 January 2013, Matt Porter wrote:
  +Optional properties:
  +- dmas: List of DMA controller phandle and DMA request ordered
  +   pairs. One tx and one rx pair is required for each chip
  +   select.
 
 The binding looks ok, but the wording is slightly incorrect here:
 strictly speaking, it's not a pair of controller phandle and request
 line number, but a DMA descriptor as specified in bindings/dma/dma.txt
 with a format specific to the dma engine being used. That can
 require more than just a single integer request number.

Correct. I will update accordingly.

-Matt

--
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_jan
___
spi-devel-general mailing list
spi-devel-general@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/spi-devel-general