Hello. Kevin Hilman wrote:
Some of the comments about my earlier EDMA patches touched on issues in that programming interface, like:
- The single call to allocate DMA resources is overly complex.
- Its programming model doesn't match the hardware well: talking about master vs. slave, not channels and parameter RAM; confusing those two resource types (especially when allocating); etc.
- Since the calls used a "davinci_" prefix, they wouldn't be very appropriate for the DMA in the OMAP-L137 chip.
We were going to move the generic part of arch/arm/mach-davinci/dma.c (alomg with other common code b/w so called OMAP-L1x and DaVinci) to arch/arm/plat-davinci/ but the rename seems reasonable anyway.
I keep hearing things like this, but have not yet seen any patches, or technical arguments for doing so.
The technical argument is simple: sharing the code for two similar platforms, the EDMA code in particular.
I am planning a push of the core davinci code for the next merge window, so if changes like that are expected to go in, I expect to see some patches and arguments *very* soon.
And we're going to start submitting RSN.
Kevin
WBR, Sergei _______________________________________________ Davinci-linux-open-source mailing list Davinci-linux-open-source@linux.davincidsp.com http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source