Skip to contentSkip to conversation

Patch "ALSA: hda - Fix broken reconfig" has been added to the 4.5-stable tree

PATCH [email protected] 1 participant 1 message May 15, 2016

Subsystems (2)
lore
Share & tools
lore

[email protected]

author maintainer Applied
#0 lore raw

This is a note to let you know that I've just added the patch titled

ALSA: hda - Fix broken reconfig

to the 4.5-stable tree which can be found at:
http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
alsa-hda-fix-broken-reconfig.patch
and it can be found in the queue-4.5 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree, please let <[email protected]> know about it.

Quoted 1 line

From addacd801e1638f41d659cb53b9b73fc14322cb1 Mon Sep 17 00:00:00 2001

From: Takashi Iwai <[email protected]>
Date: Tue, 10 May 2016 10:24:02 +0200
Subject: ALSA: hda - Fix broken reconfig

From: Takashi Iwai <[email protected]>

commit addacd801e1638f41d659cb53b9b73fc14322cb1 upstream.

The HD-audio reconfig function got broken in the recent kernels, typically resulting in a failure like:
snd_hda_intel 0000:00:1b.0: control 3:0:0:Playback Channel Map:0 is already present

This is because of the code restructuring to move the PCM and control instantiation into the codec drive probe, by the commit [bcd96557bd0a:
ALSA: hda - Build PCMs and controls at codec driver probe]. Although the commit above removed the calls of snd_hda_codec_build_pcms() and *_build_controls() at the controller driver probe, the similar calls in the reconfig were still left forgotten. This caused the conflicting and duplicated PCMs and controls.

The fix is trivial: just remove these superfluous calls from reconfig_codec().

Fixes bcd96557bd0a ('ALSA: hda - Build PCMs and controls at codec driver probe')
---
 sound/pci/hda/hda_sysfs.c |    8 --------
 1 file changed, 8 deletions(-)
sound/pci/hda/hda_sysfs.c +0 -8
@@ -141,14 +141,6 @@ static int reconfig_codec(struct hda_cod
 	err = snd_hda_codec_configure(codec);
 	if (err < 0)
 		goto error;
-	/* rebuild PCMs */
-	err = snd_hda_codec_build_pcms(codec);
-	if (err < 0)
-		goto error;
-	/* rebuild mixers */
-	err = snd_hda_codec_build_controls(codec);
-	if (err < 0)
-		goto error;
 	err = snd_card_register(codec->card);
  error:
 	snd_hda_power_down(codec);

Patches currently in stable-queue which might be from [email protected] are

queue-4.5/alsa-usb-audio-quirk-for-yet-another-phoenix-audio-devices-v2.patch queue-4.5/alsa-hda-fix-subwoofer-pin-on-asus-n751-and-n551.patch queue-4.5/alsa-hda-fix-white-noise-on-asus-ux501vw-headset.patch queue-4.5/alsa-hda-fix-broken-reconfig.patch queue-4.5/alsa-usb-audio-yet-another-phoneix-audio-device-quirk.patch