[PATCH v3 01/13] cbfs: Rename the result variable
Simon Glass
sjg at chromium.org
Fri May 22 18:32:02 CEST 2020
At present the result variable in the cbfs_priv is called 'result' as is
the local variable in a few functions. Change the latter to 'ret' which is
more common in U-Boot and avoids confusion.
Reviewed-by: Bin Meng <bmeng.cn at gmail.com>
Signed-off-by: Simon Glass <sjg at chromium.org>
---
Changes in v3: None
Changes in v2: None
fs/cbfs/cbfs.c | 20 ++++++++++----------
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/fs/cbfs/cbfs.c b/fs/cbfs/cbfs.c
index 1aa6f8ee847..70440aa80b6 100644
--- a/fs/cbfs/cbfs.c
+++ b/fs/cbfs/cbfs.c
@@ -145,18 +145,18 @@ static void file_cbfs_fill_cache(struct cbfs_priv *priv, u8 *start, u32 size,
priv->file_cache = NULL;
while (size >= align) {
- int result;
+ int ret;
u32 used;
new_node = (struct cbfs_cachenode *)
malloc(sizeof(struct cbfs_cachenode));
- result = file_cbfs_next_file(priv, start, size, align, new_node,
- &used);
+ ret = file_cbfs_next_file(priv, start, size, align, new_node,
+ &used);
- if (result < 0) {
+ if (ret < 0) {
free(new_node);
return;
- } else if (result == 0) {
+ } else if (ret == 0) {
free(new_node);
break;
}
@@ -341,15 +341,15 @@ const struct cbfs_cachenode *file_cbfs_find_uncached(uintptr_t end_of_rom,
align = priv->header.align;
while (size >= align) {
- int result;
+ int ret;
u32 used;
- result = file_cbfs_next_file(priv, start, size, align, &node,
- &used);
+ ret = file_cbfs_next_file(priv, start, size, align, &node,
+ &used);
- if (result < 0)
+ if (ret < 0)
return NULL;
- else if (result == 0)
+ else if (ret == 0)
break;
if (!strcmp(name, node.name))
--
2.27.0.rc0.183.gde8f92d652-goog
More information about the U-Boot
mailing list