From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Petazzoni Subject: Re: Regression caused by "ASoC: core: Rework SOC_DOUBLE_R_SX_TLV add SOC_SINGLE_SX_TLV" Date: Thu, 30 Jan 2014 16:05:54 +0100 Message-ID: <20140130160554.694a8a84@skate> References: <20140130144807.5b102b28@skate> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail.free-electrons.com (top.free-electrons.com [176.31.233.9]) by alsa0.perex.cz (Postfix) with ESMTP id C69F62654B9 for ; Thu, 30 Jan 2014 16:05:57 +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: Brian Austin Cc: Lior Amsalem , alsa-devel@alsa-project.org, Liam Girdwood , Mark Brown , Ezequiel Garcia , Gregory =?UTF-8?B?Q2zDqW1lbnQ=?= List-Id: alsa-devel@alsa-project.org Dear Brian Austin, On Thu, 30 Jan 2014 08:59:44 -0600, Brian Austin wrote: > That is odd. My other devices that use this don't show that behavior, I > will check on a different device, but I will see if I can get an L51 > today. So your saying the L51 for PCMA/B Mixer Volume COntrol, you can > only select 2 values in the whole range? Yes, absolutely. Not only for the PCMA/B Mixer Volume Control but also for the two other volume controls that were modified by your commit. I'll try to get back to your commit (without my revert) to verify once again my claims. > > Therefore, I believe that the commit has a problem. I haven't yet > > investigated where the problem is, maybe just the values passed to > > SOC_DOUBLE_R_SX_TLV(), or maybe in the implementation of the volume > > control functions themselves. I can certainly start investigating, but > > maybe the author of the commit will immediately realize where the > > problem could be. > > I do see however that the MAX value is not correct and should be 0x80 > instead of 0x7F. Off by 1... Hum, ok, but I don't believe this should affect the range of values that alsamixer sees, but only the fact that you're not using the full range of volumes available at the codec level, right? > I'll investigate today. Thanks! Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com