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
>
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel
