From 41cf66011fc80bd122c3ebbeda4218547c48c297 Mon Sep 17 00:00:00 2001 From: "Markus Schneider-Pargmann (TI.com)" Date: Tue, 27 Jan 2026 12:03:36 +0100 Subject: [PATCH] dm: core: lists_bind_fdt: Replace found variable '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 Reviewed-by: Mattijs Korpershoek Signed-off-by: Markus Schneider-Pargmann (TI.com) --- 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 52ba6c7bfa0..9d1ca38212e 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;