From patchwork Thu Feb 27 17:51:06 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Antheas Kapenekakis X-Patchwork-Id: 13994964 Received: from linux1587.grserver.gr (linux1587.grserver.gr [185.138.42.100]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4910B1DDC22; Thu, 27 Feb 2025 17:51:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.138.42.100 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1740678679; cv=none; b=czKHzwz6Y8EdsXDzhQrz6v2zmAYXSuEGIUt8Sqw89/s5cxd27ybHxI5nmZlJuRSG/zzJpQGkkdyL1jxjnKWUUWIEul5qDjuKozC9EwibZZQgns3LvFKaI48UYhGcZWD7Q1b+TQCf9DO06mq+yj4XS7gI+Gp9RGmooeb04mt0SCo= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1740678679; c=relaxed/simple; bh=ve1tn5m1IfcbDWyao/yi2LQcvxXCT0Ws+PInmLNUOac=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=Myk9X9/95ZrlOXYaDhsPXXUZ8Qy7C/vlxwbsELwR0hz2PX287jtnpfdqltphZ7ZFmzIjDPdiAjIjMAbublXC55y4Dgl0MqWePbJQfJ+jjGN0MbIruwuoBHYBlx54HlOH4ZzDS9DpH4HD4AEWmDdm5oPQPX5XZFXmGQ87PLXqERY= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=antheas.dev; spf=pass smtp.mailfrom=antheas.dev; dkim=pass (1024-bit key) header.d=antheas.dev header.i=@antheas.dev header.b=wyJPtQDc; arc=none smtp.client-ip=185.138.42.100 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=antheas.dev Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=antheas.dev Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=antheas.dev header.i=@antheas.dev header.b="wyJPtQDc" Received: from localhost.localdomain (unknown [IPv6:2a05:f6c2:511b:0:8d8a:5967:d692:ea4e]) by linux1587.grserver.gr (Postfix) with ESMTPSA id E0C882E0904D; Thu, 27 Feb 2025 19:51:13 +0200 (EET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=antheas.dev; s=default; t=1740678675; bh=klaMGTWuBptyqX/z1km+OwOQwMv+T6B+Vl6r4qUbBYo=; h=From:To:Subject; b=wyJPtQDcv7RaO7zCrvd7sU44rdTmdHPQ5fBXWm2RuaWROzyez/lreaFsxLFws0Pzb rRqbp4bFpoLxQWcsHJq2tNykV5SvO3RD8p0l1QacvSdlCG9UTofjtrXuyJn1A3Ld+Z yNsp1dDBLy0M731Cywr0jMeKpxs4dDcUIC7R0V9g= Authentication-Results: linux1587.grserver.gr; spf=pass (sender IP is 2a05:f6c2:511b:0:8d8a:5967:d692:ea4e) smtp.mailfrom=lkml@antheas.dev smtp.helo=localhost.localdomain Received-SPF: pass (linux1587.grserver.gr: connection is authenticated) From: Antheas Kapenekakis To: linux-sound@vger.kernel.org, kailang@realtek.com Cc: linux-kernel@vger.kernel.org, perex@perex.cz, tiwai@suse.com, baojun.xu@ti.com, simont@opensource.cirrus.com, Antheas Kapenekakis Subject: [PATCH v2 1/2] ALSA: hda/realtek: Remove (revert) duplicate Ally X config Date: Thu, 27 Feb 2025 18:51:06 +0100 Message-ID: <20250227175107.33432-2-lkml@antheas.dev> X-Mailer: git-send-email 2.48.1 In-Reply-To: <20250227175107.33432-1-lkml@antheas.dev> References: <20250227175107.33432-1-lkml@antheas.dev> Precedence: bulk X-Mailing-List: linux-sound@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-PPP-Message-ID: <174067867491.23226.12527222830439990859@linux1587.grserver.gr> X-PPP-Vhost: antheas.dev X-Virus-Scanned: clamav-milter 0.103.11 at linux1587.grserver.gr X-Virus-Status: Clean 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 --- 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), From patchwork Thu Feb 27 17:51:07 2025 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Antheas Kapenekakis X-Patchwork-Id: 13994965 Received: from linux1587.grserver.gr (linux1587.grserver.gr [185.138.42.100]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id B94681D5CD3; Thu, 27 Feb 2025 17:51:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.138.42.100 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1740678682; cv=none; b=LDPzPuldqV5JQG9kNAi3JE+/aM/zzvZtDJZt0xh4GIglAFCImz0qBKzr5jU3ARezDjUP349TbCuyOXKcDfJe+/pjjn6Y+7OryiezADLGDIolsMAjLmfjlQjMCqg7IUzu/vVX2Ns5GZ4qiyh9VHogKP55fycWvrrI0IZyLzvewYU= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1740678682; c=relaxed/simple; bh=5HJSoQYB/GNMpDTh++aUhKd+mAn9BqMW5mqV2UA3wT4=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=puaBG4qmYR6uxjhbpXHhJYs0iQEqbJihgrHqloUAp05SL6FVo5qbC3s1f1TOqdDU3ovq2SH1qxg3bjgvS33byx9a6+6IjEAXo+qI9V4LoAmeKkcnL7KyERRxPfycQPulZVRSzlPJ1nlgTFMouftkbYh9hHd6nByny4oqlJDwXi0= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=antheas.dev; spf=pass smtp.mailfrom=antheas.dev; dkim=pass (1024-bit key) header.d=antheas.dev header.i=@antheas.dev header.b=o28bQWrv; arc=none smtp.client-ip=185.138.42.100 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=antheas.dev Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=antheas.dev Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=antheas.dev header.i=@antheas.dev header.b="o28bQWrv" Received: from localhost.localdomain (unknown [IPv6:2a05:f6c2:511b:0:8d8a:5967:d692:ea4e]) by linux1587.grserver.gr (Postfix) with ESMTPSA id 610B22E000C0; Thu, 27 Feb 2025 19:51:15 +0200 (EET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=antheas.dev; s=default; t=1740678676; bh=UCsnlrVcfEZ9dmEDvjRCrlbNeiteXx7Fw8k3nqnief8=; h=From:To:Subject; b=o28bQWrvdkExkpGVCug4p1Us/oekTqZ1XE4y1fSmQEe7XH9MPyAiQ5rdnt9aN6VEu WU/VTpzYvRP+7z4cc6gpcg6w95CYast294IYRpI6kf6sjGQ+RkeY8aa6KMkLdPxbWz 9EKNRVrwc0ofVtxbPhRoyPJ41GefZfXehLBM+KyA= Authentication-Results: linux1587.grserver.gr; spf=pass (sender IP is 2a05:f6c2:511b:0:8d8a:5967:d692:ea4e) smtp.mailfrom=lkml@antheas.dev smtp.helo=localhost.localdomain Received-SPF: pass (linux1587.grserver.gr: connection is authenticated) From: Antheas Kapenekakis To: linux-sound@vger.kernel.org, kailang@realtek.com Cc: linux-kernel@vger.kernel.org, perex@perex.cz, tiwai@suse.com, baojun.xu@ti.com, simont@opensource.cirrus.com, Antheas Kapenekakis , Kyle Gospodnetich Subject: [PATCH v2 2/2] ALSA: hda/realtek: Fix Asus Z13 2025 audio Date: Thu, 27 Feb 2025 18:51:07 +0100 Message-ID: <20250227175107.33432-3-lkml@antheas.dev> X-Mailer: git-send-email 2.48.1 In-Reply-To: <20250227175107.33432-1-lkml@antheas.dev> References: <20250227175107.33432-1-lkml@antheas.dev> Precedence: bulk X-Mailing-List: linux-sound@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-PPP-Message-ID: <174067867642.23360.17289059988517600062@linux1587.grserver.gr> X-PPP-Vhost: antheas.dev X-Virus-Scanned: clamav-milter 0.103.11 at linux1587.grserver.gr X-Virus-Status: Clean Use the basic quirk for this type of amplifier. Sound works in speakers, headphones, and microphone. Whereas none worked before. Tested-by: Kyle Gospodnetich Signed-off-by: Antheas Kapenekakis --- 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),