'found' is only used at the end of the function to print a debug message. No need to maintain a variable if we can just return 0 immediately when a driver was bound successfully.
Reviewed-by: Simon Glass <[email protected]> Signed-off-by: Markus Schneider-Pargmann (TI.com) <[email protected]> --- drivers/core/lists.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/core/lists.c b/drivers/core/lists.c index 52ba6c7bfa007b660b854de62c97ad3c1761f1a5..9d1ca38212ee7f53b8894f964f096611c8ec20a5 100644 --- a/drivers/core/lists.c +++ b/drivers/core/lists.c @@ -204,7 +204,6 @@ int lists_bind_fdt(struct udevice *parent, ofnode node, struct udevice **devp, const struct udevice_id *id; struct driver *entry; struct udevice *dev; - bool found = false; const char *name, *compat_list, *compat; int compat_length, i; int ret = 0; @@ -275,14 +274,14 @@ int lists_bind_fdt(struct udevice *parent, ofnode node, struct udevice **devp, ret); return log_msg_ret("bind", ret); } else { - found = true; if (devp) *devp = dev; + return 0; } break; } - if (!found && ret != -ENODEV) + if (ret != -ENODEV) log_debug("No match for node '%s'\n", name); return 0; -- 2.51.0
