Mark,
Thanks for the review.
I will re-submit the series by taking care of your comments.
-----Original Message----- From: naresh [mailto:x0090427@ti.com] Sent: Wednesday, March 18, 2009 5:50 PM To: 'naresh@ti.com'mailto:'naresh@ti.com' Subject:
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@ti.commailto:naresh@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.
I will take elaborate the documentation while re-submitting the patches
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.
I will structure the series according to this.