summaryrefslogtreecommitdiff
path: root/src/southbridge/intel
diff options
context:
space:
mode:
authorFelix Singer <felixsinger@posteo.net>2020-11-20 12:56:44 +0000
committerMichael Niewöhner <foss@mniewoehner.de>2020-11-24 18:37:58 +0000
commit4ea08f9f5668cec116478a2fc226945c0a66259a (patch)
tree7dc777ed8e00fbf3e1c641f4e39865485135b74e /src/southbridge/intel
parentd5f1c088164d03a501fb94217bca94ffd382079e (diff)
downloadcoreboot-4ea08f9f5668cec116478a2fc226945c0a66259a.tar.xz
sb/intel/lynxpoint: Replace hard-coded IDs with defines
Replace hard-coded IDs with defines introduced in CB:47807. Used documents: - 328904-003 - 329003-003 Built lenovo/t440p with BUILD_TIMELESS=1, coreboot.rom remains identical. Change-Id: I910ab356dd8728c316018989bfb2689d4c67c2dc Signed-off-by: Felix Singer <felixsinger@posteo.net> Reviewed-on: https://review.coreboot.org/c/coreboot/+/47808 Reviewed-by: Michael Niewöhner <foss@mniewoehner.de> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/southbridge/intel')
-rw-r--r--src/southbridge/intel/lynxpoint/azalia.c6
-rw-r--r--src/southbridge/intel/lynxpoint/lpc.c41
-rw-r--r--src/southbridge/intel/lynxpoint/me_9.x.c4
-rw-r--r--src/southbridge/intel/lynxpoint/pcie.c18
-rw-r--r--src/southbridge/intel/lynxpoint/sata.c19
-rw-r--r--src/southbridge/intel/lynxpoint/serialio.c16
-rw-r--r--src/southbridge/intel/lynxpoint/smbus.c3
-rw-r--r--src/southbridge/intel/lynxpoint/usb_ehci.c7
-rw-r--r--src/southbridge/intel/lynxpoint/usb_xhci.c8
9 files changed, 79 insertions, 43 deletions
diff --git a/src/southbridge/intel/lynxpoint/azalia.c b/src/southbridge/intel/lynxpoint/azalia.c
index 68958f08d0..ef450a501f 100644
--- a/src/southbridge/intel/lynxpoint/azalia.c
+++ b/src/southbridge/intel/lynxpoint/azalia.c
@@ -132,7 +132,11 @@ static struct device_operations azalia_ops = {
.ops_pci = &pci_dev_ops_pci,
};
-static const unsigned short pci_device_ids[] = { 0x8c20, 0x9c20, 0 };
+static const unsigned short pci_device_ids[] = {
+ PCI_DEVICE_ID_INTEL_LPT_H_AUDIO,
+ PCI_DEVICE_ID_INTEL_LPT_LP_AUDIO,
+ 0
+};
static const struct pci_driver pch_azalia __pci_driver = {
.ops = &azalia_ops,
diff --git a/src/southbridge/intel/lynxpoint/lpc.c b/src/southbridge/intel/lynxpoint/lpc.c
index 232ad2ec95..915c181c2f 100644
--- a/src/southbridge/intel/lynxpoint/lpc.c
+++ b/src/southbridge/intel/lynxpoint/lpc.c
@@ -787,26 +787,27 @@ static struct device_operations device_ops = {
/* IDs for LPC device of Intel 8 Series Chipset (Lynx Point) */
static const unsigned short pci_device_ids[] = {
- 0x8c41, /* Mobile Full Featured Engineering Sample. */
- 0x8c42, /* Desktop Full Featured Engineering Sample. */
- 0x8c44, /* Z87 SKU */
- 0x8c46, /* Z85 SKU */
- 0x8c49, /* HM86 SKU */
- 0x8c4a, /* H87 SKU */
- 0x8c4b, /* HM87 SKU */
- 0x8c4c, /* Q85 SKU */
- 0x8c4e, /* Q87 SKU */
- 0x8c4f, /* QM87 SKU */
- 0x8c50, /* B85 SKU */
- 0x8c52, /* C222 SKU */
- 0x8c54, /* C224 SKU */
- 0x8c56, /* C226 SKU */
- 0x8c5c, /* H81 SKU */
- 0x9c41, /* LP Full Featured Engineering Sample */
- 0x9c43, /* LP Premium SKU */
- 0x9c45, /* LP Mainstream SKU */
- 0x9c47, /* LP Value SKU */
- 0 };
+ PCI_DEVICE_ID_INTEL_LPT_MOBILE_SAMPLE,
+ PCI_DEVICE_ID_INTEL_LPT_DESKTOP_SAMPLE,
+ PCI_DEVICE_ID_INTEL_LPT_Z87,
+ PCI_DEVICE_ID_INTEL_LPT_Z85,
+ PCI_DEVICE_ID_INTEL_LPT_HM86,
+ PCI_DEVICE_ID_INTEL_LPT_H87,
+ PCI_DEVICE_ID_INTEL_LPT_HM87,
+ PCI_DEVICE_ID_INTEL_LPT_Q85,
+ PCI_DEVICE_ID_INTEL_LPT_Q87,
+ PCI_DEVICE_ID_INTEL_LPT_QM87,
+ PCI_DEVICE_ID_INTEL_LPT_B85,
+ PCI_DEVICE_ID_INTEL_LPT_C222,
+ PCI_DEVICE_ID_INTEL_LPT_C224,
+ PCI_DEVICE_ID_INTEL_LPT_C226,
+ PCI_DEVICE_ID_INTEL_LPT_H81,
+ PCI_DEVICE_ID_INTEL_LPT_LP_SAMPLE,
+ PCI_DEVICE_ID_INTEL_LPT_LP_PREMIUM,
+ PCI_DEVICE_ID_INTEL_LPT_LP_MAINSTREAM,
+ PCI_DEVICE_ID_INTEL_LPT_LP_VALUE,
+ 0
+};
static const struct pci_driver pch_lpc __pci_driver = {
.ops = &device_ops,
diff --git a/src/southbridge/intel/lynxpoint/me_9.x.c b/src/southbridge/intel/lynxpoint/me_9.x.c
index f32ee5bf67..9910658b82 100644
--- a/src/southbridge/intel/lynxpoint/me_9.x.c
+++ b/src/southbridge/intel/lynxpoint/me_9.x.c
@@ -836,8 +836,8 @@ static struct device_operations device_ops = {
};
static const unsigned short pci_device_ids[] = {
- 0x8c3a, /* Mobile */
- 0x9c3a, /* Low Power */
+ PCI_DEVICE_ID_INTEL_LPT_H_MEI,
+ PCI_DEVICE_ID_INTEL_LPT_LP_MEI,
0
};
diff --git a/src/southbridge/intel/lynxpoint/pcie.c b/src/southbridge/intel/lynxpoint/pcie.c
index 19eb9fa396..5f21e619a5 100644
--- a/src/southbridge/intel/lynxpoint/pcie.c
+++ b/src/southbridge/intel/lynxpoint/pcie.c
@@ -749,10 +749,20 @@ static struct device_operations device_ops = {
};
static const unsigned short pci_device_ids[] = {
- /* Lynxpoint Mobile */
- 0x8c10, 0x8c12, 0x8c14, 0x8c16, 0x8c18, 0x8c1a, 0x8c1c, 0x8c1e,
- /* Lynxpoint Low Power */
- 0x9c10, 0x9c12, 0x9c14, 0x9c16, 0x9c18, 0x9c1a,
+ PCI_DEVICE_ID_INTEL_LPT_H_PCIE_RP1,
+ PCI_DEVICE_ID_INTEL_LPT_H_PCIE_RP2,
+ PCI_DEVICE_ID_INTEL_LPT_H_PCIE_RP3,
+ PCI_DEVICE_ID_INTEL_LPT_H_PCIE_RP4,
+ PCI_DEVICE_ID_INTEL_LPT_H_PCIE_RP5,
+ PCI_DEVICE_ID_INTEL_LPT_H_PCIE_RP6,
+ PCI_DEVICE_ID_INTEL_LPT_H_PCIE_RP7,
+ PCI_DEVICE_ID_INTEL_LPT_H_PCIE_RP8,
+ PCI_DEVICE_ID_INTEL_LPT_LP_PCIE_RP1,
+ PCI_DEVICE_ID_INTEL_LPT_LP_PCIE_RP2,
+ PCI_DEVICE_ID_INTEL_LPT_LP_PCIE_RP3,
+ PCI_DEVICE_ID_INTEL_LPT_LP_PCIE_RP4,
+ PCI_DEVICE_ID_INTEL_LPT_LP_PCIE_RP5,
+ PCI_DEVICE_ID_INTEL_LPT_LP_PCIE_RP6,
0
};
diff --git a/src/southbridge/intel/lynxpoint/sata.c b/src/southbridge/intel/lynxpoint/sata.c
index 6899e81351..a0a606dfbb 100644
--- a/src/southbridge/intel/lynxpoint/sata.c
+++ b/src/southbridge/intel/lynxpoint/sata.c
@@ -214,9 +214,22 @@ static struct device_operations sata_ops = {
};
static const unsigned short pci_device_ids[] = {
- 0x8c00, 0x8c02, 0x8c04, 0x8c06, 0x8c08, 0x8c0e, /* Desktop */
- 0x8c01, 0x8c03, 0x8c05, 0x8c07, 0x8c09, 0x8c0f, /* Mobile */
- 0x9c03, 0x9c05, 0x9c07, 0x9c0f, /* Low Power */
+ PCI_DEVICE_ID_INTEL_LPT_H_DESKTOP_SATA_IDE,
+ PCI_DEVICE_ID_INTEL_LPT_H_DESKTOP_SATA_AHCI,
+ PCI_DEVICE_ID_INTEL_LPT_H_DESKTOP_SATA_RAID_1,
+ PCI_DEVICE_ID_INTEL_LPT_H_DESKTOP_SATA_RAID_PREM,
+ PCI_DEVICE_ID_INTEL_LPT_H_DESKTOP_SATA_IDE_P45,
+ PCI_DEVICE_ID_INTEL_LPT_H_DESKTOP_SATA_RAID_2,
+ PCI_DEVICE_ID_INTEL_LPT_H_MOBILE_SATA_IDE,
+ PCI_DEVICE_ID_INTEL_LPT_H_MOBILE_SATA_AHCI,
+ PCI_DEVICE_ID_INTEL_LPT_H_MOBILE_SATA_RAID_1,
+ PCI_DEVICE_ID_INTEL_LPT_H_MOBILE_SATA_RAID_PREM,
+ PCI_DEVICE_ID_INTEL_LPT_H_MOBILE_SATA_IDE_P45,
+ PCI_DEVICE_ID_INTEL_LPT_H_MOBILE_SATA_RAID_2,
+ PCI_DEVICE_ID_INTEL_LPT_LP_SATA_AHCI,
+ PCI_DEVICE_ID_INTEL_LPT_LP_SATA_RAID_1,
+ PCI_DEVICE_ID_INTEL_LPT_LP_SATA_RAID_PREM,
+ PCI_DEVICE_ID_INTEL_LPT_LP_SATA_RAID_2,
0
};
diff --git a/src/southbridge/intel/lynxpoint/serialio.c b/src/southbridge/intel/lynxpoint/serialio.c
index 199cf9f896..d212d88621 100644
--- a/src/southbridge/intel/lynxpoint/serialio.c
+++ b/src/southbridge/intel/lynxpoint/serialio.c
@@ -228,14 +228,14 @@ static struct device_operations device_ops = {
};
static const unsigned short pci_device_ids[] = {
- 0x9c60, /* 0:15.0 - SDMA */
- 0x9c61, /* 0:15.1 - I2C0 */
- 0x9c62, /* 0:15.2 - I2C1 */
- 0x9c65, /* 0:15.3 - SPI0 */
- 0x9c66, /* 0:15.4 - SPI1 */
- 0x9c63, /* 0:15.5 - UART0 */
- 0x9c64, /* 0:15.6 - UART1 */
- 0x9c35, /* 0:17.0 - SDIO */
+ PCI_DEVICE_ID_INTEL_LPT_LP_SDMA,
+ PCI_DEVICE_ID_INTEL_LPT_LP_I2C0,
+ PCI_DEVICE_ID_INTEL_LPT_LP_I2C1,
+ PCI_DEVICE_ID_INTEL_LPT_LP_GSPI0,
+ PCI_DEVICE_ID_INTEL_LPT_LP_GSPI1,
+ PCI_DEVICE_ID_INTEL_LPT_LP_UART0,
+ PCI_DEVICE_ID_INTEL_LPT_LP_UART1,
+ PCI_DEVICE_ID_INTEL_LPT_LP_SD,
0
};
diff --git a/src/southbridge/intel/lynxpoint/smbus.c b/src/southbridge/intel/lynxpoint/smbus.c
index 2741951a4e..d0a621eea1 100644
--- a/src/southbridge/intel/lynxpoint/smbus.c
+++ b/src/southbridge/intel/lynxpoint/smbus.c
@@ -80,7 +80,8 @@ static struct device_operations smbus_ops = {
};
static const unsigned short pci_device_ids[] = {
- 0x8c22, 0x9c22,
+ PCI_DEVICE_ID_INTEL_LPT_H_SMBUS,
+ PCI_DEVICE_ID_INTEL_LPT_LP_SMBUS,
0
};
diff --git a/src/southbridge/intel/lynxpoint/usb_ehci.c b/src/southbridge/intel/lynxpoint/usb_ehci.c
index f121aa84ce..d718ab3280 100644
--- a/src/southbridge/intel/lynxpoint/usb_ehci.c
+++ b/src/southbridge/intel/lynxpoint/usb_ehci.c
@@ -169,7 +169,12 @@ static struct device_operations usb_ehci_ops = {
.ops_pci = &lops_pci,
};
-static const unsigned short pci_device_ids[] = { 0x9c26, 0x8c26, 0x8c2d, 0 };
+static const unsigned short pci_device_ids[] = {
+ PCI_DEVICE_ID_INTEL_LPT_LP_EHCI,
+ PCI_DEVICE_ID_INTEL_LPT_H_EHCI_1,
+ PCI_DEVICE_ID_INTEL_LPT_H_EHCI_2,
+ 0
+};
static const struct pci_driver pch_usb_ehci __pci_driver = {
.ops = &usb_ehci_ops,
diff --git a/src/southbridge/intel/lynxpoint/usb_xhci.c b/src/southbridge/intel/lynxpoint/usb_xhci.c
index 6a58d41a18..d4611a77db 100644
--- a/src/southbridge/intel/lynxpoint/usb_xhci.c
+++ b/src/southbridge/intel/lynxpoint/usb_xhci.c
@@ -342,9 +342,11 @@ static struct device_operations usb_xhci_ops = {
.ops_pci = &pci_dev_ops_pci,
};
-static const unsigned short pci_device_ids[] = { 0x8c31, /* LynxPoint-H */
- 0x9c31, /* LynxPoint-LP */
- 0 };
+static const unsigned short pci_device_ids[] = {
+ PCI_DEVICE_ID_INTEL_LPT_H_XHCI,
+ PCI_DEVICE_ID_INTEL_LPT_LP_XHCI,
+ 0
+};
static const struct pci_driver pch_usb_xhci __pci_driver = {
.ops = &usb_xhci_ops,