On Mon, Aug 17, 2009 at 11:07:37AM +0200, Jarkko Nikula wrote: > On Mon, 17 Aug 2009 11:22:30 +0300 > Peter Ujfalusi <peter.ujfalusi@xxxxxxxxx> wrote: > > > On Thursday 13 August 2009 17:35:13 Valentin Eduardo (Nokia-D/Helsinki) wrote: > > > Hello again guys, > > > > > > Here is version 3 of these changes. I've changed just 2 > > > things: > > > - all mcbsp instances are in element mode by default > > > - moved all mcbsp related code to omap-mcbsp.c > > > (although a single callback is still needed) > > > > > > Jarkko, I think we can add your patch for reading strings > > > for dma op mode later on. > > > > In my opinion we can take this series as it is now, what do you think Jarkko? > > At least I don't see too (any?) big problems with it... > > > Good to remind me. I was almost making a reviewing to this and Janusz's > patches yesterday but ended up testing the MUSB and watching some > aircrafts instead :-) > > But shortly going through the 3rd version of the set, only major issue I > point that it's not based on top of my patch below so most probably > patches 4 and 18 don't apply. Also changed API of > omap_mcbsp_start/-stop would allow to remove new exported functions > omap_mcbsp_xmit_enable and omap_mcbsp_recv_enable. > > commit c12abc012e18b362204345c323536f228d65c4db > Author: Jarkko Nikula <jhnikula@xxxxxxxxx> > Date: Fri Aug 7 09:59:47 2009 +0300 > > ARM: OMAP: McBSP: Fix ASoC on OMAP1510 by fixing API of > omap_mcbsp_start/sto > > Otherwise set looks fine since omap-mcbp.c and omap-pcm.c are not hard > glued together and the element mode is the default. Cool. I've rebased and tested the series with beagle using Takashi's tree. I'm about to send version #4 based on his tree then. > > > -- > Jarkko > -- > To unsubscribe from this list: send the line "unsubscribe alsa-devel" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Eduardo Valentin -- To unsubscribe from this list: send the line "unsubscribe alsa-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel