Sudhakar Rajashekhara <[email protected]> writes:

> This patch set corrects some issues with the existing EDMA
> driver and also adds support for EDMA resource (channel/slots)
> sharing between two processors (say ARM and DSP).
>
> The patch set has been tested on DM646x, OMAP-L137 and OMAP-L138
> EVM boards.

Hi Sudhakar,

This series looks good.  One minor request.  

For the last three patches, I think it would be helpful to describe in
the changelog or even better at the definition of the reserved arrays
what the channels are being reserved for.

Kevin


> Sudhakar Rajashekhara (9):
>   davinci: Correct return value of edma_alloc_channel api
>   davinci: Keep count of channel controllers on a platform
>   davinci: Fix edma_alloc_channel api for EDMA_CHANNEL_ANY case
>   davinci: build list of unused EDMA events dynamically
>   davinci: support for EDMA resource sharing
>   davinci: da8xx/omap-l1xx: Add EDMA platform data for da850/omap-l138
>   davinci: da830/omapl137: Specify reserved channels/slots
>   davinci: da850/omapl138: Specify reserved channels/slots
>   davinci: dm646x: Specify reserved EDMA channel/slots for DM646x
>
>  arch/arm/mach-davinci/devices-da8xx.c     |  181 ++++++++++++++++++++++++++--
>  arch/arm/mach-davinci/dm355.c             |    8 --
>  arch/arm/mach-davinci/dm644x.c            |   10 --
>  arch/arm/mach-davinci/dm646x.c            |   28 ++++-
>  arch/arm/mach-davinci/dma.c               |  101 ++++++++++++++---
>  arch/arm/mach-davinci/include/mach/edma.h |    4 +-
>  6 files changed, 276 insertions(+), 56 deletions(-)
>
> _______________________________________________
> Davinci-linux-open-source mailing list
> [email protected]
> http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to