Staging: go7007 v4l fixes
authorRoss Cohen <rcohen@snurgle.org>
Tue, 30 Sep 2008 02:36:24 +0000 (22:36 -0400)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 10 Oct 2008 22:31:11 +0000 (15:31 -0700)
Fix up some of the v4l issues that were recently changed to make the
go7007 driver a bit cleaner.

From: Ross Cohen <rcohen@snurgle.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/go7007/go7007-driver.c
drivers/staging/go7007/go7007-v4l2.c
drivers/staging/go7007/snd-go7007.c
drivers/staging/go7007/wis-ov7640.c
drivers/staging/go7007/wis-saa7113.c
drivers/staging/go7007/wis-saa7115.c
drivers/staging/go7007/wis-sony-tuner.c
drivers/staging/go7007/wis-tw2804.c
drivers/staging/go7007/wis-tw9903.c
drivers/staging/go7007/wis-uda1342.c

index 5a336ff2d9bbe5f1db3a442404a5bb04e24051ee..81ae4b0fa8902e720dc7b4a4120ae3f9c798838f 100644 (file)
@@ -31,7 +31,7 @@
 #include <linux/semaphore.h>
 #include <linux/uaccess.h>
 #include <asm/system.h>
-#include <linux/videodev.h>
+#include <linux/videodev2.h>
 #include <media/tuner.h>
 #include <media/v4l2-common.h>
 
index d54d0190d8145f0eabf4886aeefef05daf4210df..94e1141a1fcdf9e6b656531d1aa9bce2c2f36cd4 100644 (file)
@@ -26,8 +26,7 @@
 #include <linux/time.h>
 #include <linux/vmalloc.h>
 #include <linux/pagemap.h>
-#include <linux/videodev.h>
-#include <linux/video_decoder.h>
+#include <linux/videodev2.h>
 #include <media/v4l2-common.h>
 #include <media/v4l2-ioctl.h>
 #include <linux/i2c.h>
