soc/intel/{baytrail,braswell}: Make use of generic set_subsystem()
We missed some PCIe root ports with previous cleanup.
Change-Id: I8bf8f8b2ca1836316f84fb7f01820a00d7194d51
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/31991
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
diff --git a/src/soc/intel/baytrail/pcie.c b/src/soc/intel/baytrail/pcie.c
index 2740f10..33c5455 100644
--- a/src/soc/intel/baytrail/pcie.c
+++ b/src/soc/intel/baytrail/pcie.c
@@ -240,18 +240,8 @@
do_pci_scan_bridge(dev, pciexp_scan_bus);
}
-static void pcie_root_set_subsystem(struct device *dev, unsigned vid,
- unsigned did)
-{
- uint32_t didvid = ((did & 0xffff) << 16) | (vid & 0xffff);
-
- if (!didvid)
- didvid = pci_read_config32(dev, PCI_VENDOR_ID);
- pci_write_config32(dev, 0x94, didvid);
-}
-
static struct pci_operations pcie_root_ops = {
- .set_subsystem = &pcie_root_set_subsystem,
+ .set_subsystem = pci_dev_set_subsystem,
};
static struct device_operations device_ops = {
diff --git a/src/soc/intel/braswell/pcie.c b/src/soc/intel/braswell/pcie.c
index 8494974..6e387d1 100644
--- a/src/soc/intel/braswell/pcie.c
+++ b/src/soc/intel/braswell/pcie.c
@@ -160,20 +160,8 @@
southcluster_enable_dev(dev);
}
-static void pcie_root_set_subsystem(struct device *dev, unsigned int vid,
- unsigned int did)
-{
- printk(BIOS_SPEW, "%s/%s (%s, 0x%04x, 0x%04x)\n",
- __FILE__, __func__, dev_name(dev), vid, did);
- uint32_t didvid = ((did & 0xffff) << 16) | (vid & 0xffff);
-
- if (!didvid)
- didvid = pci_read_config32(dev, PCI_VENDOR_ID);
- pci_write_config32(dev, 0x94, didvid);
-}
-
static struct pci_operations pcie_root_ops = {
- .set_subsystem = &pcie_root_set_subsystem,
+ .set_subsystem = pci_dev_set_subsystem,
};
static struct device_operations device_ops = {