Re: v2.6.28-omap1 tagged, some bugs remain

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



* 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

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux