Subrata Banik has submitted this change. ( https://review.coreboot.org/c/coreboot/+/83516?usp=email )
Change subject: device/pci_ids: Add new Intel PTL device IDs for Audio ......................................................................
device/pci_ids: Add new Intel PTL device IDs for Audio
This patch adds new Audio (HDA/DSP) PCI device IDs for Intel PTL-U and PTL-H.
Additionally, updates the Audio driver's `pci_device_ids` list to include these new IDs.
Source: Intel PTL-EDS vol 1. Document Number 815002, Rev 0.51 Chapter 2
BUG=b:347669091 TEST=Able to build google/fatcat.
Change-Id: I3c9e420a6ae19d00fb5510c99d4c219dc43ad3c0 Signed-off-by: Subrata Banik subratabanik@google.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/83516 Reviewed-by: Angel Pons th3fanbus@gmail.com Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Eric Lai ericllai@google.com --- M src/include/device/pci_ids.h M src/soc/intel/common/block/dsp/dsp.c M src/soc/intel/common/block/hda/hda.c 3 files changed, 48 insertions(+), 24 deletions(-)
Approvals: Angel Pons: Looks good to me, but someone else must approve Eric Lai: Looks good to me, approved build bot (Jenkins): Verified
diff --git a/src/include/device/pci_ids.h b/src/include/device/pci_ids.h index dac7508..716f386 100644 --- a/src/include/device/pci_ids.h +++ b/src/include/device/pci_ids.h @@ -4617,14 +4617,22 @@ #define PCI_DID_INTEL_LNL_AUDIO_7 0xa82e #define PCI_DID_INTEL_LNL_AUDIO_8 0xa82f
-#define PCI_DID_INTEL_PTL_AUDIO_1 0xe428 -#define PCI_DID_INTEL_PTL_AUDIO_2 0xe429 -#define PCI_DID_INTEL_PTL_AUDIO_3 0xe42a -#define PCI_DID_INTEL_PTL_AUDIO_4 0xe42b -#define PCI_DID_INTEL_PTL_AUDIO_5 0xe42c -#define PCI_DID_INTEL_PTL_AUDIO_6 0xe42d -#define PCI_DID_INTEL_PTL_AUDIO_7 0xe42e -#define PCI_DID_INTEL_PTL_AUDIO_8 0xe42f +#define PCI_DID_INTEL_PTL_H_AUDIO_1 0xe428 +#define PCI_DID_INTEL_PTL_H_AUDIO_2 0xe429 +#define PCI_DID_INTEL_PTL_H_AUDIO_3 0xe42a +#define PCI_DID_INTEL_PTL_H_AUDIO_4 0xe42b +#define PCI_DID_INTEL_PTL_H_AUDIO_5 0xe42c +#define PCI_DID_INTEL_PTL_H_AUDIO_6 0xe42d +#define PCI_DID_INTEL_PTL_H_AUDIO_7 0xe42e +#define PCI_DID_INTEL_PTL_H_AUDIO_8 0xe42f +#define PCI_DID_INTEL_PTL_U_H_AUDIO_1 0xe328 +#define PCI_DID_INTEL_PTL_U_H_AUDIO_2 0xe329 +#define PCI_DID_INTEL_PTL_U_H_AUDIO_3 0xe32a +#define PCI_DID_INTEL_PTL_U_H_AUDIO_4 0xe32b +#define PCI_DID_INTEL_PTL_U_H_AUDIO_5 0xe32c +#define PCI_DID_INTEL_PTL_U_H_AUDIO_6 0xe32d +#define PCI_DID_INTEL_PTL_U_H_AUDIO_7 0xe32e +#define PCI_DID_INTEL_PTL_U_H_AUDIO_8 0xe32f
/* Intel HECI/ME device Ids */ #define PCI_DID_INTEL_LPT_H_MEI 0x8c3a diff --git a/src/soc/intel/common/block/dsp/dsp.c b/src/soc/intel/common/block/dsp/dsp.c index 7e8322e..ab0e116 100644 --- a/src/soc/intel/common/block/dsp/dsp.c +++ b/src/soc/intel/common/block/dsp/dsp.c @@ -13,14 +13,22 @@ };
static const unsigned short pci_device_ids[] = { - PCI_DID_INTEL_PTL_AUDIO_1, - PCI_DID_INTEL_PTL_AUDIO_2, - PCI_DID_INTEL_PTL_AUDIO_3, - PCI_DID_INTEL_PTL_AUDIO_4, - PCI_DID_INTEL_PTL_AUDIO_5, - PCI_DID_INTEL_PTL_AUDIO_6, - PCI_DID_INTEL_PTL_AUDIO_7, - PCI_DID_INTEL_PTL_AUDIO_8, + PCI_DID_INTEL_PTL_H_AUDIO_1, + PCI_DID_INTEL_PTL_H_AUDIO_2, + PCI_DID_INTEL_PTL_H_AUDIO_3, + PCI_DID_INTEL_PTL_H_AUDIO_4, + PCI_DID_INTEL_PTL_H_AUDIO_5, + PCI_DID_INTEL_PTL_H_AUDIO_6, + PCI_DID_INTEL_PTL_H_AUDIO_7, + PCI_DID_INTEL_PTL_H_AUDIO_8, + PCI_DID_INTEL_PTL_U_H_AUDIO_1, + PCI_DID_INTEL_PTL_U_H_AUDIO_2, + PCI_DID_INTEL_PTL_U_H_AUDIO_3, + PCI_DID_INTEL_PTL_U_H_AUDIO_4, + PCI_DID_INTEL_PTL_U_H_AUDIO_5, + PCI_DID_INTEL_PTL_U_H_AUDIO_6, + PCI_DID_INTEL_PTL_U_H_AUDIO_7, + PCI_DID_INTEL_PTL_U_H_AUDIO_8, PCI_DID_INTEL_LNL_AUDIO_1, PCI_DID_INTEL_LNL_AUDIO_2, PCI_DID_INTEL_LNL_AUDIO_3, diff --git a/src/soc/intel/common/block/hda/hda.c b/src/soc/intel/common/block/hda/hda.c index ca50312..59ecb50 100644 --- a/src/soc/intel/common/block/hda/hda.c +++ b/src/soc/intel/common/block/hda/hda.c @@ -21,14 +21,22 @@ };
static const unsigned short pci_device_ids[] = { - PCI_DID_INTEL_PTL_AUDIO_1, - PCI_DID_INTEL_PTL_AUDIO_2, - PCI_DID_INTEL_PTL_AUDIO_3, - PCI_DID_INTEL_PTL_AUDIO_4, - PCI_DID_INTEL_PTL_AUDIO_5, - PCI_DID_INTEL_PTL_AUDIO_6, - PCI_DID_INTEL_PTL_AUDIO_7, - PCI_DID_INTEL_PTL_AUDIO_8, + PCI_DID_INTEL_PTL_H_AUDIO_1, + PCI_DID_INTEL_PTL_H_AUDIO_2, + PCI_DID_INTEL_PTL_H_AUDIO_3, + PCI_DID_INTEL_PTL_H_AUDIO_4, + PCI_DID_INTEL_PTL_H_AUDIO_5, + PCI_DID_INTEL_PTL_H_AUDIO_6, + PCI_DID_INTEL_PTL_H_AUDIO_7, + PCI_DID_INTEL_PTL_H_AUDIO_8, + PCI_DID_INTEL_PTL_U_H_AUDIO_1, + PCI_DID_INTEL_PTL_U_H_AUDIO_2, + PCI_DID_INTEL_PTL_U_H_AUDIO_3, + PCI_DID_INTEL_PTL_U_H_AUDIO_4, + PCI_DID_INTEL_PTL_U_H_AUDIO_5, + PCI_DID_INTEL_PTL_U_H_AUDIO_6, + PCI_DID_INTEL_PTL_U_H_AUDIO_7, + PCI_DID_INTEL_PTL_U_H_AUDIO_8, PCI_DID_INTEL_LNL_AUDIO_1, PCI_DID_INTEL_LNL_AUDIO_2, PCI_DID_INTEL_LNL_AUDIO_3,
-
Subrata Banik (Code Review)