From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: Pop noise on startup when headphones are plugged in (Dell XPS13 9333) Date: Mon, 20 Apr 2015 09:18:40 +0200 Message-ID: References: <23057263.IjMh4t6SYF@xps13> 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 mx2.suse.de (cantor2.suse.de [195.135.220.15]) by alsa0.perex.cz (Postfix) with ESMTP id A2D0A2604FC for ; Mon, 20 Apr 2015 09:18:40 +0200 (CEST) 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: Gabriele Mazzotta Cc: alsa-devel@alsa-project.org List-Id: alsa-devel@alsa-project.org At Mon, 20 Apr 2015 07:58:15 +0200, Takashi Iwai wrote: > > At Sun, 19 Apr 2015 19:26:58 +0200, > Gabriele Mazzotta wrote: > > > > Hi, > > > > I've recently found that commit a551d91473 ("ALSA: hda - Use regmap for > > command verb caches, too") is somehow causing a pop noise on startup > > when headphones are plugged in, but I couldn't figure out the exact > > cause. Was this observed on other systems (mine is a Dell XPS13 9333, > > Realtek ALC3661)? Does anyone have any idea of what the cause might be? > > Are you sure that the commit is the culprit? There are lots of other > changes like the widget-based power saving. FYI, the widget power-saving can be disabled by the following. (It can be done by hints, too.) If this fixes the problem, we likely need more tweaks at path power activation. Or, as a last resort, put this to the fixup for your model. Takashi --- diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index 231d0e4b9a95..637ff2b70f6e 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -5584,7 +5584,7 @@ static int patch_alc269(struct hda_codec *codec) spec = codec->spec; spec->gen.shared_mic_vref_pin = 0x18; - codec->power_save_node = 1; + /* codec->power_save_node = 1; */ snd_hda_pick_fixup(codec, alc269_fixup_models, alc269_fixup_tbl, alc269_fixups);