[alsa-devel] [PATCH 4/4] ALSA: ASoC: Davinci: Fixed missing McBSP pinmux for SFFSDR

Hugo Villeneuve hugo at hugovil.com
Tue Mar 10 17:04:34 CET 2009

On Tue, 10 Mar 2009 08:55:06 -0700
Kevin Hilman <khilman at deeprootsystems.com> wrote:

> Hugo Villeneuve <hugo at hugovil.com> writes:
> > On Tue, 10 Mar 2009 15:16:32 +0000
> > Mark Brown <broonie at sirena.org.uk> wrote:
> >
> >> On Tue, Mar 10, 2009 at 10:03:41AM -0400, Hugo Villeneuve wrote:
> >> 
> >> > I based those patches on the latest linux-davinci git tree, which
> >> > has the function.
> >> 
> >> Do not submit patches for mainline which are not based on mainline
> >> trees.  Code which relies on out of tree changes needs to wait for
> >> those out of tree changes to be merged before submitting to
> >> mainline.
> >
> > I did not know that these changes were not in mainline yet. I was
> > told that all davinci ASoC code changes were to be submitted only
> > to ALSA, and then were imported back into the davinci git tree.
> > Apparently this is not quite like that anymore.
> >
> > Kevin, what is the new rule to submit davinci ASoC patches?
> >
> The DaVinci ASoC code is indeed in mainline, but not all of the
> DaVinci core (in this case the pin mux) is yet in mainline.  I will be
> pushing it during the next merge window.

That doesn't really answer my question.

I can see that David Brownell pushed a patch to the davinci tree
directly modifying sound/soc/davinci/davinci-evm.c which IS in
mainline. Does this means that as of now all ASoC patches should be sent first to
the Davinci list, and then you will push those to the mainline kernel?

> On a related note, sound/soc/davinci/davinci-[evm,sffsdr].c is
> actually board-level setup code and should not actually live in
> sound/asoc/.  It should live in arch/arm/mach-davinci/* and follow the
> rest of the davinci chip/board code upstream.

Kevin, you have said the same things months ago and I agree with you.
When you or someone else move the evm stuff back into
arch/arm/mach-davinci, I will be more than happy to do the same for the

Hugo V.

More information about the Alsa-devel mailing list