diff mbox series

[1/2] ALSA: hda: Add Intel DG2 PCI ID and HDMI codec vid

Message ID 20211130124732.696896-1-kai.vehmanen@linux.intel.com (mailing list archive)
State New, archived
Headers show
Series [1/2] ALSA: hda: Add Intel DG2 PCI ID and HDMI codec vid | expand

Commit Message

Kai Vehmanen Nov. 30, 2021, 12:47 p.m. UTC
Add HD Audio PCI ID and HDMI codec vendor ID for Intel DG2.

Reviewed-by: Uma Shankar <uma.shankar@intel.com>
Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
---
 sound/pci/hda/hda_intel.c  | 12 +++++++++++-
 sound/pci/hda/patch_hdmi.c |  1 +
 2 files changed, 12 insertions(+), 1 deletion(-)


base-commit: d7aca8d8f548570f650d9b1dfedf6902f56f2bce

Comments

Takashi Iwai Nov. 30, 2021, 2:08 p.m. UTC | #1
On Tue, 30 Nov 2021 13:47:31 +0100,
Kai Vehmanen wrote:
> 
> Add HD Audio PCI ID and HDMI codec vendor ID for Intel DG2.
> 
> Reviewed-by: Uma Shankar <uma.shankar@intel.com>
> Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>

Thanks, applied both patches now.


Takashi
Bjorn Helgaas Oct. 14, 2022, 8:14 p.m. UTC | #2
On Tue, Nov 30, 2021 at 02:47:31PM +0200, Kai Vehmanen wrote:
> Add HD Audio PCI ID and HDMI codec vendor ID for Intel DG2.
> 
> Reviewed-by: Uma Shankar <uma.shankar@intel.com>
> Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
> ---
>  sound/pci/hda/hda_intel.c  | 12 +++++++++++-
>  sound/pci/hda/patch_hdmi.c |  1 +
>  2 files changed, 12 insertions(+), 1 deletion(-)
> 
> diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c
> index 221afacbc7fd..b98d7975adea 100644
> --- a/sound/pci/hda/hda_intel.c
> +++ b/sound/pci/hda/hda_intel.c
> @@ -335,7 +335,10 @@ enum {
>  					((pci)->device == 0x0c0c) || \
>  					((pci)->device == 0x0d0c) || \
>  					((pci)->device == 0x160c) || \
> -					((pci)->device == 0x490d))
> +					((pci)->device == 0x490d) || \
> +					((pci)->device == 0x4f90) || \
> +					((pci)->device == 0x4f91) || \
> +					((pci)->device == 0x4f92))

Would you consider adding these device IDs to the Linux PCI ID
database so lspci knows what they are?  Everything starting with
0x490d is missing.

There's an "Add item" link at the top of https://pci-ids.ucw.cz/read/PC/8086

Bjorn
diff mbox series

Patch

diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c
index 221afacbc7fd..b98d7975adea 100644
--- a/sound/pci/hda/hda_intel.c
+++ b/sound/pci/hda/hda_intel.c
@@ -335,7 +335,10 @@  enum {
 					((pci)->device == 0x0c0c) || \
 					((pci)->device == 0x0d0c) || \
 					((pci)->device == 0x160c) || \
-					((pci)->device == 0x490d))
+					((pci)->device == 0x490d) || \
+					((pci)->device == 0x4f90) || \
+					((pci)->device == 0x4f91) || \
+					((pci)->device == 0x4f92))
 
 #define IS_BXT(pci) ((pci)->vendor == 0x8086 && (pci)->device == 0x5a98)
 
@@ -2477,6 +2480,13 @@  static const struct pci_device_id azx_ids[] = {
 	/* DG1 */
 	{ PCI_DEVICE(0x8086, 0x490d),
 	  .driver_data = AZX_DRIVER_SKL | AZX_DCAPS_INTEL_SKYLAKE},
+	/* DG2 */
+	{ PCI_DEVICE(0x8086, 0x4f90),
+	  .driver_data = AZX_DRIVER_SKL | AZX_DCAPS_INTEL_SKYLAKE},
+	{ PCI_DEVICE(0x8086, 0x4f91),
+	  .driver_data = AZX_DRIVER_SKL | AZX_DCAPS_INTEL_SKYLAKE},
+	{ PCI_DEVICE(0x8086, 0x4f92),
+	  .driver_data = AZX_DRIVER_SKL | AZX_DCAPS_INTEL_SKYLAKE},
 	/* Alderlake-S */
 	{ PCI_DEVICE(0x8086, 0x7ad0),
 	  .driver_data = AZX_DRIVER_SKL | AZX_DCAPS_INTEL_SKYLAKE},
diff --git a/sound/pci/hda/patch_hdmi.c b/sound/pci/hda/patch_hdmi.c
index 65d2c5539919..98633d2684de 100644
--- a/sound/pci/hda/patch_hdmi.c
+++ b/sound/pci/hda/patch_hdmi.c
@@ -4382,6 +4382,7 @@  HDA_CODEC_ENTRY(0x80862814, "DG1 HDMI",	patch_i915_tgl_hdmi),
 HDA_CODEC_ENTRY(0x80862815, "Alderlake HDMI",	patch_i915_tgl_hdmi),
 HDA_CODEC_ENTRY(0x8086281c, "Alderlake-P HDMI", patch_i915_tgl_hdmi),
 HDA_CODEC_ENTRY(0x80862816, "Rocketlake HDMI",	patch_i915_tgl_hdmi),
+HDA_CODEC_ENTRY(0x80862819, "DG2 HDMI",	patch_i915_tgl_hdmi),
 HDA_CODEC_ENTRY(0x8086281a, "Jasperlake HDMI",	patch_i915_icl_hdmi),
 HDA_CODEC_ENTRY(0x8086281b, "Elkhartlake HDMI",	patch_i915_icl_hdmi),
 HDA_CODEC_ENTRY(0x80862880, "CedarTrail HDMI",	patch_generic_hdmi),