On Wed, Feb 27, 2013 at 08:36:04PM +0000, Arnd Bergmann wrote:

Looks good overall, just one small comment: The generic binding requires both
"dma-channels" and "dma-requests" to be listed here.

Hmm, what I read from Documentation/devicetree/bindings/dma/dma.txt is
that both dma-channels and dma-requests are optional properties.

Shawn

My understanding of this
driver is that each channel is hardwired to one request, so you can mandate here
that the two values must be the same.