From: David Brownell <david-b@pacbell.net>
To: Kevin Hilman <khilman@deeprootsystems.com>,
Hugo Villeneuve <hugo@hugovil.com>
Cc: davinci-linux-open-source@linux.davincidsp.com,
alsa-devel@alsa-project.org, Mark Brown <broonie@sirena.org.uk>
Subject: Re: [PATCH 4/4] ALSA: ASoC: Davinci: Fixed missing McBSP pinmux for SFFSDR
Date: Tue, 10 Mar 2009 10:10:09 -0800 [thread overview]
Message-ID: <200903101110.10447.david-b@pacbell.net> (raw)
In-Reply-To: <87y6vds3ph.fsf@deeprootsystems.com>
On Tuesday 10 March 2009, Kevin Hilman wrote:
> >
> > Ok, so the patch affecting sound/soc/davinci/davinci-evm.c SHOULD have
> > been submitted to ALSA first.
>
> Yes, but it should've been a patch against an ASoC tree, not against
> DaVinci git which may contain things not (yet) in the ASoC tree.
>
> > It is not easy following you guys. You say something and then do the
> > opposite.
>
> Heh, sorry.
>
> Do as I do, not as I say. ;)
I'd say "my bad" ... but in these cases there's seems to be no
better solution than having the DaVinci GIT tree diverge a bit
from mainline ASoC, in synchrony with arch/arm/mach-davinci
which is also diverging from mainline.
There's a general rule that patches should not break builds.
And that rule is why there are several ASoC patches I've not
yet sent to mainline.
The whole ASoC/DaVinci thing has been nothing but build
breakage however, since it depends on EDMA patches which
haven't yet gone to mainline. (But, I believe, will soon
do so, at which point that breakage will stop.)
- Dave
next prev parent reply other threads:[~2009-03-10 18:10 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-03-10 3:32 Fix audio for Lyrtech SFFSDR board Hugo Villeneuve
2009-03-10 3:32 ` [PATCH 1/4] ALSA: ASoC: Davinci: Fix SFFSDR FPGA module codec FS bug Hugo Villeneuve
2009-03-10 15:53 ` Mark Brown
2009-03-10 3:32 ` [PATCH 2/4] ALSA: ASoC: Davinci: Replaced DAI format RIGHT_J by DSP_B for SFFSDR Hugo Villeneuve
[not found] ` <1236655929-31688-1-git-send-email-hugo-rg2e/+w5TaNBDgjK7y7TUQ@public.gmane.org>
2009-03-10 3:32 ` [PATCH 3/4] ALSA: ASoC: Davinci: Updated sffsdr_hw_params() function to new format Hugo Villeneuve
2009-03-10 3:32 ` [PATCH 4/4] ALSA: ASoC: Davinci: Fixed missing McBSP pinmux for SFFSDR Hugo Villeneuve
[not found] ` <20090310102921.GA12297@sirena.org.uk>
2009-03-10 14:03 ` Hugo Villeneuve
2009-03-10 15:16 ` Mark Brown
2009-03-10 15:42 ` Hugo Villeneuve
2009-03-10 15:55 ` Kevin Hilman
2009-03-10 16:02 ` Mark Brown
2009-03-10 16:08 ` Kevin Hilman
2009-03-10 16:04 ` Hugo Villeneuve
2009-03-10 16:20 ` Kevin Hilman
2009-03-10 17:07 ` Hugo Villeneuve
2009-03-10 17:11 ` Kevin Hilman
2009-03-10 18:10 ` David Brownell [this message]
2009-03-10 19:42 ` Mark Brown
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200903101110.10447.david-b@pacbell.net \
--to=david-b@pacbell.net \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@sirena.org.uk \
--cc=davinci-linux-open-source@linux.davincidsp.com \
--cc=hugo@hugovil.com \
--cc=khilman@deeprootsystems.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.