@@ -835,7 +834,6 @@ static int go7007_do_ioctl(struct inode *inode, struct file *file,
        case VIDIOC_S_STD:
        {
                v4l2_std_id *std = arg;
-               int norm;
 
                if (go->streaming)
                        return -EBUSY;
@@ -856,20 +854,17 @@ static int go7007_do_ioctl(struct inode *inode, struct file *file,
                if (*std & V4L2_STD_NTSC) {
                        go->standard = GO7007_STD_NTSC;
                        go->sensor_framerate = 30000;
-                       norm = VIDEO_MODE_NTSC;
                } else if (*std & V4L2_STD_PAL) {
                        go->standard = GO7007_STD_PAL;
                        go->sensor_framerate = 25025;
-                       norm = VIDEO_MODE_PAL;
                } else if (*std & V4L2_STD_SECAM) {
                        go->standard = GO7007_STD_PAL;
                        go->sensor_framerate = 25025;
-                       norm = VIDEO_MODE_SECAM;
                } else
                        return -EINVAL;
                if (go->i2c_adapter_online)
                        i2c_clients_command(&go->i2c_adapter,
-                                               DECODER_SET_NORM, &norm);
+                                           VIDIOC_S_STD, std);
                set_capture_size(go, NULL, 0);
                return 0;
        }
@@ -933,7 +928,7 @@ static int go7007_do_ioctl(struct inode *inode, struct file *file,
                        return -EBUSY;
                go->input = *input;
                if (go->i2c_adapter_online) {
-                       i2c_clients_command(&go->i2c_adapter, DECODER_SET_INPUT,
+                       i2c_clients_command(&go->i2c_adapter, VIDIOC_S_INPUT,
                                &go->board_info->inputs[*input].video_input);
                        i2c_clients_command(&go->i2c_adapter, VIDIOC_S_AUDIO,
                                &go->board_info->inputs[*input].audio_input);
@@ -1459,6 +1454,7 @@ static struct file_operations go7007_fops = {
 
 static struct video_device go7007_template = {
        .name           = "go7007",
+       .vfl_type       = VID_TYPE_CAPTURE,
        .fops           = &go7007_fops,
        .minor          = -1,
        .release        = go7007_vfl_release,
index f5cac08b07ea66bb6ec9daa3b49858f56d2b739c..382740c405ffe788704ca1aa5e95a33bcdd47ed3 100644 (file)
@@ -44,8 +44,8 @@ module_param_array(index, int, NULL, 0444);
 module_param_array(id, charp, NULL, 0444);
 module_param_array(enable, bool, NULL, 0444);
 MODULE_PARM_DESC(index, "Index value for the go7007 audio driver");
-MODULE_PARM_DESC(index, "ID string for the go7007 audio driver");
-MODULE_PARM_DESC(index, "Enable for the go7007 audio driver");
+MODULE_PARM_DESC(id, "ID string for the go7007 audio driver");
+MODULE_PARM_DESC(enable, "Enable for the go7007 audio driver");
 
 struct go7007_snd {
        struct snd_card *card;
index 815615a623069e67f60d364db2283084c771270c..f5f11e927af3b3bf9be168c1b50daa1bcd37b9cd 100644 (file)
@@ -19,8 +19,7 @@
 #include <linux/init.h>
 #include <linux/version.h>
 #include <linux/i2c.h>
-#include <linux/videodev.h>
-#include <linux/video_decoder.h>
+#include <linux/videodev2.h>
 
 #include "wis-i2c.h"
 
index 4b14ca88b1a9df79747094ce253b4c42d1ccb172..c1aff1b923a0f6e82a194876c7a1f187af3fbbaa 100644 (file)
@@ -19,8 +19,7 @@
 #include <linux/init.h>
 #include <linux/version.h>
 #include <linux/i2c.h>
-#include <linux/videodev.h>
-#include <linux/video_decoder.h>
+#include <linux/videodev2.h>
 #include <linux/ioctl.h>
 
 #include "wis-i2c.h"
@@ -124,7 +123,7 @@ static int wis_saa7113_command(struct i2c_client *client,
        struct wis_saa7113 *dec = i2c_get_clientdata(client);
 
        switch (cmd) {
-       case DECODER_SET_INPUT:
+       case VIDIOC_S_INPUT:
        {
                int *input = arg;
 
@@ -133,23 +132,19 @@ static int wis_saa7113_command(struct i2c_client *client,
                                *input < 6 ? 0x40 : 0x80);
                break;
        }
-       case DECODER_SET_NORM:
+       case VIDIOC_S_STD:
        {
-               int *input = arg;
+               v4l2_std_id *input = arg;
                dec->norm = *input;
-               switch (dec->norm) {
-               case VIDEO_MODE_PAL:
-                       write_reg(client, 0x0e, 0x01);
-                       write_reg(client, 0x10, 0x48);
-                       break;
-               case VIDEO_MODE_NTSC:
+               if (dec->norm & V4L2_STD_NTSC) {
                        write_reg(client, 0x0e, 0x01);
                        write_reg(client, 0x10, 0x40);
-                       break;
-               case VIDEO_MODE_SECAM:
+               } else if (dec->norm & V4L2_STD_PAL) {
+                       write_reg(client, 0x0e, 0x01);
+                       write_reg(client, 0x10, 0x48);
+               } else if (dec->norm * V4L2_STD_SECAM) {
                        write_reg(client, 0x0e, 0x50);
                        write_reg(client, 0x10, 0x48);
-                       break;
                }
                break;
        }
@@ -295,7 +290,7 @@ static int wis_saa7113_detect(struct i2c_adapter *adapter, int addr, int kind)
                kfree(client);
                return -ENOMEM;
        }
-       dec->norm = VIDEO_MODE_NTSC;
+       dec->norm = V4L2_STD_NTSC;
        dec->brightness = 128;
        dec->contrast = 71;
        dec->saturation = 64;
index bd40bf4be282c1df11007bfe2aa7f2d58ca413ce..5c94c883b312041c566d6696b79277bd3d9c443e 100644 (file)
@@ -19,8 +19,7 @@
 #include <linux/init.h>
 #include <linux/version.h>
 #include <linux/i2c.h>
-#include <linux/videodev.h>
-#include <linux/video_decoder.h>
+#include <linux/videodev2.h>
 #include <linux/ioctl.h>
 
 #include "wis-i2c.h"
@@ -204,7 +203,7 @@ static int wis_saa7115_command(struct i2c_client *client,
        struct wis_saa7115 *dec = i2c_get_clientdata(client);
 
        switch (cmd) {
-       case DECODER_SET_INPUT:
+       case VIDIOC_S_INPUT:
        {
                int *input = arg;
 
@@ -222,7 +221,7 @@ static int wis_saa7115_command(struct i2c_client *client,
                int h_scaling_increment = (704 / h_integer_scaler) *
                                        1024 / res->width;
                /* Fine-grained scaler only */
-               int v_scaling_increment = (dec->norm == VIDEO_MODE_NTSC ?
+               int v_scaling_increment = (dec->norm & V4L2_STD_NTSC ?
                                240 : 288) * 1024 / res->height;
                u8 regs[] = {
                        0x88,   0xc0,
@@ -262,20 +261,20 @@ static int wis_saa7115_command(struct i2c_client *client,
                write_regs(client, regs);
                break;
        }
-       case DECODER_SET_NORM:
+       case VIDIOC_S_STD:
        {
-               int *input = arg;
+               v4l2_std_id *input = arg;
                u8 regs[] = {
                        0x88,   0xc0,
-                       0x98,   *input == VIDEO_MODE_NTSC ? 0x12 : 0x16,
-                       0x9a,   *input == VIDEO_MODE_NTSC ? 0xf2 : 0x20,
-                       0x9b,   *input == VIDEO_MODE_NTSC ? 0x00 : 0x01,
-                       0xc8,   *input == VIDEO_MODE_NTSC ? 0x12 : 0x16,
-                       0xca,   *input == VIDEO_MODE_NTSC ? 0xf2 : 0x20,
-                       0xcb,   *input == VIDEO_MODE_NTSC ? 0x00 : 0x01,
+                       0x98,   *input & V4L2_STD_NTSC ? 0x12 : 0x16,
+                       0x9a,   *input & V4L2_STD_NTSC ? 0xf2 : 0x20,
+                       0x9b,   *input & V4L2_STD_NTSC ? 0x00 : 0x01,
+                       0xc8,   *input & V4L2_STD_NTSC ? 0x12 : 0x16,
+                       0xca,   *input & V4L2_STD_NTSC ? 0xf2 : 0x20,
+                       0xcb,   *input & V4L2_STD_NTSC ? 0x00 : 0x01,
                        0x88,   0xf0,
-                       0x30,   *input == VIDEO_MODE_NTSC ? 0x66 : 0x00,
-                       0x31,   *input == VIDEO_MODE_NTSC ? 0x90 : 0xe0,
+                       0x30,   *input & V4L2_STD_NTSC ? 0x66 : 0x00,
+                       0x31,   *input & V4L2_STD_NTSC ? 0x90 : 0xe0,
                        0,      0,
                };
                write_regs(client, regs);
@@ -424,7 +423,7 @@ static int wis_saa7115_detect(struct i2c_adapter *adapter, int addr, int kind)
                kfree(client);
                return -ENOMEM;
        }
-       dec->norm = VIDEO_MODE_NTSC;
+       dec->norm = V4L2_STD_NTSC;
        dec->brightness = 128;
        dec->contrast = 64;
        dec->saturation = 64;
index 82e66d65f36c96ca5fb681c4af7ed48b128c954f..5997fb4794596c237bcf9b203fc7550306516f18 100644 (file)
 #include <linux/init.h>
 #include <linux/version.h>
 #include <linux/i2c.h>
-#include <linux/videodev.h>
+#include <linux/videodev2.h>
 #include <media/tuner.h>
 #include <media/v4l2-common.h>
+#include <media/v4l2-ioctl.h>
 
 #include "wis-i2c.h"
 
index 69ed7bf032277dda2b68d01c7ced238604e36762..27fe4d0d4ed695e933f208e1f14c797e9976916f 100644 (file)
@@ -19,8 +19,7 @@
 #include <linux/init.h>
 #include <linux/version.h>
 #include <linux/i2c.h>
-#include <linux/videodev.h>
-#include <linux/video_decoder.h>
+#include <linux/videodev2.h>
 #include <linux/ioctl.h>
 
 #include "wis-i2c.h"
@@ -159,20 +158,20 @@ static int wis_tw2804_command(struct i2c_client *client,
        }
 
        switch (cmd) {
-       case DECODER_SET_NORM:
+       case VIDIOC_S_STD:
        {
-               int *input = arg;
+               v4l2_std_id *input = arg;
                u8 regs[] = {
-                       0x01, *input == VIDEO_MODE_NTSC ? 0xc4 : 0x84,
-                       0x09, *input == VIDEO_MODE_NTSC ? 0x07 : 0x04,
-                       0x0a, *input == VIDEO_MODE_NTSC ? 0xf0 : 0x20,
-                       0x0b, *input == VIDEO_MODE_NTSC ? 0x07 : 0x04,
-                       0x0c, *input == VIDEO_MODE_NTSC ? 0xf0 : 0x20,
-                       0x0d, *input == VIDEO_MODE_NTSC ? 0x40 : 0x4a,
-                       0x16, *input == VIDEO_MODE_NTSC ? 0x00 : 0x40,
-                       0x17, *input == VIDEO_MODE_NTSC ? 0x00 : 0x40,
-                       0x20, *input == VIDEO_MODE_NTSC ? 0x07 : 0x0f,
-                       0x21, *input == VIDEO_MODE_NTSC ? 0x07 : 0x0f,
+                       0x01, *input & V4L2_STD_NTSC ? 0xc4 : 0x84,
+                       0x09, *input & V4L2_STD_NTSC ? 0x07 : 0x04,
+                       0x0a, *input & V4L2_STD_NTSC ? 0xf0 : 0x20,
+                       0x0b, *input & V4L2_STD_NTSC ? 0x07 : 0x04,
+                       0x0c, *input & V4L2_STD_NTSC ? 0xf0 : 0x20,
+                       0x0d, *input & V4L2_STD_NTSC ? 0x40 : 0x4a,
+                       0x16, *input & V4L2_STD_NTSC ? 0x00 : 0x40,
+                       0x17, *input & V4L2_STD_NTSC ? 0x00 : 0x40,
+                       0x20, *input & V4L2_STD_NTSC ? 0x07 : 0x0f,
+                       0x21, *input & V4L2_STD_NTSC ? 0x07 : 0x0f,
                        0xff,   0xff,
                };
                write_regs(client, regs, dec->channel);
@@ -322,7 +321,7 @@ static int wis_tw2804_detect(struct i2c_adapter *adapter, int addr, int kind)
                return -ENOMEM;
        }
        dec->channel = -1;
-       dec->norm = VIDEO_MODE_NTSC;
+       dec->norm = V4L2_STD_NTSC;
        dec->brightness = 128;
        dec->contrast = 128;
        dec->saturation = 128;
index 1cdf01a8b338aa032eaa108ea45ed281e401d1b7..d8e41968022e92a057214d71dd79b0442b023f6d 100644 (file)
@@ -19,8 +19,7 @@
 #include <linux/init.h>
 #include <linux/version.h>
 #include <linux/i2c.h>
-#include <linux/videodev.h>
-#include <linux/video_decoder.h>
+#include <linux/videodev2.h>
 #include <linux/ioctl.h>
 
 #include "wis-i2c.h"
@@ -106,7 +105,7 @@ static int wis_tw9903_command(struct i2c_client *client,
        struct wis_tw9903 *dec = i2c_get_clientdata(client);
 
        switch (cmd) {
-       case DECODER_SET_INPUT:
+       case VIDIOC_S_INPUT:
        {
                int *input = arg;
 
@@ -119,7 +118,7 @@ static int wis_tw9903_command(struct i2c_client *client,
                struct video_decoder_resolution *res = arg;
                /*int hscale = 256 * 720 / res->width;*/
                int hscale = 256 * 720 / (res->width - (res->width > 704 ? 0 : 8));
-               int vscale = 256 * (dec->norm == VIDEO_MODE_NTSC ?  240 : 288)
+               int vscale = 256 * (dec->norm & V4L2_STD_NTSC ?  240 : 288)
                                / res->height;
                u8 regs[] = {
                        0x0d, vscale & 0xff,
@@ -134,14 +133,14 @@ static int wis_tw9903_command(struct i2c_client *client,
                break;
        }
 #endif
-       case DECODER_SET_NORM:
+       case VIDIOC_S_STD:
        {
-               int *input = arg;
+               v4l2_std_id *input = arg;
                u8 regs[] = {
-                       0x05, *input == VIDEO_MODE_NTSC ? 0x80 : 0x00,
-                       0x07, *input == VIDEO_MODE_NTSC ? 0x02 : 0x12,
-                       0x08, *input == VIDEO_MODE_NTSC ? 0x14 : 0x18,
-                       0x09, *input == VIDEO_MODE_NTSC ? 0xf0 : 0x20,
+                       0x05, *input & V4L2_STD_NTSC ? 0x80 : 0x00,
+                       0x07, *input & V4L2_STD_NTSC ? 0x02 : 0x12,
+                       0x08, *input & V4L2_STD_NTSC ? 0x14 : 0x18,
+                       0x09, *input & V4L2_STD_NTSC ? 0xf0 : 0x20,
                        0,      0,
                };
                write_regs(client, regs);
@@ -297,7 +296,7 @@ static int wis_tw9903_detect(struct i2c_adapter *adapter, int addr, int kind)
                kfree(client);
                return -ENOMEM;
        }
-       dec->norm = VIDEO_MODE_NTSC;
+       dec->norm = V4L2_STD_NTSC;
        dec->brightness = 0;
        dec->contrast = 0x60;
        dec->hue = 0;
index 28c10bf3a47bfd73a2a91226647e3fae2494c3fd..a0894e3cb8c725de45b09dcece074c31f1b56d73 100644 (file)
@@ -19,7 +19,7 @@
 #include <linux/init.h>
 #include <linux/version.h>
 #include <linux/i2c.h>
-#include <linux/videodev.h>
+#include <linux/videodev2.h>
 #include <media/tvaudio.h>
 #include <media/v4l2-common.h>