From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: [PATCH v2 0/3] alsa-lib: Improve 2.1 surround support Date: Fri, 28 Feb 2014 09:15:15 +0100 Message-ID: References: <1393574227-25732-1-git-send-email-david.henningsson@canonical.com> Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mx2.suse.de (cantor2.suse.de [195.135.220.15]) by alsa0.perex.cz (Postfix) with ESMTP id D04522650EE for ; Fri, 28 Feb 2014 09:15:15 +0100 (CET) In-Reply-To: <1393574227-25732-1-git-send-email-david.henningsson@canonical.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: David Henningsson Cc: alsa-devel@alsa-project.org List-Id: alsa-devel@alsa-project.org At Fri, 28 Feb 2014 08:57:04 +0100, David Henningsson wrote: > > Thanks Takashi for your encouraging review, > > Diff since last version is only the things you remarked about in the first > patch series. > > > Besides that, I find it also OK to just return the channel (>= 0) or a > > negative error. But it's no big matter in which form is. > > This one I didn't change, just because then I would just have had to change it > back in the second patch anyway. OK, applied all patches now. Thanks! Takashi > > David Henningsson (3): > route: Allow chmap syntax for slave channels in ttable > route: Select slave chmap based on ttable information > conf: Allow 2.1 surround to use different number of channels > > src/conf/pcm/surround21.conf | 7 +- > src/pcm/pcm_route.c | 323 +++++++++++++++++++++++++++++++++++++----- > 2 files changed, 290 insertions(+), 40 deletions(-) > > -- > 1.7.9.5 >