From mboxrd@z Thu Jan 1 00:00:00 1970 From: Charles Keepax Subject: Re: [PATCH v4 2/5] ASoC: Move standard kcontrol helpers to the component level Date: Mon, 12 May 2014 11:42:04 +0100 Message-ID: <20140512104204.GF25203@opensource.wolfsonmicro.com> References: <20140509150034.GH2870@dragon> <536CF16F.8010807@metafoo.de> <536CFBE7.3070805@metafoo.de> <20140510050747.GI2870@dragon> <536DCF6F.9020507@metafoo.de> <536DD108.2090607@metafoo.de> <20140510083131.GL2870@dragon> <536DE548.9050008@metafoo.de> <20140510091209.GM2870@dragon> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from opensource.wolfsonmicro.com (opensource.wolfsonmicro.com [80.75.67.52]) by alsa0.perex.cz (Postfix) with ESMTP id EFCC3261680 for ; Mon, 12 May 2014 12:42:04 +0200 (CEST) Content-Disposition: inline In-Reply-To: <20140510091209.GM2870@dragon> 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: Shawn Guo Cc: "alsa-devel@alsa-project.org" , Lars-Peter Clausen , Fabio Estevam , Mark Brown , Liam Girdwood List-Id: alsa-devel@alsa-project.org On Sat, May 10, 2014 at 05:12:11PM +0800, Shawn Guo wrote: > On Sat, May 10, 2014 at 10:37:28AM +0200, Lars-Peter Clausen wrote: > > On 05/10/2014 10:31 AM, Shawn Guo wrote: > > >On Sat, May 10, 2014 at 09:11:04AM +0200, Lars-Peter Clausen wrote: > > >>On 05/10/2014 09:04 AM, Lars-Peter Clausen wrote: > > >>[..] > > >>>>The error message only shows up with Debian wheezy and does not with > > >>>>yocto rootfs. And even when the error message shows, the audio still > > >>>>functions well on Debian wheezy. So it's just a noisy error message > > >>>>for me which is only seen after your kernel patch. > > >>> > > >>>The keyword here is "seen". The error quite likely predated the commit, but > > >>>it was silently discarded. > > >>> > > >>>regmap_read() returns -EBUSY when there is no cached register value and > > >>>cache_only is set to true. But I'm not sure why that would happen, try to > > >>>add some printks to _regmap_read() to see if this is the source and if it is > > >>>why it is. > > >> > > >>Both SGTL5000_CHIP_ANA_ADC_CTRL and SGTL5000_CHIP_MIC_CTRL don't > > >>have an entry in sgtl5000_reg_defaults. So if cache_only is true, > > >>controls which use these registers will return -EBUSY when you try > > >>to read or write them. > > > > > >Hmm, it's a wm8962 than sgtl5000 on board imx6q-sabresd. > > > > Ok, misread the boardname. But the issue will be the same. > > cache_only = true and trying to read/update a register that is not > > in the cache. > > Yea, you're right. When the error message shows up, _regmap_read() > returns -EBUSY for reg WM8962_CLASS_D_CONTROL_1. But how do I find out > why the reg is not in the cache? >>From what I can see it doesn't look like this register should be volatile. I am checking with the hardware guys here, incase I am missing something, but otherwise I will wing out a patch to make it non-volatile. Thanks, Charles