[alsa-devel] [PATCH 0/9] ARM: DaVinci: ASoC: Adds Audio support for TI DM646X

[Mark Brown]

On Mon, Mar 16, 2009 at 08:00:54AM -0400, Naresh Medisetty wrote:
> Adds Audio support for TI DM646X EVM platform

> Signed-off-by: Naresh Medisetty <naresh at ti.com>

I'm broadly OK with anything I haven't specifically commented on here.
However, the ASoC bits of the series need to be updated to reflect the
current ASoC APIs.  Some additional documentation for your changes to
the EVM would be very helpful.

It would also be very much better if you could restructure the series to
reflect the changes you're making.  You're actually doing three things:

 - Add support for a dummy codec.
 - Add support for the McASP port on the DaVinci.
 - Update the DaVinci EVM driver to use these two things on relevant
   EVMs.

and the series should be structured to reflect this.  The codec and EVM
support should each go in a single patch by themselves.  The McASP
support is big enough to be broken down but it does need to come before
the EVM support that uses it.