From: Sergey 'Jin' Bostandzhyan <jin@mediatomb.cc>
To: Takashi Iwai <tiwai@suse.de>
Cc: alsa-devel@alsa-project.org
Subject: Re: [alsa-devel] Surround speaker connection on Acer 8951G
Date: Mon, 25 Nov 2019 18:39:02 +0100 [thread overview]
Message-ID: <20191125173902.GA27981@xn--80adja5bqm.su> (raw)
In-Reply-To: <20190830114510.GA10027@xn--80adja5bqm.su>
Hi Takashi,
sorry - it's me again about the Acer 8951G LFE speaker.
On Fri, Aug 30, 2019 at 01:45:10PM +0200, Sergey 'Jin' Bostandzhyan wrote:
> > > The below HDA_FIXUP_VERBS does the trick, so I do have all 6 speakers working,
> > > finally!
> > >
> > > {0x01, AC_VERB_SET_GPIO_DIRECTION, 0x02}
> >
> > Actually this must be paired with the corresponding bit of GPIO_DATA,
> > too. Is the bit 0x02 of GPIO_DATA set or cleared? Usually setting it
> > turns on the amp, but sometimes inverted.
>
> If I understood everything correctly, then the bit is set, meaning that the
> GPIO signal is configured as output. I'll be honest, I exported the
> hda-analyzer setting as a python script (nice feature btw) and deducted the
> fixup verb setting from there (relevant part of the hda-analyzer export below):
>
> def set(nid, verb, param):
> verb = (nid << 24) | (verb << 8) | param
> res = ioctl(FD, IOCTL_VERB_WRITE, struct.pack('II', verb, 0))
>
> set(0x01, 0x717, 0x02) # 0x01071702 (SET_GPIO_DIRECTION)
it seems I indeed missed something here regarding GPIO_DATA, I really am
not sure what the influence is, but after updating to Fedora 31 my LFE
stopped working, even with the self compiled 5.4-rc8 kernel which I am running
now (all the time before I was on Fedora 29 and I just backported my patch to
5.2.x and compiled the modules outside the tree after being done with the
patch submission).
So ultimately, it seems I now need to do the following in my fixup
(original commit was 00066e9733f629e536f6b7957de2ce11a85fe15a):
--- a/sound/pci/hda/patch_realtek.c
+++ b/sound/pci/hda/patch_realtek.c
@@ -8875,7 +8875,7 @@ static const struct hda_fixup alc662_fixups[] = {
.v.verbs = (const struct hda_verb[]) {
{0x01, AC_VERB_SET_GPIO_MASK, 0x02},
{0x01, AC_VERB_SET_GPIO_DIRECTION, 0x02},
- {0x01, AC_VERB_SET_GPIO_DATA, 0x00},
+ {0x01, AC_VERB_SET_GPIO_DATA, 0x02},
{ }
},
.chained = true,
My question is: could something on the outside have influence on that? I am
really very, very sure that I have tested LFE on kernel 5.4-rc before
submitting the original patch and it has been working as submitted.
Why did the behavior change now? What else could I have missed?
I guess I have to submit the above change again, but I would like to make
sure that I am not missing something else somewhere, something that could cause
a change of behavior yet again after some future update.
Kind regards,
Jin
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel
next prev parent reply other threads:[~2019-11-25 17:40 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-04-04 19:24 Surround speaker connection on Acer 8951G Sergey 'Jin' Bostandzhyan
2019-07-19 11:12 ` Sergey 'Jin' Bostandzhyan
2019-07-19 14:44 ` Takashi Iwai
2019-07-20 16:54 ` Sergey 'Jin' Bostandzhyan
2019-08-19 19:57 ` Sergey 'Jin' Bostandzhyan
2019-08-22 14:17 ` Takashi Iwai
2019-08-22 20:30 ` Sergey 'Jin' Bostandzhyan
2019-08-29 9:30 ` Takashi Iwai
2019-08-29 10:38 ` Sergey 'Jin' Bostandzhyan
2019-08-29 11:29 ` Takashi Iwai
2019-08-30 11:45 ` Sergey 'Jin' Bostandzhyan
2019-08-30 11:45 ` [alsa-devel] " Sergey 'Jin' Bostandzhyan
2019-08-30 12:22 ` Takashi Iwai
2019-08-30 12:22 ` [alsa-devel] " Takashi Iwai
2019-09-01 19:27 ` Sergey 'Jin' Bostandzhyan
2019-09-01 19:27 ` [alsa-devel] " Sergey 'Jin' Bostandzhyan
2019-09-02 6:41 ` Takashi Iwai
2019-09-02 6:41 ` [alsa-devel] " Takashi Iwai
2019-09-02 21:39 ` Sergey 'Jin' Bostandzhyan
2019-09-05 15:07 ` Takashi Iwai
2019-09-06 9:33 ` [alsa-devel] [PATCH] Add Acer Aspire Ethos 8951G model quirk Sergey Bostandzhyan
2019-09-06 12:13 ` Takashi Iwai
2019-11-25 17:39 ` Sergey 'Jin' Bostandzhyan [this message]
2019-11-27 11:28 ` [alsa-devel] Surround speaker connection on Acer 8951G Takashi Iwai
2019-11-27 16:17 ` Sergey 'Jin' Bostandzhyan
2019-11-27 16:41 ` Takashi Iwai
2019-11-28 15:46 ` Sergey 'Jin' Bostandzhyan
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=20191125173902.GA27981@xn--80adja5bqm.su \
--to=jin@mediatomb.cc \
--cc=alsa-devel@alsa-project.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).