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


[-- Attachment #1.1: Type: text/plain, Size: 3729 bytes --]

On 05/06/2014 05:41 PM, Takashi Iwai wrote:
> 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"},
>   	{}
>   };
>
>

I have rebuilt the snd-hda-* modules with the patch above (adapted to 
3.11 kernel) and tested successfully! Sound is working fine while 
docking now.

Thanks!


[-- Attachment #1.2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 4739 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]



  reply	other threads:[~2014-05-07  9:28 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
2014-05-07  9:28                         ` Joschi Brauchle [this message]
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=5369FCB9.4020205@tum.de \
    --to=joschi.brauchle@tum.de \
    --cc=alsa-devel@alsa-project.org \
    --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