]> git.ipfire.org Git - thirdparty/u-boot.git/commitdiff
fastboot: Fix warning when CONFIG_SYS_64BIT_LBA is enable
authorPatrice Chotard <patrice.chotard@foss.st.com>
Fri, 29 Nov 2024 12:27:06 +0000 (13:27 +0100)
committerPatrice Chotard <patrice.chotard@foss.st.com>
Fri, 31 Jan 2025 07:57:39 +0000 (08:57 +0100)
If CONFIG_SYS_64BIT_LBA is enable, following compilation warning is
triggered:

 CC      drivers/fastboot/fb_mmc.o
../drivers/fastboot/fb_mmc.c: In function 'fb_mmc_erase_mmc_hwpart':
../drivers/fastboot/fb_mmc.c:215:35: warning: format '%lu' expects
argument of type 'long unsigned int', but argument 2 has type
'long long unsigned int' [-Wformat=]
  215 |         printf("........ erased %lu bytes from mmc hwpart[%u]\n",
      |                                 ~~^
      |                                   |
      |                                   long unsigned int
      |                                 %llu
  216 |                dev_desc->lba * dev_desc->blksz, dev_desc->hwpart);
      |                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |                              |
      |                              long long unsigned int
../drivers/fastboot/fb_mmc.c: In function 'fb_mmc_boot_ops':
../drivers/fastboot/fb_mmc.c:261:42: warning: format '%lu' expects
argument of type 'long unsigned int', but argument 2 has type
'long long unsigned int' [-Wformat=]
  261 |                 printf("........ wrote %lu bytes to EMMC_BOOT%d\n",
      |                                        ~~^
      |                                          |
      |                                          long unsigned int
      |                                        %llu
  262 |                        blkcnt * blksz, hwpart);
      |                        ~~~~~~~~~~~~~~
      |                               |
      |                               long long unsigned int

Signed-off-by: Patrice Chotard <patrice.chotard@foss.st.com>
Reviewed-by: Mattijs Korpershoek <mkorpershoek@baylibre.com>
Acked-by: Mattijs Korpershoek <mkorpershoek@baylibre.com>
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
drivers/fastboot/fb_mmc.c

index f11eb66761b1ed46d2e8401e71b0e4342b656ec6..dca7c222f35659b22d327541b245760a6a6d7b35 100644 (file)
@@ -211,8 +211,8 @@ static int fb_mmc_erase_mmc_hwpart(struct blk_desc *dev_desc)
                return 1;
        }
 
-       printf("........ erased %lu bytes from mmc hwpart[%u]\n",
-              dev_desc->lba * dev_desc->blksz, dev_desc->hwpart);
+       printf("........ erased %llu bytes from mmc hwpart[%u]\n",
+              (u64)(dev_desc->lba * dev_desc->blksz), dev_desc->hwpart);
 
        return 0;
 }
@@ -257,8 +257,8 @@ static void fb_mmc_boot_ops(struct blk_desc *dev_desc, void *buffer,
                        return;
                }
 
-               printf("........ wrote %lu bytes to EMMC_BOOT%d\n",
-                      blkcnt * blksz, hwpart);
+               printf("........ wrote %llu bytes to EMMC_BOOT%d\n",
+                      (u64)(blkcnt * blksz), hwpart);
        } else { /* erase */
                if (fb_mmc_erase_mmc_hwpart(dev_desc)) {
                        pr_err("Failed to erase EMMC_BOOT%d\n", hwpart);