From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 58728224895; Mon, 9 Jun 2025 22:53:38 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1749509618; cv=none; b=VY84ZgrKhR3u3XMKj475rp8yUug24Yhagcn1DYQ+GtWh9cUUU5e9vnsqF1zJhz3ENzT9Wt8CBtLwtm10Mu0EBUV/ymvagw7KOlK1ucW30FvT53BMlHwlxgWmiOs47GTFEIRUzMaWv0V54ZqCbImEGYz0Tmkz76t/0YKCqzLxiwk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1749509618; c=relaxed/simple; bh=dssgEvl4sw/Iv7BRlum/lKbHRZHNU0YpXOrZ6yd8VS8=; h=From:To:Cc:Subject:Date:Message-Id:In-Reply-To:References: MIME-Version:Content-Type; b=XmCiLeq+tPpmj7l4lF0N5J12Soxvj/6Y65Mt2invuexBsE4iu/TdSAcfqT91GdpEr/feQddL9iwXHe0PfI2Uvx328CQjfubEXjWk94RVmNXnWWo0z3AG9ktkl5gvoEmTZeKu+5IQLQ1LIO/dfjP/zgdFITIO2jmhZ/kcyLHjm5c= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=ABl+aVA5; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="ABl+aVA5" Received: by smtp.kernel.org (Postfix) with ESMTPSA id B3C69C4CEEB; Mon, 9 Jun 2025 22:53:36 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1749509618; bh=dssgEvl4sw/Iv7BRlum/lKbHRZHNU0YpXOrZ6yd8VS8=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=ABl+aVA5+cSHHiQ0G7or4Ufke6O/fSmNEFA6u+uVM8OiaWcpFDtKUcMxNCSRxXobn 1fjeDwzat4Gag5FtPn/QYFJL5iAmhjZVA5y5VofZ3+57grkcX5tHF4qC1PdKQGysSy lX1KmCWVRakrL0KQCTqa1JVvnzFJgdM1ZqZj5XsSkbg2//KL5Hxcd+U+kIGKffFj6V 5WpFFphtf7cxbdY34kM7U3xZAojBwKGtyP4UShldBdg0eESrmPt0Vhbo0vjpYdqDWS JeAhkQkGht7SsDb2njH6nvf1Bb676j96ruuP/M0q6hrgKDsXziyHqZoLFwsKZR5xVm zMeFTGJ8uu3ow== From: Sasha Levin To: patches@lists.linux.dev, stable@vger.kernel.org Cc: Vijendar Mukunda , Alex Deucher , Takashi Iwai , Sasha Levin , pierre-louis.bossart@linux.dev, yung-chuan.liao@linux.intel.com, peter.ujfalusi@linux.intel.com, kai.vehmanen@linux.intel.com, hkallweit1@gmail.com, maxtram95@gmail.com, TonyWWang-oc@zhaoxin.com, phasta@kernel.org Subject: [PATCH AUTOSEL 6.6 2/6] ALSA: hda: Add new pci id for AMD GPU display HD audio controller Date: Mon, 9 Jun 2025 18:53:28 -0400 Message-Id: <20250609225333.1444210-2-sashal@kernel.org> X-Mailer: git-send-email 2.39.5 In-Reply-To: <20250609225333.1444210-1-sashal@kernel.org> References: <20250609225333.1444210-1-sashal@kernel.org> Precedence: bulk X-Mailing-List: patches@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-stable: review X-Patchwork-Hint: Ignore X-stable-base: Linux 6.6.93 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit From: Vijendar Mukunda [ Upstream commit ab72bfce7647522e01a181e3600c3d14ff5c143e ] Add new pci id for AMD GPU display HD audio controller(device id- 0xab40). Signed-off-by: Vijendar Mukunda Reviewed-by: Alex Deucher Link: https://patch.msgid.link/20250529053838.2350071-1-Vijendar.Mukunda@amd.com Signed-off-by: Takashi Iwai Signed-off-by: Sasha Levin --- Based on my analysis of the commit and the similar historical commits provided, here is my assessment: **YES** This commit should be backported to stable kernel trees based on the following extensive analysis: **1. Hardware Support Fix Pattern:** The commit follows the exact same pattern as the historical commits marked "YES" for backporting (Similar Commits #2, #3, and #4). All of these commits: - Add new PCI device IDs for hardware support - Use identical driver configuration (`AZX_DRIVER_ATIHDMI_NS | AZX_DCAPS_PRESET_ATI_HDMI_NS | AZX_DCAPS_PM_RUNTIME`) - Are small, self-contained changes with minimal risk **2. Code Analysis:** The change adds exactly one new PCI device entry: ```c { PCI_VDEVICE(ATI, 0xab40), .driver_data = AZX_DRIVER_ATIHDMI_NS | AZX_DCAPS_PRESET_ATI_HDMI_NS | AZX_DCAPS_PM_RUNTIME }, ``` This entry is inserted in the correct sequential order between existing AMD GPU audio devices (after 0xab38) and follows the exact same pattern as all surrounding entries. The driver data flags used are: - `AZX_DRIVER_ATIHDMI_NS`: Standard ATI HDMI driver - `AZX_DCAPS_PRESET_ATI_HDMI_NS`: ATI HDMI preset capabilities - `AZX_DCAPS_PM_RUNTIME`: Runtime power management support **3. Risk Assessment - Minimal:** - **No algorithmic changes**: Pure PCI ID table addition - **No new code paths**: Uses existing, well-tested driver infrastructure - **No configuration changes**: Same capabilities as neighboring AMD GPU devices - **No architectural impact**: Confined to PCI device recognition - **Regression risk**: Nearly zero - only affects this specific AMD GPU (device ID 0xab40) **4. User Impact - Significant:** This enables HD audio functionality for users with new AMD GPU hardware (device ID 0xab40). Without this change, the audio component of these GPUs would not be recognized by the kernel, leaving users without audio output capabilities on affected systems. **5. Stable Tree Criteria Compliance:** - ✅ **Fixes important bug**: Lack of audio support on new hardware - ✅ **Minimal risk**: PCI ID additions have extremely low regression potential - ✅ **Small and contained**: Single line addition following established pattern - ✅ **No new features**: Pure hardware enablement - ✅ **Clear benefit**: Enables audio on new AMD GPUs **6. Historical Precedent:** Similar Commits #2, #3, and #4 (all marked "YES") demonstrate that AMD/ATI GPU audio PCI ID additions are consistently considered appropriate for stable backporting. The only "NO" examples (Similar Commits #1 and #5) were either preparatory changes for future features or lacked hardware urgency. **7. Maintainer Endorsement:** The commit has proper sign-offs including review by Alex Deucher (AMD maintainer), indicating technical correctness and hardware necessity. This commit perfectly fits the stable tree guidelines: it's a minimal, safe hardware enablement fix that provides significant user benefit with virtually no regression risk. sound/pci/hda/hda_intel.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c index 3a0df631d25d9..cbd864e6da05c 100644 --- a/sound/pci/hda/hda_intel.c +++ b/sound/pci/hda/hda_intel.c @@ -2725,6 +2725,9 @@ static const struct pci_device_id azx_ids[] = { { PCI_VDEVICE(ATI, 0xab38), .driver_data = AZX_DRIVER_ATIHDMI_NS | AZX_DCAPS_PRESET_ATI_HDMI_NS | AZX_DCAPS_PM_RUNTIME }, + { PCI_VDEVICE(ATI, 0xab40), + .driver_data = AZX_DRIVER_ATIHDMI_NS | AZX_DCAPS_PRESET_ATI_HDMI_NS | + AZX_DCAPS_PM_RUNTIME }, /* GLENFLY */ { PCI_DEVICE(PCI_VENDOR_ID_GLENFLY, PCI_ANY_ID), .class = PCI_CLASS_MULTIMEDIA_HD_AUDIO << 8, -- 2.39.5