Sorry, mutt's being insane uncooperative. Here's the format-patch attached.
On Sat, May 20, 2023 at 11:21 AM Adam Stylinski <[email protected]> wrote:
>
> Mutt + SMTP + Gmail are acting up and I had to head out. I'll get this sent out tonight, though. Did you want me to cc the mailing list too?
>
> On Sat, May 20, 2023, 10:27 AM Adam Stylinski <[email protected]> wrote:
>>
>> Correct. A format-patch output is coming soon, I'm just waiting for a
>> proper clone to finish.
>>
>> On Sat, May 20, 2023 at 6:42 AM Takashi Iwai <[email protected]> wrote:
>> >
>> > On Fri, 19 May 2023 05:53:09 +0200,
>> > Xian Wang wrote:
>> > >
>> > > On 18/05/2023 20:02, Adam Stylinski wrote:
>> > > > Hello,
>> > > >
>> > > > I noticed you patched in a quirk in basically the same place I am for
>> > > > a different EVGA board. This quirk is needed for me as well but it
>> > > > would seem the bugzilla and mailing lists are ignored. I was hoping
>> > > > maybe I could draw your attention to this?
>> > > >
>> > > > https://bugzilla.kernel.org/show_bug.cgi?id=67071
>> > > >
>> > > > I suffer a similar issue (namely surround is disabled without enabling
>> > > > this quirk). I still have some temperamental behavior with the quirk
>> > > > in place (the port mapping doesn't always seem consistent boot to
>> > > > boot, often needing several reboots before all channels are working).
>> > > > I'm not certain if this is a quirk of loading the firmware or what.
>> > > > At the very least, not having to patch this locally every kernel
>> > > > update would be nice. I'm CC'ing the maintainer who signed off on the
>> > > > patch as well, really hoping to get some attention on this.
>> > >
>> > > Hi Adam,
>> > >
>> > > Unfortunately I haven't experienced the flakiness of surround sound
>> > > mentioned in the bug. The channel mapping seems correct and is
>> > > consistent from boot to boot.
>> > >
>> > > For not patching locally every time there is a new kernel, are you
>> > > open to submit that one-line patch up?
>> >
>> > So you need only the addition of
>> > SND_PCI_QUIRK(0x3842, 0x104b, "EVGA X299 Dark", QUIRK_R3DI)
>> > ?
>> >
>> > If so, I can submit a oneliner from my side, too. But it's still
>> > better to be submitted from the person who actually tested the patch,
>> > of course.
>> >
>> >
>> > Takashi
On Sun, 21 May 2023 04:29:45 +0200,
Adam Stylinski wrote:
>
> Sorry, mutt's being insane uncooperative. Here's the format-patch attached.
>
> On Sat, May 20, 2023 at 11:21 AM Adam Stylinski <[email protected]> wrote:
> >
> > Mutt + SMTP + Gmail are acting up and I had to head out. I'll get this sent out tonight, though. Did you want me to cc the mailing list too?
> >
> > On Sat, May 20, 2023, 10:27 AM Adam Stylinski <[email protected]> wrote:
> >>
> >> Correct. A format-patch output is coming soon, I'm just waiting for a
> >> proper clone to finish.
> >>
> >> On Sat, May 20, 2023 at 6:42 AM Takashi Iwai <[email protected]> wrote:
> >> >
> >> > On Fri, 19 May 2023 05:53:09 +0200,
> >> > Xian Wang wrote:
> >> > >
> >> > > On 18/05/2023 20:02, Adam Stylinski wrote:
> >> > > > Hello,
> >> > > >
> >> > > > I noticed you patched in a quirk in basically the same place I am for
> >> > > > a different EVGA board. This quirk is needed for me as well but it
> >> > > > would seem the bugzilla and mailing lists are ignored. I was hoping
> >> > > > maybe I could draw your attention to this?
> >> > > >
> >> > > > https://bugzilla.kernel.org/show_bug.cgi?id=67071
> >> > > >
> >> > > > I suffer a similar issue (namely surround is disabled without enabling
> >> > > > this quirk). I still have some temperamental behavior with the quirk
> >> > > > in place (the port mapping doesn't always seem consistent boot to
> >> > > > boot, often needing several reboots before all channels are working).
> >> > > > I'm not certain if this is a quirk of loading the firmware or what.
> >> > > > At the very least, not having to patch this locally every kernel
> >> > > > update would be nice. I'm CC'ing the maintainer who signed off on the
> >> > > > patch as well, really hoping to get some attention on this.
> >> > >
> >> > > Hi Adam,
> >> > >
> >> > > Unfortunately I haven't experienced the flakiness of surround sound
> >> > > mentioned in the bug. The channel mapping seems correct and is
> >> > > consistent from boot to boot.
> >> > >
> >> > > For not patching locally every time there is a new kernel, are you
> >> > > open to submit that one-line patch up?
> >> >
> >> > So you need only the addition of
> >> > SND_PCI_QUIRK(0x3842, 0x104b, "EVGA X299 Dark", QUIRK_R3DI)
> >> > ?
> >> >
> >> > If so, I can submit a oneliner from my side, too. But it's still
> >> > better to be submitted from the person who actually tested the patch,
> >> > of course.
> >> >
> >> >
> >> > Takashi
> From fd1ff1aff3a70716b4d6d99a89934aea5d718cad Mon Sep 17 00:00:00 2001
> From: Adam Stylinski <[email protected]>
> Date: Sat, 20 May 2023 10:31:54 -0400
> Subject: [PATCH] ALSA: hda/ca0132: add quirk for EVGA X299 DARK
>
> This quirk is necessary for surround and other DSP effects to work
> with the onboard ca0132 based audio chipset for the EVGA X299 dark
> mainboard.
>
> Signed-off-by: Adam Stylinski <[email protected]>
Could you rather submit to alsa-devel ML (plus Cc to LKML)?
thanks,
Takashi
>
> ---
> sound/pci/hda/patch_ca0132.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/sound/pci/hda/patch_ca0132.c b/sound/pci/hda/patch_ca0132.c
> index 099722ebaed8..748a3c40966e 100644
> --- a/sound/pci/hda/patch_ca0132.c
> +++ b/sound/pci/hda/patch_ca0132.c
> @@ -1306,6 +1306,7 @@ static const struct snd_pci_quirk ca0132_quirks[] = {
> SND_PCI_QUIRK(0x1458, 0xA026, "Gigabyte G1.Sniper Z97", QUIRK_R3DI),
> SND_PCI_QUIRK(0x1458, 0xA036, "Gigabyte GA-Z170X-Gaming 7", QUIRK_R3DI),
> SND_PCI_QUIRK(0x3842, 0x1038, "EVGA X99 Classified", QUIRK_R3DI),
> + SND_PCI_QUIRK(0x3842, 0x104b, "EVGA X299 Dark", QUIRK_R3DI),
> SND_PCI_QUIRK(0x3842, 0x1055, "EVGA Z390 DARK", QUIRK_R3DI),
> SND_PCI_QUIRK(0x1102, 0x0013, "Recon3D", QUIRK_R3D),
> SND_PCI_QUIRK(0x1102, 0x0018, "Recon3D", QUIRK_R3D),
> --
> 2.40.1
>