leandrof/u-boot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

bootstd: Tidy up var naming in bootdev_setup_iter_order()

Browse Source 
Avoid using 'count' to mean either a count or an error, since this is
confusing. In fact, the called function never return 0, since that is an
error.

Use 'ret' instead.

Signed-off-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Simon Glass 2022-07-30 15:52:20 -06:00 committed by Tom Rini
parent 988cacaeed
commit a18686cda1
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
boot/bootdev-uclass.c
View File

@ -604,14 +604,14 @@ int bootdev_setup_iter_order(struct bootflow_iter *iter, struct udevice **devp)
log_debug("Expected %d bootdevs, found %d using aliases\n",
count, upto);
count = build_order(bootstd, order, upto);
if (count < 0) {
ret = build_order(bootstd, order, upto);
if (ret < 0) {
free(order);
return log_msg_ret("build", count);
return log_msg_ret("build", ret);
}
iter->num_devs = ret;
iter->dev_order = order;
iter->num_devs = count;
iter->cur_dev = 0;
dev = *order;