diff options
| author | John Su <john_su@compal.corp-partner.google.com> | 2022-03-25 10:37:52 +0800 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2022-03-28 02:28:52 +0000 |
| commit | 3ecc77722c9f7e857b9c034d4e01ee5f298b6c1f (patch) | |
| tree | 9ce2eec5104ec6400e0f187f2c138e09ff331057 | |
| parent | 4dac96d9680747d9826f969bcacd8b7c922e3347 (diff) | |
device/pciexp_device: Set resources for pciexp_hotplug_dummy_ops
Without setting the set_resources field for pciexp_hotplug_dummy_ops,
we will get an error during pciexp_hotplug_dummy.
[ERROR] NONE missing set_resources
Because the set_resources field is considered mandatory, explicitly set
it as no-op noop_set_resources.
BUG=b:220639445
TEST=emerge-brya coreboot
Signed-off-by: John Su <john_su@compal.corp-partner.google.com>
Change-Id: Ifee7479c69cf16025dbd4e3924056ed7f8e253cf
Reviewed-on: https://review.coreboot.org/c/coreboot/+/63101
Reviewed-by: Eric Lai <eric_lai@quanta.corp-partner.google.com>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
| -rw-r--r-- | src/device/pciexp_device.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/device/pciexp_device.c b/src/device/pciexp_device.c index c52530196d..36143402ab 100644 --- a/src/device/pciexp_device.c +++ b/src/device/pciexp_device.c @@ -585,6 +585,7 @@ static void pciexp_hotplug_dummy_read_resources(struct device *dev) static struct device_operations pciexp_hotplug_dummy_ops = { .read_resources = pciexp_hotplug_dummy_read_resources, + .set_resources = noop_set_resources, }; void pciexp_hotplug_scan_bridge(struct device *dev) |