On Mon, 08 Oct 2018 14:05:22 +0200, Håvard wrote:
I am sorry for replying once again, but since we are already fixing this, we might as well look into another one.
Another one is another one. No need to continue this too lengthy thread.
So, just start a new thread, involve the right people in Cc so that we can continue working on it. Picking up random bugzilla entries won't drive us in a better way, unfortunately.
And, don't forget: people dislike top-posting :)
thanks,
Takashi
Many people are reporting that the headphone out port (wich is also an SPDIF out port) is not working. I can get it working by overriding pin 0x16 to Headphone in hdajackretask, but this makes the internal speakers useless.
Reports online of this issue: https://bugzilla.kernel.org/show_bug.cgi?id=190681 https://bugzilla.kernel.org/show_bug.cgi?id=106771 <- - Interesting discussion https://ubuntuforums.org/archive/index.php/t-2261149.html https://rog.asus.com/forum/showthread.php?59557-Asus-G751JY-No-sound-on-Linu... <-- Links to previous https://bbs.archlinux.org/viewtopic.php?id=213137
And, as mentioned in the last link, setting model=asus-mode5 seems to fix that issue.
I know this is much to ask, but could you take a look at this as well? It seems to affect a lot of people - I have been avoiding it by plugging my headset in the line out port, but that should be a last resort option.
-Håvard
Den lør. 6. okt. 2018 kl. 20:31 skrev 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)>]
[2 <text/html; UTF-8 (quoted-printable)>]