From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mark Brown Subject: Re: [PATCH] asoc tlv320aic3x: set power bits correctly Date: Wed, 30 Apr 2008 10:57:47 +0100 Message-ID: <20080430095747.GA28593@sirena.org.uk> References: <20080430093719.GA13740@buzzloop.caiaq.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from cassiel.sirena.org.uk (cassiel.sirena.org.uk [80.68.93.111]) by alsa0.perex.cz (Postfix) with ESMTP id 64FE3245EC for ; Wed, 30 Apr 2008 11:57:48 +0200 (CEST) Content-Disposition: inline In-Reply-To: <20080430093719.GA13740@buzzloop.caiaq.de> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: alsa-devel-bounces@alsa-project.org Errors-To: alsa-devel-bounces@alsa-project.org To: Daniel Mack Cc: alsa-devel@alsa-project.org List-Id: alsa-devel@alsa-project.org On Wed, Apr 30, 2008 at 11:37:19AM +0200, Daniel Mack wrote: > Set power bits in output stage control registers to reflect the "!muted" > bits. The codec needs them set in order to operate correctly. Could you explain in more detail what this patch is doing, please (ideally in comments in the code or at least the commit message)? > + case MONOLOPM_CTRL: > + if (*val & 0x08) > + *val |= 0x01; > + else > + *val &= ~0x01; > + > + break; > + } It would be good if the code were clearer about what these register bits mean and why this isn't being handled via DAPM. If what you're trying to do is avoid powering on amplifiers when they are muted then this is best avoided since it often creates problems with pops and clicks due to things getting powered on and off out of the sequence generated by DAPM. If the amplifier is powered on before its inputs then it will amplify any noise generaetd by the inputs when they change power state. This is why ASoC mute controls for amplifiers are normally independent of their power controls.