From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751502AbbETFBQ (ORCPT ); Wed, 20 May 2015 01:01:16 -0400 Received: from cantor2.suse.de ([195.135.220.15]:39462 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750819AbbETFBO (ORCPT ); Wed, 20 May 2015 01:01:14 -0400 Date: Wed, 20 May 2015 07:01:12 +0200 Message-ID: From: Takashi Iwai To: Jonathan McDowell Cc: alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org Subject: Re: Audio crackles with 4.1-rc1 In-Reply-To: <20150519215310.GF28070@earth.li> References: <20150429112859.GY10148@earth.li> <20150430150131.GA10148@earth.li> <20150502150513.GF10148@earth.li> <20150519161911.GD28070@earth.li> <20150519215310.GF28070@earth.li> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 Emacs/24.5 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org At Tue, 19 May 2015 22:53:10 +0100, Jonathan McDowell wrote: > > On Tue, May 19, 2015 at 07:09:12PM +0200, Takashi Iwai wrote: > > At Tue, 19 May 2015 17:19:11 +0100, > > Jonathan McDowell wrote: > > > > > > On Tue, May 19, 2015 at 08:15:32AM +0200, Takashi Iwai wrote: > > > > Does the patch below have any improvement? > > > > > > I still get a clicking on the standby -> active transition with the > > > patch provided on top of 4.1-rc4. > > > > OK, just to be sure, could you check whether changing msleep(10) to > > msleep(100) makes no difference? Also put a line like > > pr_info("XXX power up %x\n", nid); > > after msleep() call to see which nodes are powered up dynamically. > > Increasing the delay, if anything, makes the crackle when powering up > seem a little longer. dmesg log looks like: > > [ 4206.075557] XXX power up 2 > [ 4206.179671] XXX power up 14 > [ 4206.291758] XXX power up 3 OK, so everything looks correct, but the codec (or the machine) seems leading to the noise when a branched path is activated while the pin was already powered up. Then let's disable the widget power-saving for these codecs. Could you confirm that the patch below works? thanks, Takashi -- 8< -- From: Takashi Iwai Subject: [PATCH] ALSA: hda - Disable widget power-saving for ALC292 & co We've got reports that ALC3226 (a Dell variant of ALC292) gives click noises at transition from D3 to D0 when the widget power-saving is enabled. Further debugging session showed that avoiding it isn't trivial, unfortunately, since paths are basically activated dynamically while the pins have been already enabled. This patch disables the widget power-saving for such codecs. Reported-by: Jonathan McDowell Signed-off-by: Takashi Iwai --- sound/pci/hda/patch_realtek.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index 2e246fe495f6..31f8f13be907 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -5623,7 +5623,8 @@ static int patch_alc269(struct hda_codec *codec) spec = codec->spec; spec->gen.shared_mic_vref_pin = 0x18; - codec->power_save_node = 1; + if (codec->core.vendor_id != 0x10ec0292) + codec->power_save_node = 1; snd_hda_pick_fixup(codec, alc269_fixup_models, alc269_fixup_tbl, alc269_fixups); -- 2.4.1