From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: [PATCH] [updated] pcm: rate: Add capability to pass configuration node to plugins Date: Tue, 21 Feb 2017 22:30:43 +0100 Message-ID: References: <6d7c74b7-da90-5ae9-c355-c6884b2edf09@gmail.com> <7b3c655f-4b7a-e10a-90ff-5cb480332f3f@IEE.org> <3b8cd9c9-8248-b22a-ebb8-dce7d901951d@IEE.org> <2cb132c7-638b-6d93-621d-3460654660b8@IEE.org> 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 (mx2.suse.de [195.135.220.15]) by alsa0.perex.cz (Postfix) with ESMTP id 28D93266AA9 for ; Tue, 21 Feb 2017 22:30:44 +0100 (CET) In-Reply-To: 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: Alan Young Cc: alsa-devel@alsa-project.org List-Id: alsa-devel@alsa-project.org On Tue, 21 Feb 2017 17:14:29 +0100, Alan Young wrote: > > On 21/02/17 15:28, Takashi Iwai wrote: > > You forgot to call > > > > snd_pcm_free(pcm); > > free(rate); > > > > before returning an error. > Ah, ok. Updated patch. > > >From de90c659c98ae4dac7de6eb225b22147f56a3d1c Mon Sep 17 00:00:00 2001 > From: Alan Young > Date: Thu, 7 Apr 2016 09:15:04 +0100 > Subject: [PATCH] pcm: rate: Add capability to pass configuration node to > plugins > > If a rate plugin uses a node (compound) instead of a plain string for > its "converter", and that compound is not a simple string array, then > the compound will be passed as an additional parameter to the new plugin > open() function (SND_PCM_RATE_PLUGIN_CONF_ENTRY(XXX)). The previous > open() function (SND_PCM_RATE_PLUGIN_ENTRY(XXX)) will be called if the > CONF version is not found. It is up to the plugin to determine whether > the presence of the conf parameter is mandatory. > > Signed-off-by: Alan Young Applied, thanks. Takashi