From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: [PATCH 0/3] ALSA: pcm: Date: Fri, 09 Jun 2017 09:19:00 +0200 Message-ID: References: <20170609003440.4178-1-o-takashi@sakamocchi.jp> 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 mx1.suse.de (mx2.suse.de [195.135.220.15]) by alsa0.perex.cz (Postfix) with ESMTP id 5B22C266CB8 for ; Fri, 9 Jun 2017 09:19:00 +0200 (CEST) In-Reply-To: <20170609003440.4178-1-o-takashi@sakamocchi.jp> 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: Takashi Sakamoto Cc: alsa-devel@alsa-project.org List-Id: alsa-devel@alsa-project.org On Fri, 09 Jun 2017 02:34:37 +0200, Takashi Sakamoto wrote: > > Hi, > > This patchset is a revised version of my former RFC. > > > [alsa-devel] [PATCH RFC 00/21] ALSA: pcm: add tracepoints for PCM params operation > http://mailman.alsa-project.org/pipermail/alsa-devel/2017-May/120548.html > > In ALSA PCM interface, applications can get hardware capability as > 'struct snd_pcm_hw_params' type of data returned by a call of ioctl(2) > with SNDRV_PCM_IOCTL_HW_REFINE/SNDRV_PCM_IOCTL_HW_PARAMS' commands. > Results of these commands should be the same in a certain conditions but > actually it's not. > > This is a part of layout of the structure. > > struct snd_pcm_hw_params { > ... > unsigned int info; > unsigned int msbits; > unsigned int rate_num; > unsigned int rate_den; > snd_pcm_uframes_t fifo_size; > ... > }; > > For example, although the 'msbits', 'rate_num' and 'rate_den' fields are > filled in a result of HW_REFINE command conditionally, they're never > filled in a result of HW_PARAMS command. This seems a bug. > > This patchset attempts to fix the bug. This affects userspace applications > in cotents of the result, but it's acceptrable with a merit for the > applications to get enough parameters. > > Takashi Sakamoto (3): > ALSA: pcm: use helper functions to refer parameters as constants > ALSA: pcm: calculate non-mask/non-interval parameters always when > possible > ALSA: pcm: move fixup of info flag after selecting single parameters Applied all three patches now. Thanks. Takashi