alsa-devel.alsa-project.org archive mirror
 help / color / mirror / Atom feed
From: Josh Boyer <jwboyer@redhat.com>
To: Takashi Iwai <tiwai@suse.de>, Jaroslav Kysela <perex@perex.cz>
Cc: alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org,
	kernel-team@fedoraproject.org
Subject: Thinkpad hda regression in 3.3
Date: Fri, 13 Apr 2012 08:43:06 -0400	[thread overview]
Message-ID: <20120413124306.GC25425@zod.bos.redhat.com> (raw)

Hi Takashi and Jaroslav,

We've had a number of bugs reported about sound not working correctly on
various models of Thinkpad machines in the 3.3 kernel.  I've listed them
below.

I looked through the commits and found:

    commit 29c5fbbcfefba5225a6783683c46c39e10877703:

    ALSA: HDA: Use model=auto for Thinkpad T510

    The user reports that model=auto works fine for him. Using
    model=auto bring in new features such as jack detection notification
    to userspace.

I did a local build here with that comment reverted and so far most of
the people reported their sound to be working again.  The trivial revert
patch is below in case you want it.

It seems there are multiple models that have PCI ID 17aa:215e and using
model=auto does not work for them all.  It particularly messes with
machines using a docking station for some reason.

Aside from just reverting the above commit, do you have other thoughts
on how to fix this?

josh

https://bugzilla.redhat.com/show_bug.cgi?id=808559
https://bugzilla.redhat.com/show_bug.cgi?id=806217
https://bugzilla.redhat.com/show_bug.cgi?id=810697

-----

From: Josh Boyer <jwboyer@redhat.com>
Subject: [PATCH] ALSA: hda - Use model=thinkpad for PCI ID 17aa:215e

Commit 29c5fb "ALSA: HDA: Use model=auto for Thinkpad T510" switched to
using model=auto for PCI ID 17aa:215e, however that breaks sound on a
number of thinkpads.  Change this back to using model=thinkpad for now.

Signed-off-by: Josh Boyer <jwboyer@redhat.com>

---

--- linux-3.3.noarch.orig/sound/pci/hda/patch_conexant.c
+++ linux-3.3.noarch/sound/pci/hda/patch_conexant.c
@@ -3027,7 +3027,7 @@ static const struct snd_pci_quirk cxt506
 	SND_PCI_QUIRK(0x17aa, 0x20f2, "Lenovo T400s", CXT5066_THINKPAD),
 	SND_PCI_QUIRK(0x17aa, 0x21c5, "Thinkpad Edge 13", CXT5066_THINKPAD),
 	SND_PCI_QUIRK(0x17aa, 0x21c6, "Thinkpad Edge 13", CXT5066_ASUS),
-	SND_PCI_QUIRK(0x17aa, 0x215e, "Lenovo T510", CXT5066_AUTO),
+	SND_PCI_QUIRK(0x17aa, 0x215e, "Lenovo T510", CXT5066_THINKPAD),
 	SND_PCI_QUIRK(0x17aa, 0x21cf, "Lenovo T520 & W520", CXT5066_AUTO),
 	SND_PCI_QUIRK(0x17aa, 0x21da, "Lenovo X220", CXT5066_THINKPAD),
 	SND_PCI_QUIRK(0x17aa, 0x21db, "Lenovo X220-tablet", CXT5066_THINKPAD),

             reply	other threads:[~2012-04-13 12:43 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-13 12:43 Josh Boyer [this message]
2012-04-13 13:07 ` Thinkpad hda regression in 3.3 Takashi Iwai
2012-04-13 13:23   ` Josh Boyer
2012-04-13 13:29     ` Takashi Iwai
2012-04-13 13:37       ` Josh Boyer
2012-04-13 13:41         ` Takashi Iwai
2012-04-13 14:05           ` Josh Boyer
2012-04-13 14:13             ` Takashi Iwai
2012-04-13 13:26   ` Takashi Iwai
2012-04-13 16:07   ` Takashi Iwai
2012-04-13 18:08     ` Josh Boyer
2012-04-13 19:21       ` Takashi Iwai
2012-04-18  0:23       ` Josh Boyer
2012-04-18  5:49         ` Takashi Iwai

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20120413124306.GC25425@zod.bos.redhat.com \
    --to=jwboyer@redhat.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=kernel-team@fedoraproject.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=perex@perex.cz \
    --cc=tiwai@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).