tools: firmware-utils: fix compiler warnings

This just fixes a lot of compiler warnings.

Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
SVN-Revision: 47181

diff --git a/tools/firmware-utils/src/airlink.c b/tools/firmware-utils/src/airlink.c
index 560a58df59ef5f95268faed783d0925443da053f..06c83a0fbb439df4d2c625c25ddab227610eb389 100644 (file)
--- a/tools/firmware-utils/src/airlink.c
+++ b/tools/firmware-utils/src/airlink.c
@@ -293,7 +293,7 @@ int main(int argc, char **argv)
        *((uint32_t *) & b[0x18]) = 0x0L;
 
        sum = crc32(b, 0x400);
-       printf("CRC32 sum0 - (%x, %x, %x)\n", sum, sum0, 0x400);
+       printf("CRC32 sum0 - (%x, %lx, %x)\n", sum, sum0, 0x400);
        if (EHDR)
                lseek(fd, 0x20, SEEK_SET);
        else
@@ -305,7 +305,7 @@ int main(int argc, char **argv)
        write(fd, &buf[0x18], 0x4);
 
        sum = crc32(buf, l0);
-       printf("CRC32 sum1 - (%x, %x, %x)\n", sum, sum1, l0);
+       printf("CRC32 sum1 - (%x, %lx, %x)\n", sum, sum1, l0);
        if (EHDR)
                lseek(fd, 0xC, SEEK_SET);
        else
@@ -319,7 +319,7 @@ int main(int argc, char **argv)
                unsigned long sum2 = buf[-0x8] | ((uint32_t)buf[-0x7] << 8) | ((uint32_t)buf[-0x6] << 16) | ((uint32_t)buf[-0x5] << 24);
                *((uint32_t *) & buf[-0x8]) = 0L;
                sum = crc32(buf - 0x4, len - 0x4);
