John Su has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/63101 )
Change subject: mb/google/brya: Set pciexp_hotplug_dummy_ops.set_resources as no-op ......................................................................
mb/google/brya: Set pciexp_hotplug_dummy_ops.set_resources as no-op
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 --- M src/device/pciexp_device.c 1 file changed, 1 insertion(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/01/63101/1
diff --git a/src/device/pciexp_device.c b/src/device/pciexp_device.c index c525301..3614340 100644 --- a/src/device/pciexp_device.c +++ b/src/device/pciexp_device.c @@ -585,6 +585,7 @@
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)