net: Fix the displayed value of bytes transferred
authorSiddharth Vadapalli <s-vadapalli@ti.com>
Mon, 14 Aug 2023 04:53:47 +0000 (10:23 +0530)
committerTom Rini <trini@konsulko.com>
Tue, 22 Aug 2023 19:17:53 +0000 (15:17 -0400)
In the case of NETLOOP_SUCCESS, the decimal value of the u32 variable
"net_boot_file_size" is printed using "%d", resulting in negative values
being reported for large file sizes. Fix this by using "%u" to print the
decimal value corresponding to the bytes transferred.

Fixes: 1411157d8578 ("net: cosmetic: Fixup var names related to boot file")
Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com>
Reviewed-by: Tom Rini <trini@konsulko.com>
net/net.c

index 43abbac..e6f61f0 100644 (file)
--- a/net/net.c
+++ b/net/net.c
@@ -716,7 +716,7 @@ restart:
                case NETLOOP_SUCCESS:
                        net_cleanup_loop();
                        if (net_boot_file_size > 0) {
-                               printf("Bytes transferred = %d (%x hex)\n",
+                               printf("Bytes transferred = %u (%x hex)\n",
                                       net_boot_file_size, net_boot_file_size);
                                env_set_hex("filesize", net_boot_file_size);
                                env_set_hex("fileaddr", image_load_addr);