-               printf("CRC32 sum2 - (%x, %x, %x)\n", sum, sum2,
+               printf("CRC32 sum2 - (%x, %lx, %lx)\n", sum, sum2,
                       len - 0x4);
                lseek(fd, 0, SEEK_SET);
                *((uint32_t *) & buf[-0x8]) = htonl(sum);

diff --git a/tools/firmware-utils/src/asustrx.c b/tools/firmware-utils/src/asustrx.c
index 80b6b1dfcc871fb9b5c883101ed39ed321330b2f..67f2680b31b6c4465abe8919bfe84c410a223d06 100644 (file)
--- a/tools/firmware-utils/src/asustrx.c
+++ b/tools/firmware-utils/src/asustrx.c
@@ -197,7 +197,7 @@ int main(int argc, char **argv) {
        }
        for (i = 0; i < bytes; i++) {
                if (buf[i]) {
-                       fprintf(stderr, "Input TRX doesn't have last 64 B empty %s\n");
+                       fprintf(stderr, "Input TRX doesn't have last 64 B empty %s\n", out_path);
                        err = -ENOSPC;
                        goto err;
                }
@@ -217,7 +217,7 @@ int main(int argc, char **argv) {
        bytes = sizeof(tail);
        fseek(out, -bytes, SEEK_CUR);
        if (fwrite(&tail, 1, bytes, out) != bytes) {
-               fprintf(stderr, "Couldn't write %zu B to %s\n", bytes);
+               fprintf(stderr, "Couldn't write %zu B to %s\n", bytes, out_path);
                err = -EIO;
                goto err;
        }
@@ -242,7 +242,7 @@ int main(int argc, char **argv) {
        hdr.crc32 = cpu_to_le32(crc32);
        rewind(out);
        if (fwrite(&hdr, 1, bytes, out) != bytes) {
-               fprintf(stderr, "Couldn't write %zu B to %s\n", bytes);
+               fprintf(stderr, "Couldn't write %zu B to %s\n", bytes, out_path);
                err = -EIO;
                goto err;
        }

diff --git a/tools/firmware-utils/src/imagetag.c b/tools/firmware-utils/src/imagetag.c
index f04f98d560ab88b5a9d814b205ee022de886b506..6a46a7ebd995a95822ffa7acc63f668e95b9e6e0 100644 (file)
--- a/tools/firmware-utils/src/imagetag.c
+++ b/tools/firmware-utils/src/imagetag.c
@@ -15,6 +15,7 @@
 #include <unistd.h>
 #include <sys/stat.h>
 #include <netinet/in.h>
+#include <inttypes.h>
 
 #include "bcm_tag.h"
 #include "imagetag_cmdline.h"
@@ -304,7 +305,7 @@ int tagfile(const char *kernel, const char *rootfs, const char *bin, \
        sprintf(tag.totalLength, "%lu", imagelen);
 
        if (args->cfe_given) {
-         sprintf(tag.cfeAddress, "%lu", flash_start);
+         sprintf(tag.cfeAddress, "%" PRIu32, flash_start);
          sprintf(tag.cfeLength, "%lu", cfelen);
        } else {
          /* We don't include CFE */
@@ -345,7 +346,7 @@ int tagfile(const char *kernel, const char *rootfs, const char *bin, \
        }
 
        if (args->altinfo_given) {
-         strncpy(&tag.information1[0], args->altinfo_arg, ALTTAGINFO_LEN);
+         strncpy(tag.information1, args->altinfo_arg, TAGINFO1_LEN);
        }
 
        if (args->second_image_flag_given) {

diff --git a/tools/firmware-utils/src/mkbrnimg.c b/tools/firmware-utils/src/mkbrnimg.c
index fde30b2ea58e210b66c4d4312695faecd4f6b585..b7a73ff59d2b7b22c1a952ebc39b14837dd02cf0 100644 (file)
--- a/tools/firmware-utils/src/mkbrnimg.c
+++ b/tools/firmware-utils/src/mkbrnimg.c
@@ -142,7 +142,7 @@ static void appendfile(int outfd, char *path, int kernel) {
 
        // write padding
        padded_len = ((len + sizeof(footer) + sizeof(padding) - 1) & ~(sizeof(padding) - 1)) - sizeof(footer);
-       fprintf(stderr, "len=%08x padded_len=%08x\n", len, padded_len);
+       fprintf(stderr, "len=%08zx padded_len=%08zx\n", len, padded_len);
        write(outfd, padding, padded_len - len);
 
        // write footer

diff --git a/tools/firmware-utils/src/mkcasfw.c b/tools/firmware-utils/src/mkcasfw.c
index 626e77d4c49bca4ae42cb906d61ceffdec024da3..5655bf1894c04d59e0769bb80f542f1ea04e0102 100644 (file)
--- a/tools/firmware-utils/src/mkcasfw.c
+++ b/tools/firmware-utils/src/mkcasfw.c
@@ -258,7 +258,7 @@ static struct board_info boards[] = {
 #define ERRS(fmt, ...) do { \
        int save = errno; \
        fflush(0); \
-       fprintf(stderr, "[%s] *** error: " fmt "\n", \
+       fprintf(stderr, "[%s] *** error: " fmt ": %s\n", \
                        progname, ## __VA_ARGS__, strerror(save)); \
 } while (0)
 
@@ -708,7 +708,7 @@ image_writeout(FILE *outfile, struct image_desc *desc)
 
        /* write padding data if neccesary */
        padlen = desc->out_size - desc->file_size;
-       DBG(1,"padding desc, length=%d", padlen);
+       DBG(1,"padding desc, length=%zu", padlen);
        res = write_out_padding(outfile, padlen, desc->padc, &css);
 
        desc->csum = csum_get(&css);
@@ -985,11 +985,11 @@ main(int argc, char *argv[])
                        res = ERR_FATAL;
 
                if (keep_invalid_images == 0) {
-                       WARN("generation of invalid images disabled", ofname);
+                       WARN("generation of invalid images \"%s\" disabled", ofname);
                        goto out;
                }
 
-               WARN("generating invalid image", ofname);
+               WARN("generating invalid image: \"%s\"", ofname);
        }
 
        outfile = fopen(ofname, "w");

diff --git a/tools/firmware-utils/src/mkcsysimg.c b/tools/firmware-utils/src/mkcsysimg.c
index c00096f87d5f8868507da2fc5c42bc8cac85e5d2..77fbbaa57fa93e9c0ef492597ced6c92a8138b39 100644 (file)
--- a/tools/firmware-utils/src/mkcsysimg.c
+++ b/tools/firmware-utils/src/mkcsysimg.c
@@ -199,7 +199,7 @@ static struct board_info boards[] = {
 #define ERRS(fmt, ...) do { \
        int save = errno; \
        fflush(0); \
-       fprintf(stderr, "[%s] *** error: " fmt "\n", progname, ## __VA_ARGS__ \
+       fprintf(stderr, "[%s] *** error: " fmt ": %s\n", progname, ## __VA_ARGS__ \
                , strerror(save)); \
 } while (0)
 
@@ -647,7 +647,7 @@ block_writeout_data(FILE *outfile, struct csys_block *block)
 
        /* write padding data if neccesary */
        padlen = block->size_avail - block->file_size;
-       DBG(1,"padding block, length=%d", padlen);
+       DBG(1,"padding block, length=%zu", padlen);
        res = write_out_padding(outfile, padlen, block->padc, block->css);
 
        return res;
@@ -1122,11 +1122,11 @@ main(int argc, char *argv[])
                        res = ERR_FATAL;
 
                if (keep_invalid_images == 0) {
-                       WARN("generation of invalid images disabled", ofname);
+                       WARN("generation of invalid images \"%s\" disabled", ofname);
                        goto out;
                }
 
-               WARN("generating invalid image", ofname);
+               WARN("generating invalid image: \"%s\"", ofname);
        }
 
        outfile = fopen(ofname, "w");

diff --git a/tools/firmware-utils/src/mkdniimg.c b/tools/firmware-utils/src/mkdniimg.c
index 7230f09d4921c2b5b75a9a388499a69075a3c40f..852b07dd9f652c5e7cdc8c9335cf26e5f7fb1d7c 100644 (file)
--- a/tools/firmware-utils/src/mkdniimg.c
+++ b/tools/firmware-utils/src/mkdniimg.c
@@ -43,7 +43,7 @@ static char *board_id;
 #define ERRS(fmt, ...) do { \
        int save = errno; \
        fflush(0); \
-       fprintf(stderr, "[%s] *** error: " fmt "\n", \
+       fprintf(stderr, "[%s] *** error: " fmt ": %s\n", \
                        progname, ## __VA_ARGS__, strerror(save)); \
 } while (0)
 

diff --git a/tools/firmware-utils/src/mkplanexfw.c b/tools/firmware-utils/src/mkplanexfw.c
index 1bdccb7c0263ab41b479fa5203a3824cf1c8fa44..0b71f80438f38a45b23a34b3c131305d8830ecc3 100644 (file)
--- a/tools/firmware-utils/src/mkplanexfw.c
+++ b/tools/firmware-utils/src/mkplanexfw.c
@@ -82,7 +82,7 @@ static struct board_info boards[] = {
 #define ERRS(fmt, ...) do { \
        int save = errno; \
        fflush(0); \
-       fprintf(stderr, "[%s] *** error: " fmt "\n", \
+       fprintf(stderr, "[%s] *** error: " fmt ": %s\n", \
                        progname, ## __VA_ARGS__, strerror(save)); \
 } while (0)
 

diff --git a/tools/firmware-utils/src/mktplinkfw.c b/tools/firmware-utils/src/mktplinkfw.c
index 1302d467e143e1d90d8b6c182774e5de88b3e4d8..96fd6a00cd9469a70c000b64d8d60c16ab4d0407 100644 (file)
--- a/tools/firmware-utils/src/mktplinkfw.c
+++ b/tools/firmware-utils/src/mktplinkfw.c
@@ -450,7 +450,7 @@ static struct board_info boards[] = {
 #define ERRS(fmt, ...) do { \
        int save = errno; \
        fflush(0); \
-       fprintf(stderr, "[%s] *** error: " fmt "\n", \
+       fprintf(stderr, "[%s] *** error: " fmt ": %s\n", \
                        progname, ## __VA_ARGS__, strerror(save)); \
 } while (0)
 

diff --git a/tools/firmware-utils/src/mktplinkfw2.c b/tools/firmware-utils/src/mktplinkfw2.c
index 25915e68bfdcdd35e22170ccbb08cc428c9ce078..92775290f816f95d461829ad9eb55439d54d82f6 100644 (file)
--- a/tools/firmware-utils/src/mktplinkfw2.c
+++ b/tools/firmware-utils/src/mktplinkfw2.c
@@ -189,7 +189,7 @@ static struct board_info boards[] = {
 #define ERRS(fmt, ...) do { \
        int save = errno; \
        fflush(0); \
-       fprintf(stderr, "[%s] *** error: " fmt "\n", \
+       fprintf(stderr, "[%s] *** error: " fmt ": %s\n", \
                        progname, ## __VA_ARGS__, strerror(save)); \
 } while (0)
 

diff --git a/tools/firmware-utils/src/mkzcfw.c b/tools/firmware-utils/src/mkzcfw.c
index 7674e706c99b2c6c6f21ea2efb94d6d84fc7d93f..2326f1ff5c5a4b4925446712f52720116c2d6bf6 100644 (file)
--- a/tools/firmware-utils/src/mkzcfw.c
+++ b/tools/firmware-utils/src/mkzcfw.c
@@ -100,7 +100,7 @@ static struct board_info boards[] = {
 #define ERRS(fmt, ...) do { \
        int save = errno; \
        fflush(0); \
-       fprintf(stderr, "[%s] *** error: " fmt "\n", \
+       fprintf(stderr, "[%s] *** error: " fmt ": %s\n", \
                        progname, ## __VA_ARGS__, strerror(save)); \
 } while (0)
 

diff --git a/tools/firmware-utils/src/mkzynfw.c b/tools/firmware-utils/src/mkzynfw.c
index 36c176f4325f34c7c45098d2dbb0886cb49b2c4f..ccbabfe7f20b2bfb3155eaa053b6e1dff3b8d15a 100644 (file)
--- a/tools/firmware-utils/src/mkzynfw.c
+++ b/tools/firmware-utils/src/mkzynfw.c
@@ -27,6 +27,7 @@
 #if defined(__CYGWIN__)
 #  include <byteswap.h>
 #endif
+#include <inttypes.h>
 
 #include "zynos.h"
 
@@ -687,7 +688,7 @@ write_out_file(FILE *outfile, char *name, size_t len, struct csum_state *css)
        FILE *f;
        int res;
 
-       DBG(2, "writing out file, name=%s, len=%d",
+       DBG(2, "writing out file, name=%s, len=%zu",
                name, len);
 
        errno = 0;
@@ -988,7 +989,7 @@ calc_block_offsets(int type, uint32_t *offset)
        uint32_t avail;
        int i, res;
 
-       DBG(1,"calculating block offsets, starting with %lu",
+       DBG(1,"calculating block offsets, starting with %" PRIu32,
                *offset);
 
        res = 0;

diff --git a/tools/firmware-utils/src/osbridge-crc.c b/tools/firmware-utils/src/osbridge-crc.c
index f2e3920a48c6549aeb2be4a4d4d7c593e8b95275..5fd236a0746c8885a0e96e6ba90065d4bd59509e 100644 (file)
--- a/tools/firmware-utils/src/osbridge-crc.c
+++ b/tools/firmware-utils/src/osbridge-crc.c
@@ -51,7 +51,7 @@ static char *ofname;
 #define ERRS(fmt, ...) do { \
        int save = errno; \
        fflush(0); \
-       fprintf(stderr, "[%s] *** error: " fmt "\n", \
+       fprintf(stderr, "[%s] *** error: " fmt ": %s\n", \
                        progname, ## __VA_ARGS__, strerror(save)); \
 } while (0)
 

diff --git a/tools/firmware-utils/src/pc1crypt.c b/tools/firmware-utils/src/pc1crypt.c
index 9c2eb83f666161887691385b45d990bf8f1ad786..fe41b3dab5c0c70e55bf6327d55d12f9be8a55ec 100644 (file)
--- a/tools/firmware-utils/src/pc1crypt.c
+++ b/tools/firmware-utils/src/pc1crypt.c
@@ -208,7 +208,7 @@ static int decrypt;
 #define ERRS(fmt, ...) do { \
        int save = errno; \
        fflush(0); \
-       fprintf(stderr, "[%s] *** error: " fmt "\n", \
+       fprintf(stderr, "[%s] *** error: " fmt ": %s\n", \
                        progname, ## __VA_ARGS__, strerror(save)); \
 } while (0)
 

diff --git a/tools/firmware-utils/src/seama.c b/tools/firmware-utils/src/seama.c
index 944ed3a444e5701949f6c2ffbd8a25981b3afdca..05aee8e76aacd8824bdfa300f5bc2e6092d05c5b 100644 (file)
--- a/tools/firmware-utils/src/seama.c
+++ b/tools/firmware-utils/src/seama.c
@@ -235,10 +235,10 @@ static int verify_seama(const char * fname, int msg)
                        {
                                printf("SEAMA ==========================================\n");
                                printf("  magic      : %08x\n", ntohl(shdr.magic));
-                               printf("  meta size  : %d bytes\n", msize);
+                               printf("  meta size  : %zu bytes\n", msize);
                                for (i=0; i<msize; i+=(strlen((const char *)&buf[i])+1))
                                        printf("  meta data  : %s\n", &buf[i]);
-                               printf("  image size : %d bytes\n", isize);
+                               printf("  image size : %zu bytes\n", isize);
                        }
 
                        /* verify checksum */
@@ -473,7 +473,7 @@ static void extract_file(const char * output)
                                fread(buf, sizeof(char), msize, ifh);
                                if (match_meta((const char *)buf, msize))
                                {
-                                       printf("SEAMA: found image @ '%s', image size: %d\n", o_images[i], isize);
+                                       printf("SEAMA: found image @ '%s', image size: %zu\n", o_images[i], isize);
                                        /* open output file */
                                        ofh = fopen(output, "w");
                                        if (!ofh) printf("SEAMA: unable to open '%s' for writting.\n",output);