From: Liam Girdwood <lrg@slimlogic.co.uk>
To: Peter Ujfalusi <peter.ujfalusi@nokia.com>
Cc: "alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>,
"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
"tony@atomide.com" <tony@atomide.com>,
"broonie@opensource.wolfsonmicro.com"
<broonie@opensource.wolfsonmicro.com>,
"Valentin Eduardo (Nokia-D/Helsinki)"
<eduardo.valentin@nokia.com>,
"Nurkkala Eero.An (EXT-Offcode/Oulu)"
<ext-Eero.Nurkkala@nokia.com>,
"Koskinen Ilkka (Nokia-D/Tampere)" <ilkka.koskinen@nokia.com>
Subject: Re: [PATCHv5 0/2] McBSP: OMAP3: Add sidetone feature
Date: Mon, 22 Feb 2010 14:33:18 +0000 [thread overview]
Message-ID: <1266849198.3159.37.camel@odin> (raw)
In-Reply-To: <201002221618.15859.peter.ujfalusi@nokia.com>
On Mon, 2010-02-22 at 16:18 +0200, Peter Ujfalusi wrote:
> On Monday 22 February 2010 14:21:10 Koskinen Ilkka (Nokia-D/Tampere) wrote:
> > The first patch implements McBSP sidetone feature on OMAP3 while the second
> > one provides ALSA interface for using it.
> >
> > The patch set is based on the patch Eduardo Valentin sent to alsa-devel
> > mailing list in October 2009. Channel gain control and enabling the
> > sidetone have been moved to ALSA interface as suggested in comments.
> >
> > These patches apply on top of Janusz Krzysztofik's McBSP register cache
> > patches in linux-omap-2.6.git
> >
> > Since the version 4, ASoC part has been modified to deal with McBSP ids
> > consistently (zero based indexing).
> >
> > Eero Nurkkala (1):
> > McBSP: OMAP3: Add sidetone feature
> >
> > Ilkka Koskinen (1):
> > ASoC: OMAP-McBSP: ASoC interface for McBSP sidetone
> >
> > arch/arm/mach-omap2/mcbsp.c | 2 +
> > arch/arm/plat-omap/include/plat/mcbsp.h | 60 +++++
> > arch/arm/plat-omap/mcbsp.c | 402
> > ++++++++++++++++++++++++++++++- sound/soc/omap/omap-mcbsp.c |
> > 138 +++++++++++
> > sound/soc/omap/omap-mcbsp.h | 2 +
> > 5 files changed, 603 insertions(+), 1 deletions(-)
>
> Both looks OK for me:
>
> Acked-by: Peter Ujfalusi <peter.ujfalusi@nokia.com>
Acked-by: Liam Girdwood <lrg@slimlogic.co.uk>
--
Freelance Developer, SlimLogic Ltd
ASoC and Voltage Regulator Maintainer.
http://www.slimlogic.co.uk
next prev parent reply other threads:[~2010-02-22 14:33 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-02-22 12:21 [PATCHv5 0/2] McBSP: OMAP3: Add sidetone feature Ilkka Koskinen
2010-02-22 12:21 ` [PATCHv5 1/2] " Ilkka Koskinen
2010-02-22 12:21 ` [PATCHv5 2/2] ASoC: OMAP-McBSP: ASoC interface for McBSP sidetone Ilkka Koskinen
2010-02-22 14:18 ` [PATCHv5 0/2] McBSP: OMAP3: Add sidetone feature Peter Ujfalusi
2010-02-22 14:33 ` Liam Girdwood [this message]
2010-02-22 14:39 ` Mark Brown
2010-02-22 18:35 ` Tony Lindgren
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=1266849198.3159.37.camel@odin \
--to=lrg@slimlogic.co.uk \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@opensource.wolfsonmicro.com \
--cc=eduardo.valentin@nokia.com \
--cc=ext-Eero.Nurkkala@nokia.com \
--cc=ilkka.koskinen@nokia.com \
--cc=linux-omap@vger.kernel.org \
--cc=peter.ujfalusi@nokia.com \
--cc=tony@atomide.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox