V4L/DVB (10039): m5602 - ov9650: Add CIF mode

Signed-off-by: Erik Andrén <erik.andren@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

2 files changed, 51 insertions, 1 deletions

diff --git a/drivers/media/video/gspca/m5602/m5602_ov9650.c b/drivers/media/video/gspca/m5602/m5602_ov9650.c
index 98c979dbbe3..2abb626c1c7 100644
--- a/drivers/media/video/gspca/m5602/m5602_ov9650.c
+++ b/drivers/media/video/gspca/m5602/m5602_ov9650.c
@@ -163,6 +163,19 @@ int ov9650_start(struct sd *sd)
                 }
                 break;
 
+        case 352:
+                PDEBUG(D_V4L2, "Configuring camera for CIF mode");
+
+                for (i = 0; i < ARRAY_SIZE(CIF_ov9650) && !err; i++) {
+                        u8 data = CIF_ov9650[i][2];
+                        if (CIF_ov9650[i][0] == SENSOR)
+                                err = m5602_write_sensor(sd,
+                                        CIF_ov9650[i][1], &data, 1);
+                        else
+                                err = m5602_write_bridge(sd, CIF_ov9650[i][1], data);
+                }
+                break;
+
         case 320:
                 PDEBUG(D_V4L2, "Configuring camera for QVGA mode");
 
diff --git a/drivers/media/video/gspca/m5602/m5602_ov9650.h b/drivers/media/video/gspca/m5602/m5602_ov9650.h
index b577017f6e5..9e955691aa6 100644
--- a/drivers/media/video/gspca/m5602/m5602_ov9650.h
+++ b/drivers/media/video/gspca/m5602/m5602_ov9650.h
@@ -93,6 +93,7 @@
 
 #define OV9650_REGISTER_RESET           (1 << 7)
 #define OV9650_VGA_SELECT               (1 << 6)
+#define OV9650_CIF_SELECT               (1 << 5)
 #define OV9650_QVGA_SELECT              (1 << 4)
 #define OV9650_RGB_SELECT               (1 << 2)
 #define OV9650_RAW_RGB_SELECT           (1 << 0)
@@ -258,7 +259,7 @@ static struct m5602_sensor ov9650 = {
         }
         },
 
-        .nmodes = 2,
+        .nmodes = 3,
         .modes = {
         {
                 320,
@@ -271,6 +272,16 @@ static struct m5602_sensor ov9650 = {
                 .colorspace = V4L2_COLORSPACE_SRGB,
                 .priv = 0
         }, {
+                352,
+                288,
+                V4L2_PIX_FMT_SBGGR8,
+                V4L2_FIELD_NONE,
+                .sizeimage =
+                        352 * 288,
+                .bytesperline = 352,
+                .colorspace = V4L2_COLORSPACE_SRGB,
+                .priv = 0
+        }, {
                 640,
                 480,
                 V4L2_PIX_FMT_SBGGR8,
@@ -469,6 +480,32 @@ static const unsigned char VGA_ov9650[][3] =
         {BRIDGE, M5602_XB_HSYNC_PARA, 0xe2}
 };
 
+static const unsigned char CIF_ov9650[][3] =
+{
+        {SENSOR, OV9650_COM7, OV9650_CIF_SELECT |
+                              OV9650_RGB_SELECT |
+                              OV9650_RAW_RGB_SELECT},
+
+        {BRIDGE, M5602_XB_LINE_OF_FRAME_H, 0x82},
+        {BRIDGE, M5602_XB_LINE_OF_FRAME_L, 0x00},
+        {BRIDGE, M5602_XB_PIX_OF_LINE_H, 0x82},
+        {BRIDGE, M5602_XB_PIX_OF_LINE_L, 0x00},
+        {BRIDGE, M5602_XB_SIG_INI, 0x01},
+
+        /* Moves the view window in a vertical orientation */
+        {BRIDGE, M5602_XB_VSYNC_PARA, 0x00},
+        {BRIDGE, M5602_XB_VSYNC_PARA, 0x09},
+        {BRIDGE, M5602_XB_VSYNC_PARA, 0x00},
+        {BRIDGE, M5602_XB_VSYNC_PARA, 0x01},
+        {BRIDGE, M5602_XB_VSYNC_PARA, 0x20}, /* 288 */
+        {BRIDGE, M5602_XB_VSYNC_PARA, 0x00},
+        {BRIDGE, M5602_XB_VSYNC_PARA, 0x00},
+        {BRIDGE, M5602_XB_HSYNC_PARA, 0x00},
+        {BRIDGE, M5602_XB_HSYNC_PARA, 0x62}, /* 98 */
+        {BRIDGE, M5602_XB_HSYNC_PARA, 0x01}, /* 352 + 98 */
+        {BRIDGE, M5602_XB_HSYNC_PARA, 0xc2}
+};
+
 static const unsigned char QVGA_ov9650[][3] =
 {
         {SENSOR, OV9650_COM7, OV9650_QVGA_SELECT |