[alsa-devel] [PATCH] ASOC: imx: audmux: add a integrated configuration function

Shawn Guo shawn.guo at linaro.org
Fri Feb 1 14:10:09 CET 2013


On Fri, Feb 01, 2013 at 12:35:19AM +0800, Mark Brown wrote:
> On Thu, Jan 31, 2013 at 10:22:45PM +0800, Shawn Guo wrote:
> 
> > Mark,
> 
> > I just found there are much more code between imx_sgtl5000_probe
> > and imx_wm8962_probe than just above audmux setup.  I'm thinking
> > about consolidate them at a higher level.  But can you give it a
> > go on Gary's patch, and then I can start the consolidation from
> > there?  I promise I will send a follow-up patch to do it.
> 
> I'm not sure what you mean here?  I don't have any intention to work on
> this stuff.

You had a comment on Gary's imx-wm8962 patch, saying that the audmux
setup should become a helper.  And that's why Gary came up this patch.
But the patch does not look good.  While reviewing the patch, I found
there are much more than just audmux setup code could be shared between
imx-sgtl5000 and imx-wm8960 driver.  I'm asking if you can merge Gary's
the patch with leaving audmux setup as it is, and let me consolidate
the common part between imx-sgtl5000 and imx-wm8960 with a follow-up
patch.

In any case, I'm not expecting you work on this stuff :)

Shawn



More information about the Alsa-devel mailing list