[PATCH v2 2/6] dm: core: lists_bind_fdt: Replace found variable
Markus Schneider-Pargmann (TI.com)
msp at baylibre.com
Thu Jan 8 12:16:28 CET 2026
'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.
Signed-off-by: Markus Schneider-Pargmann (TI.com) <msp at baylibre.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 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
More information about the U-Boot
mailing list