* Jarkko Nikula <jarkko.nikula@xxxxxxxxx> [090118 23:54]: > On Wed, 14 Jan 2009 17:28:07 +0200 > "ext Tony Lindgren" <tony@xxxxxxxxxxx> wrote: > > > > You need to have the counterpart (in /sound/soc/omap/omap-mcbsp.c) for > > > that patch, othwerwise there's no chance to hear any DMA transferred > > > audio. > > > > Jarkko, can you post the patch queued up for alsa? We might want to > > have that in linux-omap too. > > > Patch was made by Misael and I think he is waiting until the plat-omap > part goes into mainline. And which means that OMAP3 audio is broken > until sound/soc/omap/omap-mcbsp.c is patched. > > I would say that we wouldn't want to hack with sound/ directory in > linux-omap. Especially code which is not in Takashi's tree since then > we have yet another conflict point if one wants to merge from his tree > on top of linux-omap. > > Probably we should revert the patch and split it into three > separate patches: > > 1. Header change in linux-omap adding those registers (and queue it) > 2. Patch to alsa-devel setting default values to those registers > 3. Patch to linux-omap writing those registers OK, I've reverted #3 above in l-o master and omap-2.6.28 branches. > Or better: send those two patches to LKML, cc linux-omap and alsa-devel > and let it fall down from -mm or linux-next tree. Well since #3 is already queued in omap-fixes, I'd rather let it go to mainline and not mess with that queue. Maybe after that you could try to get #1 & #2 above merged to mainline via alsa-devel list as one patch? Regards, Tony -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html