From 0e5dde5d996cb81cc91877fad519c22622c17a97 Mon Sep 17 00:00:00 2001 From: Felix Held Date: Tue, 17 Nov 2020 14:57:40 +0100 Subject: [PATCH] include/device/pci_ids: add model number to data fabric devices Different models within family 17h have different PCI IDs for their data fabric PCI devices. Change-Id: I44f8d32c950710e962dc519495b08c92f357ed20 Signed-off-by: Felix Held Reviewed-on: https://review.coreboot.org/c/coreboot/+/47676 Tested-by: build bot (Jenkins) Reviewed-by: Marshall Dawson --- src/include/device/pci_ids.h | 14 +++++++------- src/soc/amd/picasso/data_fabric.c | 28 ++++++++++++++-------------- 2 files changed, 21 insertions(+), 21 deletions(-) diff --git a/src/include/device/pci_ids.h b/src/include/device/pci_ids.h index cb2d2ddb24..7dacb65a32 100644 --- a/src/include/device/pci_ids.h +++ b/src/include/device/pci_ids.h @@ -461,13 +461,13 @@ #define PCI_DEVICE_ID_AMD_FAM17H_MODEL18H_XHCI0 0x15E0 #define PCI_DEVICE_ID_AMD_FAM17H_MODEL18H_XHCI1 0x15E1 #define PCI_DEVICE_ID_AMD_FAM17H_MODEL20H_XHCI0 0x15E5 -#define PCI_DEVICE_ID_AMD_FAM17H_DF0 0x15E8 -#define PCI_DEVICE_ID_AMD_FAM17H_DF1 0x15E9 -#define PCI_DEVICE_ID_AMD_FAM17H_DF2 0x15EA -#define PCI_DEVICE_ID_AMD_FAM17H_DF3 0x15EB -#define PCI_DEVICE_ID_AMD_FAM17H_DF4 0x15EC -#define PCI_DEVICE_ID_AMD_FAM17H_DF5 0x15ED -#define PCI_DEVICE_ID_AMD_FAM17H_DF6 0x15EE +#define PCI_DEVICE_ID_AMD_FAM17H_MODEL18H_DF0 0x15E8 +#define PCI_DEVICE_ID_AMD_FAM17H_MODEL18H_DF1 0x15E9 +#define PCI_DEVICE_ID_AMD_FAM17H_MODEL18H_DF2 0x15EA +#define PCI_DEVICE_ID_AMD_FAM17H_MODEL18H_DF3 0x15EB +#define PCI_DEVICE_ID_AMD_FAM17H_MODEL18H_DF4 0x15EC +#define PCI_DEVICE_ID_AMD_FAM17H_MODEL18H_DF5 0x15ED +#define PCI_DEVICE_ID_AMD_FAM17H_MODEL18H_DF6 0x15EE #define PCI_DEVICE_ID_AMD_FAM17H_SATA_AHCI_VER0 0x7901 #define PCI_DEVICE_ID_AMD_FAM17H_SATA_AHCI_VER1 0x7904 #define PCI_DEVICE_ID_AMD_FAM17H_SATA_AHCI_RAID_VER0 0x7916 diff --git a/src/soc/amd/picasso/data_fabric.c b/src/soc/amd/picasso/data_fabric.c index ac4e23d318..79fdba1267 100644 --- a/src/soc/amd/picasso/data_fabric.c +++ b/src/soc/amd/picasso/data_fabric.c @@ -123,19 +123,19 @@ void data_fabric_set_mmio_np(void) static const char *data_fabric_acpi_name(const struct device *dev) { switch (dev->device) { - case PCI_DEVICE_ID_AMD_FAM17H_DF0: + case PCI_DEVICE_ID_AMD_FAM17H_MODEL18H_DF0: return "DFD0"; - case PCI_DEVICE_ID_AMD_FAM17H_DF1: + case PCI_DEVICE_ID_AMD_FAM17H_MODEL18H_DF1: return "DFD1"; - case PCI_DEVICE_ID_AMD_FAM17H_DF2: + case PCI_DEVICE_ID_AMD_FAM17H_MODEL18H_DF2: return "DFD2"; - case PCI_DEVICE_ID_AMD_FAM17H_DF3: + case PCI_DEVICE_ID_AMD_FAM17H_MODEL18H_DF3: return "DFD3"; - case PCI_DEVICE_ID_AMD_FAM17H_DF4: + case PCI_DEVICE_ID_AMD_FAM17H_MODEL18H_DF4: return "DFD4"; - case PCI_DEVICE_ID_AMD_FAM17H_DF5: + case PCI_DEVICE_ID_AMD_FAM17H_MODEL18H_DF5: return "DFD5"; - case PCI_DEVICE_ID_AMD_FAM17H_DF6: + case PCI_DEVICE_ID_AMD_FAM17H_MODEL18H_DF6: return "DFD6"; default: printk(BIOS_ERR, "%s: Unhandled device id 0x%x\n", __func__, dev->device); @@ -152,13 +152,13 @@ static struct device_operations data_fabric_ops = { }; static const unsigned short pci_device_ids[] = { - PCI_DEVICE_ID_AMD_FAM17H_DF0, - PCI_DEVICE_ID_AMD_FAM17H_DF1, - PCI_DEVICE_ID_AMD_FAM17H_DF2, - PCI_DEVICE_ID_AMD_FAM17H_DF3, - PCI_DEVICE_ID_AMD_FAM17H_DF4, - PCI_DEVICE_ID_AMD_FAM17H_DF5, - PCI_DEVICE_ID_AMD_FAM17H_DF6, + PCI_DEVICE_ID_AMD_FAM17H_MODEL18H_DF0, + PCI_DEVICE_ID_AMD_FAM17H_MODEL18H_DF1, + PCI_DEVICE_ID_AMD_FAM17H_MODEL18H_DF2, + PCI_DEVICE_ID_AMD_FAM17H_MODEL18H_DF3, + PCI_DEVICE_ID_AMD_FAM17H_MODEL18H_DF4, + PCI_DEVICE_ID_AMD_FAM17H_MODEL18H_DF5, + PCI_DEVICE_ID_AMD_FAM17H_MODEL18H_DF6, 0 };