From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Ujfalusi Subject: Re: [PATCHv5 0/2] McBSP: OMAP3: Add sidetone feature Date: Mon, 22 Feb 2010 16:18:15 +0200 Message-ID: <201002221618.15859.peter.ujfalusi@nokia.com> References: <1266841272-13135-1-git-send-email-ilkka.koskinen@nokia.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mgw-mx06.nokia.com (smtp.nokia.com [192.100.122.233]) by alsa0.perex.cz (Postfix) with ESMTP id 1C5CF10382F for ; Mon, 22 Feb 2010 15:18:43 +0100 (CET) In-Reply-To: <1266841272-13135-1-git-send-email-ilkka.koskinen@nokia.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: alsa-devel-bounces@alsa-project.org Errors-To: alsa-devel-bounces@alsa-project.org To: "Koskinen Ilkka (Nokia-D/Tampere)" Cc: "alsa-devel@alsa-project.org" , "tony@atomide.com" , "broonie@opensource.wolfsonmicro.com" , "Valentin Eduardo (Nokia-D/Helsinki)" , "Nurkkala Eero.An (EXT-Offcode/Oulu)" , "linux-omap@vger.kernel.org" List-Id: alsa-devel@alsa-project.org 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