summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTim Wawrzynczak <twawrzynczak@chromium.org>2020-05-15 11:28:24 -0600
committerPatrick Georgi <pgeorgi@google.com>2020-05-18 07:18:00 +0000
commit979e80dc47d4f6bd1c1333c79ef59706d6c9403b (patch)
treed113173c0be669c3ad85fd542b329fa7871b89e7
parentfa42d568a00e5daadd35722790c529539227130e (diff)
downloadcoreboot-979e80dc47d4f6bd1c1333c79ef59706d6c9403b.tar.xz
device/pci_device: Remove useless pci_bus_ops_pci
The struct (formerly assigned to default_pci_ops_bus.ops_pci) only contained a NULL (well, 0) pointer for the set_subsystem callback, but usage of that callback is guarded with NULL checks when it is used, therefore it can be removed. TEST=still compiles Change-Id: I3943c8ae73b95e744a317264d7ceb8929cb28341 Signed-off-by: Tim Wawrzynczak <twawrzynczak@chromium.org> Reviewed-on: https://review.coreboot.org/c/coreboot/+/41432 Reviewed-by: Aaron Durbin <adurbin@chromium.org> Reviewed-by: Furquan Shaikh <furquan@google.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
-rw-r--r--src/device/pci_device.c5
1 files changed, 0 insertions, 5 deletions
diff --git a/src/device/pci_device.c b/src/device/pci_device.c
index 05848717e2..689325d2a9 100644
--- a/src/device/pci_device.c
+++ b/src/device/pci_device.c
@@ -770,17 +770,12 @@ struct device_operations default_pci_ops_dev = {
};
/** Default device operations for PCI bridges */
-static struct pci_operations pci_bus_ops_pci = {
- .set_subsystem = 0,
-};
-
struct device_operations default_pci_ops_bus = {
.read_resources = pci_bus_read_resources,
.set_resources = pci_dev_set_resources,
.enable_resources = pci_bus_enable_resources,
.scan_bus = pci_scan_bridge,
.reset_bus = pci_bus_reset,
- .ops_pci = &pci_bus_ops_pci,
};
/**