From: Hui Wang <hui.wang@canonical.com>
To: David Henningsson <david.henningsson@canonical.com>
Cc: tiwai@suse.de, alsa-devel@alsa-project.org, stable@vger.kernel.org
Subject: Re: [PATCH] ALSA: hda - Add Dell headset detection quirk for one more laptop model
Date: Wed, 18 Dec 2013 10:56:51 +0800 [thread overview]
Message-ID: <52B10EF3.3090500@canonical.com> (raw)
In-Reply-To: <52B029DA.3030401@canonical.com>
On 12/17/2013 06:39 PM, David Henningsson wrote:
>>>> SND_PCI_QUIRK(0x1028, 0x0638, "Dell Inspiron 5439",
>>>> >>>ALC290_FIXUP_MONO_SPEAKERS),
>>>> >>> SND_PCI_QUIRK(0x1028, 0x063e, "Dell",
>>>> >>>ALC269_FIXUP_DELL1_MIC_NO_PRESENCE),
>>>> >>> SND_PCI_QUIRK(0x1028, 0x063f, "Dell",
>>>> >>>ALC255_FIXUP_DELL1_MIC_NO_PRESENCE),
>>>> >>>+ SND_PCI_QUIRK(0x1028, 0x0640, "Dell",
>>>> >>>ALC269_FIXUP_DELL1_MIC_NO_PRESENCE),
>>> >>Are you sure it shouldn't be ALC255_FIXUP_... in this line instead of
>>> >>ALC269_FIXUP_... ?
>> >I also had the same concern as yours yesterday, so i let ShawnWang
>> >perform the test, he confirmed that adding "model=dell-headset-multi"
>> >can solve this problem. Below is the irc log:
>> >
>> ><hwang4> shawnwang, Does adding "options snd-hda-intel
>> >model=,dell-headset-multi" solve the problem on this machine (10280640)
>> ><shawnwang> hwang4, yes
>> ><hwang4> shawnwang, ok, got it, i will fix it.
> Well, just because the alc269 model works does not mean that the alc255
> model does not work. They are quite similar, with the only difference
> being a few verbs set on initial boot. Maybe those verbs handle some
> corner case that was not tested explicitly by Shawn.
All right, I am going to let Shawn redo the test.
Regards,
Hui.
next prev parent reply other threads:[~2013-12-18 2:56 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-12-17 9:39 [PATCH] ALSA: hda - Add Dell headset detection quirk for one more laptop model Hui Wang
2013-12-17 9:43 ` David Henningsson
2013-12-17 9:57 ` Hui Wang
2013-12-17 10:39 ` David Henningsson
2013-12-18 2:56 ` Hui Wang [this message]
2013-12-18 9:16 ` Hui Wang
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=52B10EF3.3090500@canonical.com \
--to=hui.wang@canonical.com \
--cc=alsa-devel@alsa-project.org \
--cc=david.henningsson@canonical.com \
--cc=stable@vger.kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.