mirror of
https://source.denx.de/u-boot/u-boot.git
synced 2026-05-05 04:36:13 +02:00
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 <simon.glass@canonical.com> Reviewed-by: Mattijs Korpershoek <mkorpershoek@kernel.org> Signed-off-by: Markus Schneider-Pargmann (TI.com) <msp@baylibre.com>
This commit is contained in:
parent
6f3e63b4de
commit
41cf66011f
@ -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;
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user