[PATCH V6 2/2] dmaengine: add helper function to request a slave DMA channel

Vinod Koul vinod.koul at linux.intel.com
Mon Sep 17 23:13:55 EDT 2012


On Mon, 2012-09-17 at 23:36 +0100, Russell King - ARM Linux wrote:
> > 
> > I believe that Jon is on vacation this week, so if this is the only issue
> > holding up the merge, maybe you can change this in his patch directly, or
> > I can send an updated version if you prefer.
> 
> I worry that too much is going on here too quickly.  We have some people
> working on changing the way DMA engine selects channels.  Meanwhile we
> have other people trying to create an OF DMA engine API.
> 
> It seems that Vinod's working on a way for platforms to specify bindings
> to the DMA engine code, and the DMA engine code itself selects the
> appropriate channel.  This patch, on the other hand, introduces a set of
> translation functions which need to be provided by platform code,
> which returns the dma_chan pointer.
> 
> This sounds like a recipe for a total abortion of interfaces.  Only one
> of those two activities should be going on at any one time, or if they
> have to occur, they need coordination so that the we don't end up with
> two totally different schemes.
> 
> In the mad rush to DTify everything, don't make hasty decisions, because
> it is very difficult to change it later - especially something like this
> which defines how DT encodes this information.
We discussed this in KS and IMHO we need to merge these two approaches.

For DT bindings, I think the binding itself shouldn't change based on my
work but I would like these same bindings to help build the DMA engine
code mappings.

Now would it make sense to NOT merge these changes for 3.7 and postpone
to 3.8. I can host these patches on a topic branch and merge them when
we are ready. I plan to spend some good amount of time on my work this
week so we should be ready pretty soon.
One these changes are merged, users can start moving to this scheme.


-- 
~Vinod




More information about the linux-arm-kernel mailing list