diff options
| author | Tim Wawrzynczak <twawrzynczak@chromium.org> | 2020-05-15 11:28:24 -0600 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2020-05-18 07:18:00 +0000 |
| commit | 979e80dc47d4f6bd1c1333c79ef59706d6c9403b (patch) | |
| tree | d113173c0be669c3ad85fd542b329fa7871b89e7 | |
| parent | fa42d568a00e5daadd35722790c529539227130e (diff) | |
| download | coreboot-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.c | 5 |
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, }; /** |