* [PATCH v2 1/2] ALSA: hda/realtek: Remove (revert) duplicate Ally X config
2025-02-27 17:51 [PATCH v2 0/2] ALSA: hda/realtek: Sort Ally X properly, fix Asus Z13 2025 audio Antheas Kapenekakis
@ 2025-02-27 17:51 ` Antheas Kapenekakis
2025-02-27 17:51 ` [PATCH v2 2/2] ALSA: hda/realtek: Fix Asus Z13 2025 audio Antheas Kapenekakis
2025-02-28 9:45 ` [PATCH v2 0/2] ALSA: hda/realtek: Sort Ally X properly, fix " Takashi Iwai
2 siblings, 0 replies; 4+ messages in thread
From: Antheas Kapenekakis @ 2025-02-27 17:51 UTC (permalink / raw)
To: linux-sound, kailang
Cc: linux-kernel, perex, tiwai, baojun.xu, simont,
Antheas Kapenekakis
In commit 1e9c708dc3ae ("ALSA: hda/tas2781: Add new quirk for Lenovo,
ASUS, Dell projects") Baojun adds a bunch of projects to the file,
including for the Ally X. Turns out the initial Ally X was not sorted
properly, so the kernel had 2 quirks for it.
The previous quirk overrode the new one due to being earlier and they
are different. When AB testing, the normal pin fixup seems to work ok
but causes a bit of a minor popping. Given the other config is more
complicated and may cause undefined behavior, revert it.
Signed-off-by: Antheas Kapenekakis <lkml@antheas.dev>
---
sound/pci/hda/patch_realtek.c | 8 --------
1 file changed, 8 deletions(-)
diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
index 224616fbec4f..84e60c50cdb4 100644
--- a/sound/pci/hda/patch_realtek.c
+++ b/sound/pci/hda/patch_realtek.c
@@ -7690,7 +7690,6 @@ enum {
ALC285_FIXUP_THINKPAD_X1_GEN7,
ALC285_FIXUP_THINKPAD_HEADSET_JACK,
ALC294_FIXUP_ASUS_ALLY,
- ALC294_FIXUP_ASUS_ALLY_X,
ALC294_FIXUP_ASUS_ALLY_PINS,
ALC294_FIXUP_ASUS_ALLY_VERBS,
ALC294_FIXUP_ASUS_ALLY_SPEAKER,
@@ -9138,12 +9137,6 @@ static const struct hda_fixup alc269_fixups[] = {
.chained = true,
.chain_id = ALC294_FIXUP_ASUS_ALLY_PINS
},
- [ALC294_FIXUP_ASUS_ALLY_X] = {
- .type = HDA_FIXUP_FUNC,
- .v.func = tas2781_fixup_i2c,
- .chained = true,
- .chain_id = ALC294_FIXUP_ASUS_ALLY_PINS
- },
[ALC294_FIXUP_ASUS_ALLY_PINS] = {
.type = HDA_FIXUP_PINS,
.v.pins = (const struct hda_pintbl[]) {
@@ -10644,7 +10637,6 @@ static const struct hda_quirk alc269_fixup_tbl[] = {
SND_PCI_QUIRK(0x1043, 0x1740, "ASUS UX430UA", ALC295_FIXUP_ASUS_DACS),
SND_PCI_QUIRK(0x1043, 0x17d1, "ASUS UX431FL", ALC294_FIXUP_ASUS_DUAL_SPK),
SND_PCI_QUIRK(0x1043, 0x17f3, "ROG Ally NR2301L/X", ALC294_FIXUP_ASUS_ALLY),
- SND_PCI_QUIRK(0x1043, 0x1eb3, "ROG Ally X RC72LA", ALC294_FIXUP_ASUS_ALLY_X),
SND_PCI_QUIRK(0x1043, 0x1863, "ASUS UX6404VI/VV", ALC245_FIXUP_CS35L41_SPI_2),
SND_PCI_QUIRK(0x1043, 0x1881, "ASUS Zephyrus S/M", ALC294_FIXUP_ASUS_GX502_PINS),
SND_PCI_QUIRK(0x1043, 0x18b1, "Asus MJ401TA", ALC256_FIXUP_ASUS_HEADSET_MIC),
--
2.48.1
^ permalink raw reply related [flat|nested] 4+ messages in thread* [PATCH v2 2/2] ALSA: hda/realtek: Fix Asus Z13 2025 audio
2025-02-27 17:51 [PATCH v2 0/2] ALSA: hda/realtek: Sort Ally X properly, fix Asus Z13 2025 audio Antheas Kapenekakis
2025-02-27 17:51 ` [PATCH v2 1/2] ALSA: hda/realtek: Remove (revert) duplicate Ally X config Antheas Kapenekakis
@ 2025-02-27 17:51 ` Antheas Kapenekakis
2025-02-28 9:45 ` [PATCH v2 0/2] ALSA: hda/realtek: Sort Ally X properly, fix " Takashi Iwai
2 siblings, 0 replies; 4+ messages in thread
From: Antheas Kapenekakis @ 2025-02-27 17:51 UTC (permalink / raw)
To: linux-sound, kailang
Cc: linux-kernel, perex, tiwai, baojun.xu, simont,
Antheas Kapenekakis, Kyle Gospodnetich
Use the basic quirk for this type of amplifier. Sound works in speakers,
headphones, and microphone. Whereas none worked before.
Tested-by: Kyle Gospodnetich <me@kylegospodneti.ch>
Signed-off-by: Antheas Kapenekakis <lkml@antheas.dev>
---
sound/pci/hda/patch_realtek.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
index 84e60c50cdb4..67c06ee9e332 100644
--- a/sound/pci/hda/patch_realtek.c
+++ b/sound/pci/hda/patch_realtek.c
@@ -10692,6 +10692,7 @@ static const struct hda_quirk alc269_fixup_tbl[] = {
SND_PCI_QUIRK(0x1043, 0x1f1f, "ASUS H7604JI/JV/J3D", ALC245_FIXUP_CS35L41_SPI_2),
SND_PCI_QUIRK(0x1043, 0x1f62, "ASUS UX7602ZM", ALC245_FIXUP_CS35L41_SPI_2),
SND_PCI_QUIRK(0x1043, 0x1f92, "ASUS ROG Flow X16", ALC289_FIXUP_ASUS_GA401),
+ SND_PCI_QUIRK(0x1043, 0x1fb3, "ASUS ROG Flow Z13 GZ302EA", ALC287_FIXUP_CS35L41_I2C_2),
SND_PCI_QUIRK(0x1043, 0x3030, "ASUS ZN270IE", ALC256_FIXUP_ASUS_AIO_GPIO2),
SND_PCI_QUIRK(0x1043, 0x31d0, "ASUS Zen AIO 27 Z272SD_A272SD", ALC274_FIXUP_ASUS_ZEN_AIO_27),
SND_PCI_QUIRK(0x1043, 0x3a20, "ASUS G614JZR", ALC285_FIXUP_ASUS_SPI_REAR_SPEAKERS),
--
2.48.1
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH v2 0/2] ALSA: hda/realtek: Sort Ally X properly, fix Asus Z13 2025 audio
2025-02-27 17:51 [PATCH v2 0/2] ALSA: hda/realtek: Sort Ally X properly, fix Asus Z13 2025 audio Antheas Kapenekakis
2025-02-27 17:51 ` [PATCH v2 1/2] ALSA: hda/realtek: Remove (revert) duplicate Ally X config Antheas Kapenekakis
2025-02-27 17:51 ` [PATCH v2 2/2] ALSA: hda/realtek: Fix Asus Z13 2025 audio Antheas Kapenekakis
@ 2025-02-28 9:45 ` Takashi Iwai
2 siblings, 0 replies; 4+ messages in thread
From: Takashi Iwai @ 2025-02-28 9:45 UTC (permalink / raw)
To: Antheas Kapenekakis
Cc: linux-sound, kailang, linux-kernel, perex, tiwai, baojun.xu,
simont
On Thu, 27 Feb 2025 18:51:05 +0100,
Antheas Kapenekakis wrote:
>
> The Ally X has two quirks in the kernel currently. This is due to the
> previous quirk not being sorted properly, whoopsies. Therefore, only one
> has to stay. During AB testing, the configs were found to be close to
> identical. Around 20 reboots later, it seems they feature a small pop
> when at full volume on boot at an identical probability.
>
> Then, add the Asus Z13 2025 quirk to the list. In the V1 of this series,
> the Ally config was used, which (see above) is a bit ambitious. Testing
> found that the stock quirk works just as well, so use that instead.
>
> @Stefan: the Z13 is a big launch for Asus, could we do something to
> expedite things just for this laptop? As far as Bazzite is concerned,
> we shipped the stock quirk as of today, and included the firmware. This
> is moreso for mainline distributions.
>
> The Z13 was tested with the following firmware files:
>
> sha256sum *
> 79cc046b2a1a89fd250dadfb47bdcb8a3c3d7df80452079f64e5417b237f4889 cs35l41-dsp1-spk-cali-10431fb3-l0.bin
> 3027249f8a340a8fb0034d67e5af8787789c83e9b3623664dab744668272b54e cs35l41-dsp1-spk-cali-10431fb3-r0.bin
> a8f3d1735c03383cd8567713e510034b0f98df0264fa7135e6e8186e0ea346de cs35l41-dsp1-spk-prot-10431fb3-l0.bin
> f0cdcf6d72bc591dec8a7bacba3ad8b46742207807c1e3c7787171de66e6f815 cs35l41-dsp1-spk-prot-10431fb3-r0.bin
> e4e73d47203e0badc913b4a1317d48cb0e1b54d7d737f339148068f0cde34b16 cs35l41-dsp1-spk-prot-10431fb3.wmfw
>
> Changelog since V1:
> - Revert Ally X config to use the stock quirk instead
> - Use a stock quirk for the Z13 too
>
> Antheas Kapenekakis (2):
> ALSA: hda/realtek: Remove (revert) duplicate Ally X config
> ALSA: hda/realtek: Fix Asus Z13 2025 audio
Applied both patches now to for-linus branch.
I added Fixes tag to the former.
thanks,
Takashi
^ permalink raw reply [flat|nested] 4+ messages in thread