On Tue, 17 Sep 2019 00:07:56 +0200, James McDonnell wrote:
On Mon, Sep 16, 2019 at 11:50:42PM +0200, Takashi Iwai wrote:
On Mon, 16 Sep 2019 16:53:38 +0200, James McDonnell wrote:
@@ -7680,6 +7690,11 @@ static const struct snd_hda_pin_quirk alc269_pin_fixup_tbl[] = { {0x17, 0x90170110}, {0x1a, 0x03011020}, {0x21, 0x03211030}),
- SND_HDA_PIN_QUIRK(0x10ec0298, 0x1028, "Dell", ALC298_FIXUP_ALIENWARE_MIC_NO_PRESENCE,
{0x12, 0xb7a60140},{0x17, 0x90170110},{0x1a, 0x03a11030},{0x21, 0x03211020}),It's a slightly uncertain whether this is 100% safe to apply via the pin-quirk table, but also there is no big other reason to push back, so I applied it now.
I've been running with this patched to my kernel for a while now. Any particular things I should do to test?
Not in your side. My concern is whether this may hit too widely to other devices falsely because it's matching with the pin config patterns. The alternative is to match with PCI (or codec) SSID.
But the pin config matching is done in a lower priority, so let's hope my afraid isn't true :)
thanks,
Takashi
Some background on how I got to this patch was I couldn't get the mic to work with a TRRS pinout headset. I discovered that if I set modprobe options for laptop-amic and dell-headset-multi in a particular order the mic would start working. I worked backwards to find this solution.
Thanks!
Takashi
Cheers, James