linux-kernel - RE: [PATCH] ALSA: hda: Add new pci id for AMD GPU display HD audio controller

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID:
 <BL1PR12MB51449240DD771DB4C8005890F766A@BL1PR12MB5144.namprd12.prod.outlook.com>
Date: Thu, 29 May 2025 20:26:07 +0000
From: "Deucher, Alexander" <Alexander.Deucher@....com>
To: "Mukunda, Vijendar" <Vijendar.Mukunda@....com>, "perex@...ex.cz"
	<perex@...ex.cz>, "tiwai@...e.com" <tiwai@...e.com>
CC: "alsa-devel@...a-project.org" <alsa-devel@...a-project.org>, "Dommati,
 Sunil-kumar" <Sunil-kumar.Dommati@....com>, "linux-sound@...r.kernel.org"
	<linux-sound@...r.kernel.org>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>
Subject: RE: [PATCH] ALSA: hda: Add new pci id for AMD GPU display HD audio
 controller

[Public]

> -----Original Message-----
> From: Mukunda, Vijendar <Vijendar.Mukunda@....com>
> Sent: Thursday, May 29, 2025 1:38 AM
> To: perex@...ex.cz; tiwai@...e.com
> Cc: alsa-devel@...a-project.org; Dommati, Sunil-kumar <Sunil-
> kumar.Dommati@....com>; Deucher, Alexander
> <Alexander.Deucher@....com>; linux-sound@...r.kernel.org; linux-
> kernel@...r.kernel.org; Mukunda, Vijendar <Vijendar.Mukunda@....com>
> Subject: [PATCH] ALSA: hda: Add new pci id for AMD GPU display HD audio
> controller
>
> Add new pci id for AMD GPU display HD audio controller(device id- 0xab40).
>
> Signed-off-by: Vijendar Mukunda <Vijendar.Mukunda@....com>

Is there a way we can just add a default entry for all ATI HDMI HDA endpoints?  It would avoid then need to add an entry every time we release a new endpoint DID.  Either way:
Reviewed-by: Alex Deucher <alexander.deucher@....com>

> ---
>  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
> 512fb22f5e5e..3f8b2bf6eab5 100644
> --- a/sound/pci/hda/hda_intel.c
> +++ b/sound/pci/hda/hda_intel.c
> @@ -2715,6 +2715,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.45.2