Re: Microphone detected, but no output for all ASUS G751xx with ALC668 chipset

[Håvard <hovardslill@gmail.com>]

OK!

Thank you very much for all that you have done!!

I am looking forward to seeing this in (probably) the 4.20 kernel!

-Håvard

Den lør. 6. okt. 2018 kl. 15:57 skrev Takashi Iwai <tiwai@suse.de>:

> On Sat, 06 Oct 2018 10:20:37 +0200,
> Håvard wrote:
> >
> > Ok!
> >
> > Thank you very much!
> >
> > So for them to work, they have to set model=asus-g751 ?
>
> They can test in that way, and they need to report their PCI SSIDs for
> applying the quirk as default.
>
>
> Takashi
>
> > Sorry for all these questions, but I want to respond to these people as
> > fast as possible!
> >
> > -Håvard
> >
> > Den lør. 6. okt. 2018 kl. 08:55 skrev Takashi Iwai <tiwai@suse.de>:
> >
> > > On Fri, 05 Oct 2018 21:56:20 +0200,
> > > Håvard wrote:
> > > >
> > > > I couldn't get the patch installed automatically, so I added the
> changed
> > > > code myself.
> > > >
> > > > Everything works fine and as expected right after reboot! :) Just
> like we
> > > > want!
> > > >
> > > > I don't understand. So this won't work for other G751xx users? Or do
> they
> > > > have to set model=asus-g751? I'm thinking of reaching out and saying
> it
> > > > will be fixed in 4.19.
> > >
> > > We need to know the exact PCI SSIDs for the matching models.
> > > Yours is 1043:12ff, and others might be different.  They can be added
> > > eventually to the quirk table in the same way once when we are
> > > informed.
> > >
> > > Since it's already a very late stage for 4.19, this fix will go into
> > > 4.20 at earliest.
> > >
> > >
> > > thanks,
> > >
> > > Takashi
> > >
> > > >
> > > > Thank you!
> > > >
> > > > -Håvard
> > > >
> > > > Den fre. 5. okt. 2018 kl. 14:46 skrev Takashi Iwai <tiwai@suse.de>:
> > > >
> > > > > On Fri, 05 Oct 2018 12:31:37 +0200,
> > > > > Håvard wrote:
> > > > > >
> > > > > > Thank you so much!
> > > > > >
> > > > > > Will this work for all G751 models, and will this be in the 4.19
> > > kernel?
> > > > >
> > > > > It's only for yours (the matching PCI SSID).  And it will be
> included
> > > > > only when you test the patch and confirm it working.  So, please
> test
> > > > > it at first.  Then I'll merge after the test rest.
> > > > >
> > > > >
> > > > > thanks,
> > > > >
> > > > > Takashi
> > > > >
> > > > >
> > > > > > I am very thankful for all your hard work!!
> > > > > >
> > > > > > -Håvard
> > > > > >
> > > > > > Den fre. 5. okt. 2018 kl. 12:29 skrev Takashi Iwai <
> tiwai@suse.de>:
> > > > > >
> > > > > > > On Fri, 05 Oct 2018 12:10:52 +0200,
> > > > > > > Håvard wrote:
> > > > > > > >
> > > > > > > > I'll try to answer your previous email first.
> > > > > > > >
> > > > > > > > Everything seems identical to how it was in the 4.18-gentoo
> > > kernel
> > > > > when
> > > > > > > not
> > > > > > > > setting the model=alc668-headset option
> > > > > > > >
> > > > > > > > And using the trick we found:
> > > > > > > >
> > > > > > > > ./hda-verb /dev/snd/hwC0D0 0x20 0x500 0xc3
> > > > > > > > ./hda-verb /dev/snd/hwC0D0 0x20 0x440 0x00
> > > > > > >
> > > > > > > OK, then the following patch may work.  Give it a try.
> > > > > > >
> > > > > > >
> > > > > > > > Made it so loopback worked perfectly fine for both earpieces.
> > > > > > > >
> > > > > > > > One curiosity however (this is just a small thing) is that
> the
> > > > > "base" mic
> > > > > > > > level is far lower than when the "Mic" option in alsamixer is
> > > set to
> > > > > > > 100. I
> > > > > > > > don't know if it was this way in 4.18-gentoo, but that's the
> only
> > > > > thing
> > > > > > > > that doesnt seem correct.
> > > > > > > > Here is a screenshot explaining it:
> > > > > > > > http://i.imgur.com/dKPELX6.png
> > > > > > >
> > > > > > > The Mic volume is only for analog loopback, so it can be
> normal.
> > > > > > >
> > > > > > >
> > > > > > > Takashi
> > > > > > >
> > > > > > > ---
> > > > > > > --- a/sound/pci/hda/patch_realtek.c
> > > > > > > +++ b/sound/pci/hda/patch_realtek.c
> > > > > > > @@ -7738,6 +7738,7 @@ enum {
> > > > > > >         ALC662_FIXUP_ASUS_Nx50,
> > > > > > >         ALC668_FIXUP_ASUS_Nx51_HEADSET_MODE,
> > > > > > >         ALC668_FIXUP_ASUS_Nx51,
> > > > > > > +       ALC668_FIXUP_ASUS_G751,
> > > > > > >         ALC891_FIXUP_HEADSET_MODE,
> > > > > > >         ALC891_FIXUP_DELL_MIC_NO_PRESENCE,
> > > > > > >         ALC662_FIXUP_ACER_VERITON,
> > > > > > > @@ -8007,6 +8008,14 @@ static const struct hda_fixup
> > > alc662_fixups[] =
> > > > > {
> > > > > > >                 .chained = true,
> > > > > > >                 .chain_id =
> ALC668_FIXUP_ASUS_Nx51_HEADSET_MODE,
> > > > > > >         },
> > > > > > > +       [ALC668_FIXUP_ASUS_G751] = {
> > > > > > > +               .type = HDA_FIXUP_VERBS,
> > > > > > > +               .v.verbs = (const struct hda_verb[]) {
> > > > > > > +                       { 0x20, AC_VERB_SET_COEF_INDEX, 0xc3 },
> > > > > > > +                       { 0x20, AC_VERB_SET_PROC_COEF, 0x4000
> },
> > > > > > > +                       {}
> > > > > > > +               },
> > > > > > > +       },
> > > > > > >         [ALC891_FIXUP_HEADSET_MODE] = {
> > > > > > >                 .type = HDA_FIXUP_FUNC,
> > > > > > >                 .v.func = alc_fixup_headset_mode,
> > > > > > > @@ -8080,6 +8089,7 @@ static const struct snd_pci_quirk
> > > > > alc662_fixup_tbl[]
> > > > > > > = {
> > > > > > >         SND_PCI_QUIRK(0x1043, 0x11cd, "Asus N550",
> > > > > ALC662_FIXUP_ASUS_Nx50),
> > > > > > >         SND_PCI_QUIRK(0x1043, 0x13df, "Asus N550JX",
> > > > > ALC662_FIXUP_BASS_1A),
> > > > > > >         SND_PCI_QUIRK(0x1043, 0x129d, "Asus N750",
> > > > > ALC662_FIXUP_ASUS_Nx50),
> > > > > > > +       SND_PCI_QUIRK(0x1043, 0x12ff, "ASUS G751",
> > > > > ALC668_FIXUP_ASUS_G751),
> > > > > > >         SND_PCI_QUIRK(0x1043, 0x1477, "ASUS N56VZ",
> > > > > > > ALC662_FIXUP_BASS_MODE4_CHMAP),
> > > > > > >         SND_PCI_QUIRK(0x1043, 0x15a7, "ASUS UX51VZH",
> > > > > > > ALC662_FIXUP_BASS_16),
> > > > > > >         SND_PCI_QUIRK(0x1043, 0x177d, "ASUS N551",
> > > > > ALC668_FIXUP_ASUS_Nx51),
> > > > > > > @@ -8184,6 +8194,7 @@ static const struct hda_model_fixup
> > > > > > > alc662_fixup_models[] = {
> > > > > > >         {.id = ALC668_FIXUP_DELL_XPS13, .name = "dell-xps13"},
> > > > > > >         {.id = ALC662_FIXUP_ASUS_Nx50, .name = "asus-nx50"},
> > > > > > >         {.id = ALC668_FIXUP_ASUS_Nx51, .name = "asus-nx51"},
> > > > > > > +       {.id = ALC668_FIXUP_ASUS_G751, .name = "asus-g751"},
> > > > > > >         {.id = ALC891_FIXUP_HEADSET_MODE, .name =
> > > "alc891-headset"},
> > > > > > >         {.id = ALC891_FIXUP_DELL_MIC_NO_PRESENCE, .name =
> > > > > > > "alc891-headset-multi"},
> > > > > > >         {.id = ALC662_FIXUP_ACER_VERITON, .name =
> "acer-veriton"},
> > > > > > >
> > > > > > [2  <text/html; UTF-8 (quoted-printable)>]
> > > > > >
> > > > >
> > > > [2  <text/html; UTF-8 (quoted-printable)>]
> > > >
> > >
> > [2  <text/html; UTF-8 (quoted-printable)>]
> >
>
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel