Sekhar Nori <nsek...@ti.com> writes:
> From: Sudhakar Rajashekhara <sudhakar....@ti.com>
>
> The drivers on da8xx/omapl EVMs do not utilize all the channels
> and slots provided by EDMA. Some of these are better utilitzed by
> the DSP on the SoC for speeding up codec operations.
>
> Reserve these channels/slots for the DSP.
>
> Signed-off-by: Sudhakar Rajashekhara <sudhakar....@ti.com>
> Signed-off-by: Sekhar Nori <nsek...@ti.com>
> ---
> arch/arm/mach-davinci/board-da830-evm.c | 32 ++++++++++++++++++-
> arch/arm/mach-davinci/board-da850-evm.c | 48
> +++++++++++++++++++++++++++-
> arch/arm/mach-davinci/devices-da8xx.c | 21 +++++++-----
> arch/arm/mach-davinci/include/mach/da8xx.h | 3 +-
> 4 files changed, 92 insertions(+), 12 deletions(-)
>
Thanks, I like this one better. Still a small problem though..
> +int __init da850_register_edma(struct edma_rsv_info *rsv)
> +{
> + if (rsv) {
> + da850_edma_info[0].rsv = &rsv[0];
> + da850_edma_info[1].rsv = &rsv[1];
> + }
> +
> + return platform_device_register(&da850_edma_device);
What if the caller only has reserved chans/slots for controller 0?
&rsv[1] will be an undefined pointer.
I think you need some sort of terminator on the list passed in.
so only edma_rsv_info pointers that are valid are passed along.
Kevin
_______________________________________________
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
