summaryrefslogtreecommitdiff
path: root/src/southbridge
diff options
context:
space:
mode:
authorFelix Singer <felixsinger@posteo.net>2019-11-21 21:23:32 +0100
committerFelix Held <felix-coreboot@felixheld.de>2020-03-20 23:14:52 +0000
commit838fbc71cf38b441a8a28ba04916c23a0b9dc80c (patch)
tree550794958ce463bcbadd17dc46517259044976d3 /src/southbridge
parent8ca1ada083e9029defb2d40f33702bd2553d113f (diff)
downloadcoreboot-838fbc71cf38b441a8a28ba04916c23a0b9dc80c.tar.xz
sb/ibexpeak: Use macros instead of hard-coded IDs
This patch replaces hard-coded PCI IDs with macros from pci_ids.h and adds the related IDs to it. The resulting binary doesn't differ from the one without this patch. Used documents: - Intel 322170 Change-Id: I3326f142d483f5008fb2ac878f30c1a3a72f500f Signed-off-by: Felix Singer <felixsinger@posteo.net> Reviewed-on: https://review.coreboot.org/c/coreboot/+/37116 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net> Reviewed-by: Michael Niewöhner
Diffstat (limited to 'src/southbridge')
-rw-r--r--src/southbridge/intel/ibexpeak/azalia.c7
-rw-r--r--src/southbridge/intel/ibexpeak/lpc.c6
-rw-r--r--src/southbridge/intel/ibexpeak/me.c7
-rw-r--r--src/southbridge/intel/ibexpeak/sata.c7
-rw-r--r--src/southbridge/intel/ibexpeak/smbus.c7
-rw-r--r--src/southbridge/intel/ibexpeak/thermal.c5
-rw-r--r--src/southbridge/intel/ibexpeak/usb_ehci.c6
7 files changed, 37 insertions, 8 deletions
diff --git a/src/southbridge/intel/ibexpeak/azalia.c b/src/southbridge/intel/ibexpeak/azalia.c
index 2fceced9b4..fe5cc2ea1d 100644
--- a/src/southbridge/intel/ibexpeak/azalia.c
+++ b/src/southbridge/intel/ibexpeak/azalia.c
@@ -323,7 +323,12 @@ static struct device_operations azalia_ops = {
.ops_pci = &azalia_pci_ops,
};
-static const unsigned short pci_device_ids[] = { 0x1c20, 0x1e20, 0x3b56, 0 };
+static const unsigned short pci_device_ids[] = {
+ 0x1c20,
+ 0x1e20,
+ PCI_DID_INTEL_IBEXPEAK_AUDIO,
+ 0
+};
static const struct pci_driver pch_azalia __pci_driver = {
.ops = &azalia_ops,
diff --git a/src/southbridge/intel/ibexpeak/lpc.c b/src/southbridge/intel/ibexpeak/lpc.c
index 5c1b77fe43..0062e0901c 100644
--- a/src/southbridge/intel/ibexpeak/lpc.c
+++ b/src/southbridge/intel/ibexpeak/lpc.c
@@ -777,7 +777,11 @@ static struct device_operations device_ops = {
};
-static const unsigned short pci_device_ids[] = { 0x3b07, 0x3b09, 0 };
+static const unsigned short pci_device_ids[] = {
+ PCI_DID_INTEL_IBEXPEAK_LPC_QM57,
+ PCI_DID_INTEL_IBEXPEAK_LPC_HM55,
+ 0
+};
static const struct pci_driver pch_lpc __pci_driver = {
.ops = &device_ops,
diff --git a/src/southbridge/intel/ibexpeak/me.c b/src/southbridge/intel/ibexpeak/me.c
index 31a0261da6..e8974d8491 100644
--- a/src/southbridge/intel/ibexpeak/me.c
+++ b/src/southbridge/intel/ibexpeak/me.c
@@ -614,8 +614,11 @@ static struct device_operations device_ops = {
.ops_pci = &pci_ops,
};
-static const unsigned short pci_device_ids[] = { 0x1c3a, 0x3b64,
- 0 };
+static const unsigned short pci_device_ids[] = {
+ 0x1c3a,
+ PCI_DID_INTEL_IBEXPEAK_HECI1,
+ 0
+};
static const struct pci_driver intel_me __pci_driver = {
diff --git a/src/southbridge/intel/ibexpeak/sata.c b/src/southbridge/intel/ibexpeak/sata.c
index e4eebcb8a9..33437ef7d6 100644
--- a/src/southbridge/intel/ibexpeak/sata.c
+++ b/src/southbridge/intel/ibexpeak/sata.c
@@ -245,7 +245,12 @@ static struct device_operations sata_ops = {
.ops_pci = &sata_pci_ops,
};
-static const unsigned short pci_device_ids[] = { 0x3b28, 0x3b29, 0x3b2e, 0 };
+static const unsigned short pci_device_ids[] = {
+ PCI_DID_INTEL_IBEXPEAK_MOBILE_SATA_IDE_1,
+ PCI_DID_INTEL_IBEXPEAK_MOBILE_SATA_AHCI,
+ PCI_DID_INTEL_IBEXPEAK_MOBILE_SATA_IDE_2,
+ 0
+};
static const struct pci_driver pch_sata __pci_driver = {
.ops = &sata_ops,
diff --git a/src/southbridge/intel/ibexpeak/smbus.c b/src/southbridge/intel/ibexpeak/smbus.c
index eafb1ee2df..b06b1cd135 100644
--- a/src/southbridge/intel/ibexpeak/smbus.c
+++ b/src/southbridge/intel/ibexpeak/smbus.c
@@ -96,7 +96,12 @@ static struct device_operations smbus_ops = {
.ops_pci = &smbus_pci_ops,
};
-static const unsigned short pci_device_ids[] = { 0x1c22, 0x1e22, 0x3b30, 0 };
+static const unsigned short pci_device_ids[] = {
+ 0x1c22,
+ 0x1e22,
+ PCI_DID_INTEL_IBEXPEAK_SMBUS,
+ 0
+};
static const struct pci_driver pch_smbus __pci_driver = {
.ops = &smbus_ops,
diff --git a/src/southbridge/intel/ibexpeak/thermal.c b/src/southbridge/intel/ibexpeak/thermal.c
index e9a542e692..2664c65ea7 100644
--- a/src/southbridge/intel/ibexpeak/thermal.c
+++ b/src/southbridge/intel/ibexpeak/thermal.c
@@ -56,7 +56,10 @@ static struct device_operations thermal_ops = {
.ops_pci = &pci_ops,
};
-static const unsigned short pci_device_ids[] = { 0x3b32, 0 };
+static const unsigned short pci_device_ids[] = {
+ PCI_DID_INTEL_IBEXPEAK_THERMAL,
+ 0
+};
static const struct pci_driver pch_thermal __pci_driver = {
.ops = &thermal_ops,
diff --git a/src/southbridge/intel/ibexpeak/usb_ehci.c b/src/southbridge/intel/ibexpeak/usb_ehci.c
index ce169945fc..f4b975a4a0 100644
--- a/src/southbridge/intel/ibexpeak/usb_ehci.c
+++ b/src/southbridge/intel/ibexpeak/usb_ehci.c
@@ -96,7 +96,11 @@ static struct device_operations usb_ehci_ops = {
.ops_pci = &lops_pci,
};
-static const unsigned short pci_device_ids[] = { 0x3b34, 0x3b3c, 0 };
+static const unsigned short pci_device_ids[] = {
+ PCI_DID_INTEL_IBEXPEAK_EHCI_1,
+ PCI_DID_INTEL_IBEXPEAK_EHCI_2,
+ 0
+};
static const struct pci_driver pch_usb_ehci __pci_driver = {
.ops = &usb_ehci_ops,