V4L/DVB (10177): Fix sparse warnings on em28xx
authorMauro Carvalho Chehab <mchehab@redhat.com>
Mon, 5 Jan 2009 04:00:40 +0000 (01:00 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Thu, 8 Jan 2009 00:18:55 +0000 (22:18 -0200)
/home/v4l/master/v4l/em28xx-core.c:396:25: warning: symbol 'outputs' was not declared. Should it be static?
/home/v4l/master/v4l/em28xx-input.c:324:6: warning: symbol 'em28xx_ir_start' was not declared. Should it be static?
/home/v4l/master/v4l/em28xx-cards.c:1925:5: warning: symbol 'em28xx_init_dev' was not declared. Should it be static?

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/video/em28xx/em28xx-cards.c
drivers/media/video/em28xx/em28xx-core.c
drivers/media/video/em28xx/em28xx-input.c

index e776699..6d2e4a6 100644 (file)
@@ -1842,7 +1842,7 @@ void em28xx_release_resources(struct em28xx *dev)
  * em28xx_init_dev()
  * allocates and inits the device structs, registers i2c bus and v4l device
  */
-int em28xx_init_dev(struct em28xx **devhandle, struct usb_device *udev,
+static int em28xx_init_dev(struct em28xx **devhandle, struct usb_device *udev,
                           int minor)
 {
        struct em28xx *dev = *devhandle;
index 819ccea..eb5fb05 100644 (file)
@@ -393,7 +393,7 @@ static int em28xx_set_audio_source(struct em28xx *dev)
        return ret;
 }
 
-struct em28xx_vol_table outputs[] = {
+static const struct em28xx_vol_table outputs[] = {
        { EM28XX_AOUT_MASTER, AC97_MASTER_VOL      },
        { EM28XX_AOUT_LINE,   AC97_LINE_LEVEL_VOL  },
        { EM28XX_AOUT_MONO,   AC97_MASTER_MONO_VOL },
index 42bbaf6..0443afe 100644 (file)
@@ -307,7 +307,7 @@ static void em28xx_ir_work(struct work_struct *work)
        mod_timer(&ir->timer, jiffies + msecs_to_jiffies(ir->polling));
 }
 
-void em28xx_ir_start(struct em28xx_IR *ir)
+static void em28xx_ir_start(struct em28xx_IR *ir)
 {
        setup_timer(&ir->timer, ir_timer, (unsigned long)ir);
        INIT_WORK(&ir->work, em28xx_ir_work);