diff options
author | Marcel Holtmann <marcel@holtmann.org> | 2009-01-29 17:58:28 +0100 |
---|---|---|
committer | Marcel Holtmann <marcel@holtmann.org> | 2009-01-29 17:58:28 +0100 |
commit | 9c0b5859e6cc4b7a0e925fde8665990281b265d3 (patch) | |
tree | 8994dbe0942d56c9a7cf470fc3b2316d1f91e0a3 /tools/dfutool.c | |
parent | 1ce81e1e0794a5d619016e17b33b533b614fc6a9 (diff) |
Fix mostly every warning caused by -Wsign-compare
Diffstat (limited to 'tools/dfutool.c')
-rw-r--r-- | tools/dfutool.c | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/tools/dfutool.c b/tools/dfutool.c index 10387bbf..cab39feb 100644 --- a/tools/dfutool.c +++ b/tools/dfutool.c @@ -321,9 +321,10 @@ static void cmd_verify(char *device, int argc, char **argv) uint16_t bcd; char str[16]; unsigned char *buf; - unsigned long size; + size_t size; char *filename; - int i, fd, len; + unsigned int i, len; + int fd; if (argc < 2) { usage(); @@ -350,7 +351,7 @@ static void cmd_verify(char *device, int argc, char **argv) exit(1); } - if (read(fd, buf, size) < size) { + if (read(fd, buf, size) < (ssize_t) size) { perror("Can't load firmware"); free(buf); close(fd); @@ -419,10 +420,12 @@ static void cmd_upgrade(char *device, int argc, char **argv) struct dfu_suffix suffix; struct stat st; char *buf; - unsigned long filesize, count, timeout = 0; + size_t filesize; + unsigned long count, timeout = 0; char *filename; uint32_t crc, dwCRC; - int fd, i, block, len, size, sent = 0, try = 10; + unsigned int i; + int fd, block, len, size, sent = 0, try = 10; if (argc < 2) { usage(); @@ -449,7 +452,7 @@ static void cmd_upgrade(char *device, int argc, char **argv) exit(1); } - if (read(fd, buf, filesize) < filesize) { + if (read(fd, buf, filesize) < (ssize_t) filesize) { perror("Can't load firmware"); free(buf); close(fd); |