device: Export enable_static_device() function
The work done by enable_static_devices() and scan_generic_bus()
is common and can be used by other device handlers to enable a
single static device.
Signed-off-by: Duncan Laurie <dlaurie@google.com>
Change-Id: Ibfde9c4eb794714ebd9800e52b91169ceba15266
Reviewed-on: https://review.coreboot.org/c/coreboot/+/46541
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Furquan Shaikh <furquan@google.com>
diff --git a/src/device/root_device.c b/src/device/root_device.c
index 640ea50..1f51786 100644
--- a/src/device/root_device.c
+++ b/src/device/root_device.c
@@ -7,6 +7,18 @@
const char mainboard_name[] = CONFIG_MAINBOARD_VENDOR " " CONFIG_MAINBOARD_PART_NUMBER;
+void enable_static_device(struct device *dev)
+{
+ if (dev->chip_ops && dev->chip_ops->enable_dev)
+ dev->chip_ops->enable_dev(dev);
+
+ if (dev->ops && dev->ops->enable)
+ dev->ops->enable(dev);
+
+ printk(BIOS_DEBUG, "%s %s\n", dev_path(dev),
+ dev->enabled ? "enabled" : "disabled");
+}
+
/**
* Enable devices on static buses.
*
@@ -32,15 +44,7 @@
for (link = bus->link_list; link; link = link->next) {
for (child = link->children; child; child = child->sibling) {
-
- if (child->chip_ops && child->chip_ops->enable_dev)
- child->chip_ops->enable_dev(child);
-
- if (child->ops && child->ops->enable)
- child->ops->enable(child);
-
- printk(BIOS_DEBUG, "%s %s\n", dev_path(child),
- child->enabled ? "enabled" : "disabled");
+ enable_static_device(child);
}
}
}
@@ -58,18 +62,9 @@
link->secondary = ++bus_max;
for (child = link->children; child; child = child->sibling) {
-
- if (child->chip_ops && child->chip_ops->enable_dev)
- child->chip_ops->enable_dev(child);
-
- if (child->ops && child->ops->enable)
- child->ops->enable(child);
-
+ enable_static_device(child);
printk(BIOS_DEBUG, "bus: %s[%d]->", dev_path(child->bus->dev),
child->bus->link_num);
-
- printk(BIOS_DEBUG, "%s %s\n", dev_path(child),
- child->enabled ? "enabled" : "disabled");
}
}