summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFurquan Shaikh <furquan@google.com>2021-09-22 13:35:48 -0700
committerPaul Fagerburg <pfagerburg@chromium.org>2021-09-24 21:20:07 +0000
commit20c709daade1b894ffc8559c67d35af0172efe94 (patch)
tree4d12fefe9932d362b2c801b876e33664b528d8b1
parent0f73791606b216912be8d88c83e21ac0b8fe0d6a (diff)
device: Drop unused function `dev_find_matching_device_on_bus`
With use of device pointers, `dev_find_matching_device_on_bus()` is now unused and hence this change drops the function. Change-Id: I30fcb2d9932d770ca614cceffb15646ce8256465 Signed-off-by: Furquan Shaikh <furquan@google.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/57846 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Karthik Ramasubramanian <kramasub@google.com> Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
-rw-r--r--src/device/device_const.c13
-rw-r--r--src/include/device/device.h14
2 files changed, 0 insertions, 27 deletions
diff --git a/src/device/device_const.c b/src/device/device_const.c
index 92bae37930..c629f0524e 100644
--- a/src/device/device_const.c
+++ b/src/device/device_const.c
@@ -70,19 +70,6 @@ DEVTREE_CONST struct device *dev_find_path(
return result;
}
-DEVTREE_CONST struct device *dev_find_matching_device_on_bus(const struct bus *bus,
- match_device_fn fn)
-{
- DEVTREE_CONST struct device *child = NULL;
-
- while ((child = dev_bus_each_child(bus, child)) != NULL) {
- if (fn(child))
- break;
- }
-
- return child;
-}
-
/**
* Given a device pointer, find the next PCI device.
*
diff --git a/src/include/device/device.h b/src/include/device/device.h
index 0fadc0cd1e..6bd6e0e551 100644
--- a/src/include/device/device.h
+++ b/src/include/device/device.h
@@ -214,20 +214,6 @@ DEVTREE_CONST struct device *dev_find_path(
enum device_path_type path_type);
struct device *dev_find_lapic(unsigned int apic_id);
int dev_count_cpu(void);
-
-/*
- * Signature for matching function that is used by dev_find_matching_device_on_bus() to decide
- * if the device being considered is the one that matches the caller's criteria. This function
- * is supposed to return true if the provided device matches the criteria, else false.
- */
-typedef bool (*match_device_fn)(DEVTREE_CONST struct device *dev);
-/*
- * Returns the first device on the bus that the match_device_fn returns true for. If no such
- * device is found, it returns NULL.
- */
-DEVTREE_CONST struct device *dev_find_matching_device_on_bus(const struct bus *bus,
- match_device_fn fn);
-
struct device *add_cpu_device(struct bus *cpu_bus, unsigned int apic_id,
int enabled);
void set_cpu_topology(struct device *cpu, unsigned int node,