Liam Girdwood wrote:
On Fri, 2010-09-24 at 11:31 -0500, Lopez Cruz, Misael wrote:
Hi Jarkko,
ASoC Multi-Component Support moves some code from sound/soc/omap/zoom2.c into arch/arm/mach-omap2/board-zoom2.c. However, that code should go to board-zoom-peripherals.c instead as there is common code and registration for zoom boards.
Signed-off-by: Jarkko Nikula jhnikula@gmail.com Cc: Vikram Pandita vikram.pandita@ti.com Cc: Lopez Cruz, Misael x0052729@ti.com Cc: Jorge Eduardo Candelaria jorge.candelaria@ti.com Cc: Tony Lindgren tony@atomide.com
I don't have this HW so not tested.
Tested on zoom2 using the 2 patches of your series, compilation break is gone and audio works fine.
Thanks, -Misael
Applied, and manually added your Tested-by:
Thanks
Liam
Misael, Liam,
Sorry I noticed this patch only now. I have a minor comment on the patch:
Specifically, the part below
+ if (machine_is_omap_zoom2()) { + zoom_audio_data.ramp_delay_value = 3; /* 161 ms */ + zoom_audio_data.hs_extmute = 1; + zoom_audio_data.set_hs_extmute = zoom2_set_hs_extmute; + }
Isn't the audio on a zoom2/zoom3/3630 SDP the same? (Or at least the same on a zoom2 and zoom3)? If so, we could probably get rid of the machine_is_omap_zoom2() check, right?
- Anand