firmware-utils: mkmerakifw: fix compiler warnings/errors

mkmerakifw.c:253:11: error: comparison of integer expressions of different signedness: ‘long int’ and ‘size_t’ {aka ‘long unsigned int’} [-Werror=sign-compare]
mkmerakifw.c:273:2: error: ignoring return value of ‘fread’, declared with attribute warn_unused_result [-Werror=unused-result]

Signed-off-by: Petr Štetiar <ynezz@true.cz>

diff --git a/tools/firmware-utils/src/mkmerakifw.c b/tools/firmware-utils/src/mkmerakifw.c
index 1a50f1658f7f9f90ca7fee7102d9065037085f16..45b9a601bda9128651078946e1f368930255e483 100644 (file)
--- a/tools/firmware-utils/src/mkmerakifw.c
+++ b/tools/firmware-utils/src/mkmerakifw.c
@@ -176,7 +176,7 @@ int main(int argc, char *argv[])
 {
        int ret = EXIT_FAILURE;
        long klen;
-       size_t kspace;
+       long kspace;
        unsigned char *kernel;
        size_t buflen;
        unsigned char *buf;
@@ -270,7 +270,11 @@ int main(int argc, char *argv[])
 
        /* Load kernel */
        kernel = buf + HDR_LENGTH;
-       fread(kernel, klen, 1, in);
+       size_t r = fread(kernel, klen, 1, in);
+       if (r != 1) {
+               ERRS("unable to load kernel: %s\n");
+               goto err_close_in;
+       }
 
        /* Write magic values */
        writel(buf, HDR_OFF_MAGIC1, board->magic1);