]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
cbfs: Simplify file iteration
authorSimon Glass <sjg@chromium.org>
Mon, 15 Mar 2021 05:00:15 +0000 (18:00 +1300)
committerSimon Glass <sjg@chromium.org>
Sat, 27 Mar 2021 00:59:37 +0000 (13:59 +1300)
In file_cbfs_next_file() there is a lot of complicated code to move to
the next file. Use the ALIGN() macros to simplify this.

Signed-off-by: Simon Glass <sjg@chromium.org>
fs/cbfs/cbfs.c

index a93dc3d0c16040564fd147a1682c692be7169445..9e534d15f28ac26b73e586baefb897cf7fe142ea 100644 (file)
@@ -133,7 +133,6 @@ static int file_cbfs_next_file(struct cbfs_priv *priv, void *start, int size,
 
        while (size >= align) {
                const struct cbfs_fileheader *file_header = start;
-               u32 step;
                int ret;
 
                /* Check if there's a file here. */
@@ -152,11 +151,7 @@ static int file_cbfs_next_file(struct cbfs_priv *priv, void *start, int size,
                        return log_msg_ret("fill", ret);
                }
 
-               step = header.len;
-               if (step % align)
-                       step = step + align - step % align;
-
-               *used += step;
+               *used += ALIGN(header.len, align);
                return 0;
        }