libv4l: Fix errors and warnings when compiling with ICC

From: Gregor Jasny <jasny@vidsoft.de>

libv4l: Fix errors and warnings when compiling with ICC

Priority: normal

Signed-off-by: Gregor Jasny <jasny@vidsoft.de>
Signed-off-by: Hans de Goede <j.w.r.degoede@hhs.nl>

diff --git a/lib/libv4l1/Makefile b/lib/libv4l1/Makefile
index 4efc740..769811a 100644 (file)
--- a/lib/libv4l1/Makefile
+++ b/lib/libv4l1/Makefile
@@ -1,7 +1,7 @@
 override CPPFLAGS += -I../include -I../../../../linux/include
 
 CFLAGS := -g -O1
-CFLAGS += -Wall -W -Wno-unused -Wpointer-arith -Wstrict-prototypes
+CFLAGS += -Wall -Wno-unused -Wpointer-arith -Wstrict-prototypes -Wmissing-prototypes
 
 LIBS = -lpthread
 

diff --git a/lib/libv4l1/libv4l1.c b/lib/libv4l1/libv4l1.c
index 80d2fa0..39f2d29 100644 (file)
--- a/lib/libv4l1/libv4l1.c
+++ b/lib/libv4l1/libv4l1.c
@@ -59,6 +59,7 @@
 #include <linux/videodev.h>
 #include <linux/videodev2.h>
 #include <libv4l2.h>
+#include "libv4l1.h"
 #include "libv4l1-priv.h"
 
 #define V4L1_SUPPORTS_ENUMINPUT 0x01
