]> git.baikalelectronics.ru Git - kernel.git/commit
ALSA: hda/realtek - Remove now-unnecessary XPS 13 headphone noise fixups
authorThomas Hebb <tommyhebb@gmail.com>
Mon, 30 Mar 2020 16:09:39 +0000 (12:09 -0400)
committerTakashi Iwai <tiwai@suse.de>
Tue, 31 Mar 2020 08:54:06 +0000 (10:54 +0200)
commit0fa4d070c7680c9a6eda84ed7283f2bd8884fca9
tree62ae44059eb9c46857fdfa12802288e9352719b4
parent67acc2dfb1ee462f443fb3a40f139d00005fed9b
ALSA: hda/realtek - Remove now-unnecessary XPS 13 headphone noise fixups

patch_realtek.c has historically failed to properly configure the PC
Beep Hidden Register for the ALC256 codec (among others). Depending on
your kernel version, symptoms of this misconfiguration can range from
chassis noise, picked up by a poorly-shielded PCBEEP trace, getting
amplified and played on your internal speaker and/or headphones to loud
feedback, which responds to the "Headphone Mic Boost" ALSA control,
getting played through your headphones. For details of the problem, see
the patch in this series titled "ALSA: hda/realtek - Set principled PC
Beep configuration for ALC256", which fixes the configuration.

These symptoms have been most noticed on the Dell XPS 13 9350 and 9360,
popular laptops that use the ALC256. As a result, several model-specific
fixups have been introduced to try and fix the problem, the most
egregious of which locks the "Headphone Mic Boost" control as a hack to
minimize noise from a feedback loop that shouldn't have been there in
the first place.

Now that the underlying issue has been fixed, remove all these fixups.
Remaining fixups needed by the XPS 13 are all picked up by existing pin
quirks.

This change should, for the XPS 13 9350/9360

 - Significantly increase volume and audio quality on headphones
 - Eliminate headphone popping on suspend/resume
 - Allow "Headphone Mic Boost" to be set again, making the headphone
   jack fully usable as a microphone jack too.

Fixes: 24fffba49194 ("ALSA: hda - Fix headphone noise after Dell XPS 13 resume back from S3")
Fixes: d359c895dddf ("ALSA: hda - Fix headphone noise on Dell XPS 13 9360")
Fixes: d07e2688e343 ("ALSA: hda - Apply headphone noise quirk for another Dell XPS 13 variant")
Fixes: 53849275a282 ("ALSA: hda/realtek: Reduce the Headphone static noise on XPS 9350/9360")
Cc: stable@vger.kernel.org
Signed-off-by: Thomas Hebb <tommyhebb@gmail.com>
Link: https://lore.kernel.org/r/b649a00edfde150cf6eebbb4390e15e0c2deb39a.1585584498.git.tommyhebb@gmail.com
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Documentation/sound/hd-audio/models.rst
sound/pci/hda/patch_realtek.c