cmd: mvebu/bubt: Rename variable image_size to hdr_size
authorPali Rohár <pali@kernel.org>
Sun, 8 Jan 2023 13:31:28 +0000 (14:31 +0100)
committerStefan Roese <sr@denx.de>
Wed, 1 Mar 2023 05:39:17 +0000 (06:39 +0100)
Variable image_size contains size of the header, not size of the whole
image. Rename this variable to reflect content.

Signed-off-by: Pali Rohár <pali@kernel.org>
cmd/mvebu/bubt.c

index df6b73c..72ed87b 100644 (file)
@@ -905,12 +905,12 @@ static int check_image_header(void)
        u32 offset, size;
        const struct a38x_main_hdr_v1 *hdr =
                (struct a38x_main_hdr_v1 *)get_load_addr();
-       const size_t image_size = a38x_header_size(hdr);
+       const size_t hdr_size = a38x_header_size(hdr);
 
-       if (!image_size)
+       if (!hdr_size)
                return -ENOEXEC;
 
-       checksum = image_checksum8(hdr, image_size);
+       checksum = image_checksum8(hdr, hdr_size);
        checksum -= hdr->checksum;
        if (checksum != hdr->checksum) {
                printf("Error: Bad A38x image header checksum. 0x%x != 0x%x\n",
@@ -944,7 +944,7 @@ static int check_image_header(void)
 #if defined(CONFIG_ARMADA_38X)
 static int a38x_image_is_secure(const struct a38x_main_hdr_v1 *hdr)
 {
-       u32 image_size = a38x_header_size(hdr);
+       const size_t hdr_size = a38x_header_size(hdr);
        struct a38x_opt_hdr_v1 *ohdr;
        u32 ohdr_size;
 
@@ -965,7 +965,7 @@ static int a38x_image_is_secure(const struct a38x_main_hdr_v1 *hdr)
                        break;
 
                ohdr = (struct a38x_opt_hdr_v1 *)((u8 *)ohdr + ohdr_size);
-               if ((u8 *)ohdr >= (u8 *)hdr + image_size)
+               if ((u8 *)ohdr >= (u8 *)hdr + hdr_size)
                        break;
        } while (1);