Alsa-Devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Joschi Brauchle <joschi.brauchle@tum.de>
Cc: alsa-devel@alsa-project.org
Subject: Re: No sound on Thinkpad T440s in Docking Station
Date: Tue, 06 May 2014 17:41:17 +0200	[thread overview]
Message-ID: <s5hzjiuud7m.wl%tiwai@suse.de> (raw)
In-Reply-To: <5368FDBD.2030400@tum.de>

At Tue, 06 May 2014 17:20:29 +0200,
Joschi Brauchle wrote:
> 
> Alright, I think I found the correct settings:
> 
> [codec]
> 0x10ec0292 0x17aa220c 0
> 
> [pincfg]
> 0x16 0x21211010
> 0x19 0x21a11010
> 
> If
> a) no headphones plugged in anywhere, laptop speakers work
> b) headphones only plugged in dock and not at laptop, dock headphones 
> work, laptop speakers silent
> c) headphones plugged in laptop (and/or dock), only laptop headphones 
> work, laptop speakers and dock headphone silent
> 
> If this is the intended behavior, these values work.

Looks good.  Below is the revised patch I'm going to apply to the
sound git tree.  Could you check whether it works as expected?

Once confirmed, I can take it to openSUSE 13.1, too.


thanks,

Takashi

---
From: Takashi Iwai <tiwai@suse.de>
Subject: [PATCH] ALSA: hda - Add dock pin setups for Thinkpad T440

The headphone and mic jacks on Thinkpad T440 are assigned to pins NID
0x16 and 0x19, respectively.  These need to be set up manually by a
fixup.

Reported-by: Joschi Brauchle <joschi.brauchle@tum.de>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
---
 sound/pci/hda/patch_realtek.c | 15 ++++++++++++++-
 1 file changed, 14 insertions(+), 1 deletion(-)

diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
index b60de0dc40d3..40f9b5d712eb 100644
--- a/sound/pci/hda/patch_realtek.c
+++ b/sound/pci/hda/patch_realtek.c
@@ -4236,6 +4236,7 @@ enum {
 	ALC255_FIXUP_HEADSET_MODE,
 	ALC255_FIXUP_HEADSET_MODE_NO_HP_MIC,
 	ALC293_FIXUP_DELL1_MIC_NO_PRESENCE,
+	ALC292_FIXUP_TPT440_DOCK,
 };
 
 static const struct hda_fixup alc269_fixups[] = {
@@ -4639,6 +4640,16 @@ static const struct hda_fixup alc269_fixups[] = {
 		.chained = true,
 		.chain_id = ALC269_FIXUP_HEADSET_MODE
 	},
+	[ALC292_FIXUP_TPT440_DOCK] = {
+		.type = HDA_FIXUP_PINS,
+		.v.pins = (const struct hda_pintbl[]) {
+			{ 0x16, 0x21211010 }, /* dock headphone */
+			{ 0x19, 0x21a11010 }, /* dock mic */
+			{ }
+		},
+		.chained = true,
+		.chain_id = ALC269_FIXUP_LIMIT_INT_MIC_BOOST
+	},
 };
 
 static const struct snd_pci_quirk alc269_fixup_tbl[] = {
@@ -4803,7 +4814,8 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = {
 	SND_PCI_QUIRK(0x17aa, 0x21fb, "Thinkpad T430s", ALC269_FIXUP_LENOVO_DOCK),
 	SND_PCI_QUIRK(0x17aa, 0x2203, "Thinkpad X230 Tablet", ALC269_FIXUP_LENOVO_DOCK),
 	SND_PCI_QUIRK(0x17aa, 0x2208, "Thinkpad T431s", ALC269_FIXUP_LENOVO_DOCK),
-	SND_PCI_QUIRK(0x17aa, 0x220c, "Thinkpad", ALC269_FIXUP_LIMIT_INT_MIC_BOOST),
+	SND_PCI_QUIRK(0x17aa, 0x220c, "Thinkpad T440s", ALC292_FIXUP_TPT440_DOCK),
+	SND_PCI_QUIRK(0x17aa, 0x220e, "Thinkpad T440p", ALC292_FIXUP_TPT440_DOCK),
 	SND_PCI_QUIRK(0x17aa, 0x2212, "Thinkpad", ALC269_FIXUP_LIMIT_INT_MIC_BOOST),
 	SND_PCI_QUIRK(0x17aa, 0x2214, "Thinkpad", ALC269_FIXUP_LIMIT_INT_MIC_BOOST),
 	SND_PCI_QUIRK(0x17aa, 0x2215, "Thinkpad", ALC269_FIXUP_LIMIT_INT_MIC_BOOST),
@@ -4881,6 +4893,7 @@ static const struct hda_model_fixup alc269_fixup_models[] = {
 	{.id = ALC269_FIXUP_DELL2_MIC_NO_PRESENCE, .name = "dell-headset-dock"},
 	{.id = ALC283_FIXUP_CHROME_BOOK, .name = "alc283-dac-wcaps"},
 	{.id = ALC283_FIXUP_SENSE_COMBO_JACK, .name = "alc283-sense-combo"},
+	{.id = ALC292_FIXUP_TPT440_DOCK, .name = "tpt440-dock"},
 	{}
 };
 
-- 
1.9.2

  reply	other threads:[~2014-05-06 15:41 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-30  8:56 No sound on Thinkpad T440s in Docking Station Joschi Brauchle
2014-04-30 16:43 ` Takashi Iwai
2014-04-30 17:05   ` Joschi Brauchle
2014-05-02  7:31     ` Takashi Iwai
2014-05-02  8:16       ` Joschi Brauchle
2014-05-02  8:21         ` Takashi Iwai
2014-05-02  9:02           ` Joschi Brauchle
2014-05-02  9:27             ` Joschi Brauchle
2014-05-02 13:59               ` Joschi Brauchle
2014-05-06  9:46                 ` Takashi Iwai
2014-05-06 13:49                   ` Joschi Brauchle
2014-05-06 15:20                     ` Joschi Brauchle
2014-05-06 15:41                       ` Takashi Iwai [this message]
2014-05-07  9:28                         ` Joschi Brauchle
2014-05-07  9:41                           ` Takashi Iwai
2014-05-02 16:09             ` Takashi Iwai
     [not found] <32716219.3.1403810435694.JavaMail.jminter@jminter>
2014-06-26 19:23 ` Jim Minter
2014-06-27 10:16   ` Takashi Iwai
  -- strict thread matches above, loose matches on Subject: below --
2014-04-29 15:22 Joschi Brauchle

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=s5hzjiuud7m.wl%tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=alsa-devel@alsa-project.org \
    --cc=joschi.brauchle@tum.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