@@ -582,7 +583,7 @@ int v4l1_ioctl (int fd, unsigned long int request, ...)
          input2.index = chan->channel;
          result = v4l2_ioctl(fd, VIDIOC_ENUMINPUT, &input2);
          if (result == 0) {
-           snprintf(chan->name, sizeof(chan->name), "%s", input2.name);
+           snprintf(chan->name, sizeof(chan->name), "%s", (char*)input2.name);
            if (input2.type == V4L2_INPUT_TYPE_TUNER) {
              chan->tuners = 1;
              chan->type = VIDEO_TYPE_TV;

diff --git a/lib/libv4l1/log.c b/lib/libv4l1/log.c
index cccc3e6..74ce0f2 100644 (file)
--- a/lib/libv4l1/log.c
+++ b/lib/libv4l1/log.c
@@ -25,6 +25,7 @@
 #include <asm/types.h>
 /* end broken header workaround includes */
 #include <linux/videodev.h>
+#include "libv4l1-priv.h"
 
 #define ARRAY_SIZE(x) (sizeof(x)/sizeof((x)[0]))
 
@@ -92,9 +93,9 @@ void v4l1_log_ioctl(unsigned long int request, void *arg, int result)
                          break;
     case VIDIOCGWIN:
     case VIDIOCSWIN:
-      fprintf(v4l1_log_file,"width\t%d\n",
+      fprintf(v4l1_log_file,"width\t%u\n",
        ((struct video_window *)arg)->width);
-      fprintf(v4l1_log_file,"height\t%d\n",
+      fprintf(v4l1_log_file,"height\t%u\n",
        ((struct video_window *)arg)->height);
       break;
 
@@ -115,7 +116,7 @@ void v4l1_log_ioctl(unsigned long int request, void *arg, int result)
                          fprintf(v4l1_log_file,"palette        %d\n",( (int)((struct video_picture*)arg)->palette)     );
                          break;
 
-         case VIDIOCCAPTURE: fprintf(v4l1_log_file,"on/of?     %d\n",  *((int *)arg)                                   );
+         case VIDIOCCAPTURE: fprintf(v4l1_log_file,"on/off?    %d\n",  *((int *)arg)                                   );
                          break;
 
          case VIDIOCSYNC: fprintf(v4l1_log_file,"sync  %d\n",  *((int *)arg)                                   );

diff --git a/lib/libv4l1/v4l1compat.c b/lib/libv4l1/v4l1compat.c
index f1134fe..8edc9f9 100644 (file)
--- a/lib/libv4l1/v4l1compat.c
+++ b/lib/libv4l1/v4l1compat.c
@@ -26,6 +26,9 @@
 #include <fcntl.h>
 #include <libv4l1.h>
 
+#include <sys/ioctl.h>
+#include <sys/mman.h>
+
 /* Check that open/read/mmap is not a define */
 #if defined open || defined read || defined mmap
 #error open/read/mmap is a prepocessor macro !!

diff --git a/lib/libv4l2/Makefile b/lib/libv4l2/Makefile
index ff3cb9d..63c2283 100644 (file)
--- a/lib/libv4l2/Makefile
+++ b/lib/libv4l2/Makefile
@@ -1,7 +1,7 @@
 override CPPFLAGS += -I../include -I../../../../linux/include
 
 CFLAGS := -g -O1
-CFLAGS += -Wall -W -Wno-unused -Wpointer-arith -Wstrict-prototypes
+CFLAGS += -Wall -Wno-unused -Wpointer-arith -Wstrict-prototypes -Wmissing-prototypes
 
 LIBS = -lpthread
 

diff --git a/lib/libv4l2/log.c b/lib/libv4l2/log.c
index 05f6c46..414634b 100644 (file)
--- a/lib/libv4l2/log.c
+++ b/lib/libv4l2/log.c
@@ -25,6 +25,7 @@
 #include <asm/types.h>
 /* end broken header workaround includes */
 #include <linux/videodev2.h>
+#include "libv4l2-priv.h"
 
 #define ARRAY_SIZE(x) (sizeof(x)/sizeof((x)[0]))
 
@@ -116,7 +117,7 @@ void v4l2_log_ioctl(unsigned long int request, void *arg, int result)
        int pixfmt = fmt->fmt.pix.pixelformat;
 
        if (fmt->type == V4L2_BUF_TYPE_VIDEO_CAPTURE) {
-         fprintf(v4l2_log_file, "  pixelformat: %c%c%c%c %dx%d\n",
+         fprintf(v4l2_log_file, "  pixelformat: %c%c%c%c %ux%u\n",
            pixfmt & 0xff,
            (pixfmt >> 8) & 0xff,
            (pixfmt >> 16) & 0xff,
@@ -136,7 +137,7 @@ void v4l2_log_ioctl(unsigned long int request, void *arg, int result)
        struct v4l2_requestbuffers *req = arg;
 
        fprintf(v4l2_log_file, "  count: %u type: %d memory: %d\n",
-         req->count, req->type, req->memory);
+         req->count, (int)req->type, (int)req->memory);
       }
       break;
   }

diff --git a/lib/libv4l2/v4l2convert.c b/lib/libv4l2/v4l2convert.c
index f312828..9420e68 100644 (file)
--- a/lib/libv4l2/v4l2convert.c
+++ b/lib/libv4l2/v4l2convert.c
@@ -27,6 +27,8 @@
 #include <syscall.h>
 #include <fcntl.h>
 #include <string.h>
+#include <sys/ioctl.h>
+#include <sys/mman.h>
 /* These headers are not needed by us, but by linux/videodev2.h,
    which is broken on some systems and doesn't include them itself :( */
 #include <sys/time.h>

diff --git a/lib/libv4lconvert/Makefile b/lib/libv4lconvert/Makefile
index b7aa58b..a294104 100644 (file)
--- a/lib/libv4lconvert/Makefile
+++ b/lib/libv4lconvert/Makefile
@@ -1,7 +1,7 @@
 override CPPFLAGS += -I../include -I../../../../linux/include
 
 CFLAGS := -g -O1
-CFLAGS += -Wall -W -Wno-unused -Wpointer-arith -Wstrict-prototypes
+CFLAGS += -Wall -Wno-unused -Wpointer-arith -Wstrict-prototypes -Wmissing-prototypes
 
 ifeq ($(LINKTYPE),static)
 CONVERT_LIB   = libv4lconvert.a

diff --git a/lib/libv4lconvert/libv4lconvert.c b/lib/libv4lconvert/libv4lconvert.c
index 4c9e67d..ade797c 100644 (file)
--- a/lib/libv4lconvert/libv4lconvert.c
+++ b/lib/libv4lconvert/libv4lconvert.c
@@ -275,7 +275,7 @@ int v4lconvert_convert(struct v4lconvert_data *data,
 
       if (header_width != dest_fmt->fmt.pix.width || header_height != dest_fmt->fmt.pix.height) {
        V4LCONVERT_ERR("unexpected width / height in JPEG header\n");
-       V4LCONVERT_ERR("expected: %dx%d, header: %ux%u\n",
+       V4LCONVERT_ERR("expected: %ux%u, header: %ux%u\n",
          dest_fmt->fmt.pix.width, dest_fmt->fmt.pix.height,
          header_width, header_height);
        errno = EIO;

diff --git a/lib/libv4lconvert/pac207.c b/lib/libv4lconvert/pac207.c
index 4887c25..97ef4eb 100644 (file)
--- a/lib/libv4lconvert/pac207.c
+++ b/lib/libv4lconvert/pac207.c
@@ -38,7 +38,7 @@ static struct {
        signed char val;
 } table[256];
 
-void init_pixart_decoder(void)
+static void init_pixart_decoder(void)
 {
     int i;
     int is_abs, val, len;

diff --git a/lib/libv4lconvert/rgbyuv.c b/lib/libv4lconvert/rgbyuv.c
index 742dd06..883d623 100644 (file)
--- a/lib/libv4lconvert/rgbyuv.c
+++ b/lib/libv4lconvert/rgbyuv.c
@@ -20,6 +20,8 @@
 
 */
 
+#include "libv4lconvert-priv.h"
+
 #define RGB2YUV(r,g,b,y,u,v) \
     (y) = ((  8453*(r) + 16594*(g) +  3223*(b) +  524288) >> 15); \
     (u) = (( -4878*(r) -  9578*(g) + 14456*(b) + 4210688) >> 15); \

diff --git a/lib/libv4lconvert/sn9c10x.c b/lib/libv4lconvert/sn9c10x.c
index 98a5133..4ea526d 100644 (file)
--- a/lib/libv4lconvert/sn9c10x.c
+++ b/lib/libv4lconvert/sn9c10x.c
@@ -50,7 +50,7 @@ static int sonix_unknown = 0;
        present at the MSB of byte x.
 
 */
-void sonix_decompress_init(void)
+static void sonix_decompress_init(void)
 {
        int i;
        int is_abs, val, len, unk;

diff --git a/lib/libv4lconvert/spca561-decompress.c b/lib/libv4lconvert/spca561-decompress.c
index 802345a..01eed4e 100644 (file)
--- a/lib/libv4lconvert/spca561-decompress.c
+++ b/lib/libv4lconvert/spca561-decompress.c
@@ -28,6 +28,7 @@
  *     but it might work with other spca561 cameras
  */
 #include <string.h>
+#include "libv4lconvert-priv.h"
 
 /*fixme: not reentrant */
 static unsigned int bit_bucket;

diff --git a/lib/libv4lconvert/tinyjpeg.c b/lib/libv4lconvert/tinyjpeg.c
index 5c3b4e2..9ef21d7 100644 (file)
--- a/lib/libv4lconvert/tinyjpeg.c
+++ b/lib/libv4lconvert/tinyjpeg.c
@@ -246,7 +246,7 @@ static const unsigned char val_ac_chrominance[] =
       unsigned char c; \
       if (stream >= priv->stream_end) { \
        snprintf(priv->error_string, sizeof(priv->error_string), \
-         "fill_nbits error: need %d more bits\n", \
+         "fill_nbits error: need %u more bits\n", \
          nbits_wanted - nbits_in_reservoir); \
        longjmp(priv->jump_state, -EIO); \
       } \
@@ -1676,9 +1676,9 @@ static int parse_SOS(struct jdec_private *priv, const unsigned char *stream)
        error("We do not support more than %d DC Huffman table\n",
          HUFFMAN_TABLES);
      if (cid != priv->component_infos[i].cid)
-       error("SOS cid order (%d:%d) isn't compatible with the SOF marker (%d:%d)\n",
+       error("SOS cid order (%u:%u) isn't compatible with the SOF marker (%u:%u)\n",
              i, cid, i, priv->component_infos[i].cid);
-     trace("ComponentId:%d  tableAC:%d tableDC:%d\n", cid, table&0xf, table>>4);
+     trace("ComponentId:%u  tableAC:%d tableDC:%d\n", cid, table&0xf, table>>4);
 #endif
      priv->component_infos[i].AC_table = &priv->HTAC[table&0xf];
      priv->component_infos[i].DC_table = &priv->HTDC[table>>4];