Use a signed type to test against negative return values from write(). While at it, reduce the scope of 'tmp'.
Signed-off-by: Sascha Hauer <[email protected]> --- arch/arm/mach-at91/boot_test_cmd.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/arch/arm/mach-at91/boot_test_cmd.c b/arch/arm/mach-at91/boot_test_cmd.c index 4fd1998ad0..7bb40f2e40 100644 --- a/arch/arm/mach-at91/boot_test_cmd.c +++ b/arch/arm/mach-at91/boot_test_cmd.c @@ -24,7 +24,6 @@ static int do_at91_boot_test(int argc, char *argv[]) int ret = 1; char *sram = "/dev/sram0"; u32 read_size, write_size; - u32 tmp = 0; while ((opt = getopt(argc, argv, "j:s:")) > 0) { switch (opt) { @@ -64,7 +63,7 @@ static int do_at91_boot_test(int argc, char *argv[]) } while (write_size) { - tmp = write(fd, buf, write_size); + int tmp = write(fd, buf, write_size); if (tmp < 0) { perror("write"); goto err_open; -- 2.29.2 _______________________________________________ barebox mailing list [email protected] http://lists.infradead.org/mailman/listinfo/barebox
