/**
 * OV519 driver
 *
 * Copyright (C) 2008-2011 Jean-François Moine <moinejf@free.fr>
 * Copyright (C) 2009 Hans de Goede <hdegoede@redhat.com>
 *
 * This module is adapted from the ov51x-jpeg package, which itself
 * was adapted from the ov511 driver.
 *
 * Original copyright for the ov511 driver is:
 *
 * Copyright (c) 1999-2006 Mark W. McClelland
 * Support for OV519, OV8610 Copyright (c) 2003 Joerg Heckenbach
 * Many improvements by Bret Wallach <bwallac1@san.rr.com>
 * Color fixes by by Orion Sky Lawlor <olawlor@acm.org> (2/26/2000)
 * OV7620 fixes by Charl P. Botha <cpbotha@ieee.org>
 * Changes by Claudio Matsuoka <claudio@conectiva.com>
 *
 * ov51x-jpeg original copyright is:
 *
 * Copyright (c) 2004-2007 Romain Beauxis <toots@rastageeks.org>
 * Support for OV7670 sensors was contributed by Sam Skipsey <aoanla@yahoo.com>
 *
 * This program is free software; you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by
 * the Free Software Foundation; either version 2 of the License, or
 * any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with this program; if not, write to the Free Software
 * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
 *
 */

#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt

#define MODULE_NAME "ov519"

#include <linux/input.h>
#include "gspca.h"

/* The jpeg_hdr is used by w996Xcf only */
/* The CONEX_CAM define for jpeg.h needs renaming, now its used here too */
#define CONEX_CAM
#include "jpeg.h"

MODULE_AUTHOR("Jean-Francois Moine <http://moinejf.free.fr>");
MODULE_DESCRIPTION("OV519 USB Camera Driver");
MODULE_LICENSE("GPL");

/* global parameters */
static int frame_rate;

/* Number of times to retry a failed I2C transaction. Increase this if you
 * are getting "Failed to read sensor ID..." */
static int i2c_detect_tries = 10;

/* controls */
enum e_ctrl {
        BRIGHTNESS,
        CONTRAST,
        EXPOSURE,
        COLORS,
        HFLIP,
        VFLIP,
        AUTOBRIGHT,
        AUTOGAIN,
        FREQ,
        NCTRL           /* number of controls */
};

/* ov519 device descriptor */
struct sd {
        struct gspca_dev gspca_dev;             /* !! must be the first item */

        struct gspca_ctrl ctrls[NCTRL];

        u8 packet_nr;

        char bridge;
#define BRIDGE_OV511            0
#define BRIDGE_OV511PLUS        1
#define BRIDGE_OV518            2
#define BRIDGE_OV518PLUS        3
#define BRIDGE_OV519            4               /* = ov530 */
#define BRIDGE_OVFX2            5
#define BRIDGE_W9968CF          6
#define BRIDGE_MASK             7

        char invert_led;
#define BRIDGE_INVERT_LED       8

        char snapshot_pressed;
        char snapshot_needs_reset;

        /* Determined by sensor type */
        u8 sif;

        u8 quality;
#define QUALITY_MIN 50
#define QUALITY_MAX 70
#define QUALITY_DEF 50

        u8 stopped;             /* Streaming is temporarily paused */
        u8 first_frame;

        u8 frame_rate;          /* current Framerate */
        u8 clockdiv;            /* clockdiv override */

        s8 sensor;              /* Type of image sensor chip (SEN_*) */

        u8 sensor_addr;
        u16 sensor_width;
        u16 sensor_height;
        s16 sensor_reg_cache[256];

        u8 jpeg_hdr[JPEG_HDR_SZ];
};
enum sensors {
        SEN_OV2610,
        SEN_OV2610AE,
        SEN_OV3610,
        SEN_OV6620,
        SEN_OV6630,
        SEN_OV66308AF,
        SEN_OV7610,
        SEN_OV7620,
        SEN_OV7620AE,
        SEN_OV7640,
        SEN_OV7648,
        SEN_OV7660,
        SEN_OV7670,
        SEN_OV76BE,
        SEN_OV8610,
        SEN_OV9600,
};

/* Note this is a bit of a hack, but the w9968cf driver needs the code for all
   the ov sensors which is already present here. When we have the time we
   really should move the sensor drivers to v4l2 sub drivers. */
#include "w996Xcf.c"

/* V4L2 controls supported by the driver */
static void setbrightness(struct gspca_dev *gspca_dev);
static void setcontrast(struct gspca_dev *gspca_dev);
static void setexposure(struct gspca_dev *gspca_dev);
static void setcolors(struct gspca_dev *gspca_dev);
static void sethvflip(struct gspca_dev *gspca_dev);
static void setautobright(struct gspca_dev *gspca_dev);
static int sd_setautogain(struct gspca_dev *gspca_dev, __s32 val);
static void setfreq(struct gspca_dev *gspca_dev);
static void setfreq_i(struct sd *sd);

static const struct ctrl sd_ctrls[] = {
[BRIGHTNESS] = {
            {
                .id      = V4L2_CID_BRIGHTNESS,
                .type    = V4L2_CTRL_TYPE_INTEGER,
                .name    = "Brightness",
                .minimum = 0,
                .maximum = 255,
                .step    = 1,
                .default_value = 127,
            },
            .set_control = setbrightness,
        },
[CONTRAST] = {
            {
                .id      = V4L2_CID_CONTRAST,
                .type    = V4L2_CTRL_TYPE_INTEGER,
                .name    = "Contrast",
                .minimum = 0,
                .maximum = 255,
                .step    = 1,
                .default_value = 127,
            },
            .set_control = setcontrast,
        },
[EXPOSURE] = {
            {
                .id      = V4L2_CID_EXPOSURE,
                .type    = V4L2_CTRL_TYPE_INTEGER,
                .name    = "Exposure",
                .minimum = 0,
                .maximum = 255,
                .step    = 1,
                .default_value = 127,
            },
            .set_control = setexposure,
        },
[COLORS] = {
            {
                .id      = V4L2_CID_SATURATION,
                .type    = V4L2_CTRL_TYPE_INTEGER,
                .name    = "Color",
                .minimum = 0,
                .maximum = 255,
                .step    = 1,
                .default_value = 127,
            },
            .set_control = setcolors,
        },
/* The flip controls work for sensors ov7660 and ov7670 only */
[HFLIP] = {
            {
                .id      = V4L2_CID_HFLIP,
                .type    = V4L2_CTRL_TYPE_BOOLEAN,
                .name    = "Mirror",
                .minimum = 0,
                .maximum = 1,
                .step    = 1,
                .default_value = 0,
            },
            .set_control = sethvflip,
        },
[VFLIP] = {
            {
                .id      = V4L2_CID_VFLIP,
                .type    = V4L2_CTRL_TYPE_BOOLEAN,
                .name    = "Vflip",
                .minimum = 0,
                .maximum = 1,
                .step    = 1,
                .default_value = 0,
            },
            .set_control = sethvflip,
        },
[AUTOBRIGHT] = {
            {
                .id      = V4L2_CID_AUTOBRIGHTNESS,
                .type    = V4L2_CTRL_TYPE_BOOLEAN,
                .name    = "Auto Brightness",
                .minimum = 0,
                .maximum = 1,
                .step    = 1,
                .default_value = 1,
            },
            .set_control = setautobright,
        },
[AUTOGAIN] = {
            {
                .id      = V4L2_CID_AUTOGAIN,
                .type    = V4L2_CTRL_TYPE_BOOLEAN,
                .name    = "Auto Gain",
                .minimum = 0,
                .maximum = 1,
                .step    = 1,
                .default_value = 1,
                .flags   = V4L2_CTRL_FLAG_UPDATE
            },
            .set = sd_setautogain,
        },
[FREQ] = {
            {
                .id      = V4L2_CID_POWER_LINE_FREQUENCY,
                .type    = V4L2_CTRL_TYPE_MENU,
                .name    = "Light frequency filter",
                .minimum = 0,
                .maximum = 2,   /* 0: no flicker, 1: 50Hz, 2:60Hz, 3: auto */
                .step    = 1,
                .default_value = 0,
            },
            .set_control = setfreq,
        },
};

/* table of the disabled controls */
static const unsigned ctrl_dis[] = {
[SEN_OV2610] =          ((1 << NCTRL) - 1)      /* no control */
                        ^ ((1 << EXPOSURE)      /* but exposure */
                         | (1 << AUTOGAIN)),    /* and autogain */

[SEN_OV2610AE] =        ((1 << NCTRL) - 1)      /* no control */
                        ^ ((1 << EXPOSURE)      /* but exposure */
                         | (1 << AUTOGAIN)),    /* and autogain */

[SEN_OV3610] =          (1 << NCTRL) - 1,       /* no control */

[SEN_OV6620] =          (1 << HFLIP) |
                        (1 << VFLIP) |
                        (1 << EXPOSURE) |
                        (1 << AUTOGAIN),

[SEN_OV6630] =          (1 << HFLIP) |
                        (1 << VFLIP) |
                        (1 << EXPOSURE) |
                        (1 << AUTOGAIN),

[SEN_OV66308AF] =       (1 << HFLIP) |
                        (1 << VFLIP) |
                        (1 << EXPOSURE) |
                        (1 << AUTOGAIN),

[SEN_OV7610] =          (1 << HFLIP) |
                        (1 << VFLIP) |
                        (1 << EXPOSURE) |
                        (1 << AUTOGAIN),

[SEN_OV7620] =          (1 << HFLIP) |
                        (1 << VFLIP) |
                        (1 << EXPOSURE) |
                        (1 << AUTOGAIN),

[SEN_OV7620AE] =        (1 << HFLIP) |
                        (1 << VFLIP) |
                        (1 << EXPOSURE) |
                        (1 << AUTOGAIN),

[SEN_OV7640] =          (1 << HFLIP) |
                        (1 << VFLIP) |
                        (1 << AUTOBRIGHT) |
                        (1 << CONTRAST) |
                        (1 << EXPOSURE) |
                        (1 << AUTOGAIN),

[SEN_OV7648] =          (1 << HFLIP) |
                        (1 << VFLIP) |
                        (1 << AUTOBRIGHT) |
                        (1 << CONTRAST) |
                        (1 << EXPOSURE) |
                        (1 << AUTOGAIN),

[SEN_OV7660] =          (1 << AUTOBRIGHT) |
                        (1 << EXPOSURE) |
                        (1 << AUTOGAIN),

[SEN_OV7670] =          (1 << COLORS) |
                        (1 << AUTOBRIGHT) |
                        (1 << EXPOSURE) |
                        (1 << AUTOGAIN),

[SEN_OV76BE] =          (1 << HFLIP) |
                        (1 << VFLIP) |
                        (1 << EXPOSURE) |
                        (1 << AUTOGAIN),

[SEN_OV8610] =          (1 << HFLIP) |
                        (1 << VFLIP) |
                        (1 << EXPOSURE) |
                        (1 << AUTOGAIN) |
                        (1 << FREQ),
[SEN_OV9600] =          ((1 << NCTRL) - 1)      /* no control */
                        ^ ((1 << EXPOSURE)      /* but exposure */
                         | (1 << AUTOGAIN)),    /* and autogain */

};

static const struct v4l2_pix_format ov519_vga_mode[] = {
        {320, 240, V4L2_PIX_FMT_JPEG, V4L2_FIELD_NONE,
                .bytesperline = 320,
                .sizeimage = 320 * 240 * 3 / 8 + 590,
                .colorspace = V4L2_COLORSPACE_JPEG,
                .priv = 1},
        {640, 480, V4L2_PIX_FMT_JPEG, V4L2_FIELD_NONE,
                .bytesperline = 640,
                .sizeimage = 640 * 480 * 3 / 8 + 590,
                .colorspace = V4L2_COLORSPACE_JPEG,
                .priv = 0},
};
static const struct v4l2_pix_format ov519_sif_mode[] = {
        {160, 120, V4L2_PIX_FMT_JPEG, V4L2_FIELD_NONE,
                .bytesperline = 160,
                .sizeimage = 160 * 120 * 3 / 8 + 590,
                .colorspace = V4L2_COLORSPACE_JPEG,
                .priv = 3},
        {176, 144, V4L2_PIX_FMT_JPEG, V4L2_FIELD_NONE,
                .bytesperline = 176,
                .sizeimage = 176 * 144 * 3 / 8 + 590,
                .colorspace = V4L2_COLORSPACE_JPEG,
                .priv = 1},
        {320, 240, V4L2_PIX_FMT_JPEG, V4L2_FIELD_NONE,
                .bytesperline = 320,
                .sizeimage = 320 * 240 * 3 / 8 + 590,
                .colorspace = V4L2_COLORSPACE_JPEG,
                .priv = 2},
        {352, 288, V4L2_PIX_FMT_JPEG, V4L2_FIELD_NONE,
                .bytesperline = 352,
                .sizeimage = 352 * 288 * 3 / 8 + 590,
                .colorspace = V4L2_COLORSPACE_JPEG,
                .priv = 0},
};

/* Note some of the sizeimage values for the ov511 / ov518 may seem
   larger then necessary, however they need to be this big as the ov511 /
   ov518 always fills the entire isoc frame, using 0 padding bytes when
   it doesn't have any data. So with low framerates the amount of data
   transferred can become quite large (libv4l will remove all the 0 padding
   in userspace). */
static const struct v4l2_pix_format ov518_vga_mode[] = {
        {320, 240, V4L2_PIX_FMT_OV518, V4L2_FIELD_NONE,
                .bytesperline = 320,
                .sizeimage = 320 * 240 * 3,
                .colorspace = V4L2_COLORSPACE_JPEG,
                .priv = 1},
        {640, 480, V4L2_PIX_FMT_OV518, V4L2_FIELD_NONE,
                .bytesperline = 640,
                .sizeimage = 640 * 480 * 2,
                .colorspace = V4L2_COLORSPACE_JPEG,
                .priv = 0},
};
static const struct v4l2_pix_format ov518_sif_mode[] = {
        {160, 120, V4L2_PIX_FMT_OV518, V4L2_FIELD_NONE,
                .bytesperline = 160,
                .sizeimage = 70000,
                .colorspace = V4L2_COLORSPACE_JPEG,
                .priv = 3},
        {176, 144, V4L2_PIX_FMT_OV518, V4L2_FIELD_NONE,
                .bytesperline = 176,
                .sizeimage = 70000,
                .colorspace = V4L2_COLORSPACE_JPEG,
                .priv = 1},
        {320, 240, V4L2_PIX_FMT_OV518, V4L2_FIELD_NONE,
                .bytesperline = 320,
                .sizeimage = 320 * 240 * 3,
                .colorspace = V4L2_COLORSPACE_JPEG,
                .priv = 2},
        {352, 288, V4L2_PIX_FMT_OV518, V4L2_FIELD_NONE,
                .bytesperline = 352,
                .sizeimage = 352 * 288 * 3,
                .colorspace = V4L2_COLORSPACE_JPEG,
                .priv = 0},
};

static const struct v4l2_pix_format ov511_vga_mode[] = {
        {320, 240, V4L2_PIX_FMT_OV511, V4L2_FIELD_NONE,
                .bytesperline = 320,
                .sizeimage = 320 * 240 * 3,
                .colorspace = V4L2_COLORSPACE_JPEG,
                .priv = 1},
        {640, 480, V4L2_PIX_FMT_OV511, V4L2_FIELD_NONE,
                .bytesperline = 640,
                .sizeimage = 640 * 480 * 2,
                .colorspace = V4L2_COLORSPACE_JPEG,
                .priv = 0},
};
static const struct v4l2_pix_format ov511_sif_mode[] = {
        {160, 120, V4L2_PIX_FMT_OV511, V4L2_FIELD_NONE,
                .bytesperline = 160,
                .sizeimage = 70000,
                .colorspace = V4L2_COLORSPACE_JPEG,
                .priv = 3},
        {176, 144, V4L2_PIX_FMT_OV511, V4L2_FIELD_NONE,
                .bytesperline = 176,
                .sizeimage = 70000,
                .colorspace = V4L2_COLORSPACE_JPEG,
                .priv = 1},
        {320, 240, V4L2_PIX_FMT_OV511, V4L2_FIELD_NONE,
                .bytesperline = 320,
                .sizeimage = 320 * 240 * 3,
                .colorspace = V4L2_COLORSPACE_JPEG,
                .priv = 2},
        {352, 288, V4L2_PIX_FMT_OV511, V4L2_FIELD_NONE,
                .bytesperline = 352,
                .sizeimage = 352 * 288 * 3,
                .colorspace = V4L2_COLORSPACE_JPEG,
                .priv = 0},
};

static const struct v4l2_pix_format ovfx2_vga_mode[] = {
        {320, 240, V4L2_PIX_FMT_SBGGR8, V4L2_FIELD_NONE,
                .bytesperline = 320,
                .sizeimage = 320 * 240,
                .colorspace = V4L2_COLORSPACE_SRGB,
                .priv = 1},
        {640, 480, V4L2_PIX_FMT_SBGGR8, V4L2_FIELD_NONE,
                .bytesperline = 640,
                .sizeimage = 640 * 480,
                .colorspace = V4L2_COLORSPACE_SRGB,
                .priv = 0},
};
static const struct v4l2_pix_format ovfx2_cif_mode[] = {
        {160, 120, V4L2_PIX_FMT_SBGGR8, V4L2_FIELD_NONE,
                .bytesperline = 160,
                .sizeimage = 160 * 120,
                .colorspace = V4L2_COLORSPACE_SRGB,
                .priv = 3},
        {176, 144, V4L2_PIX_FMT_SBGGR8, V4L2_FIELD_NONE,
                .bytesperline = 176,
                .sizeimage = 176 * 144,
                .colorspace = V4L2_COLORSPACE_SRGB,
                .priv = 1},
        {320, 240, V4L2_PIX_FMT_SBGGR8, V4L2_FIELD_NONE,
                .bytesperline = 320,
                .sizeimage = 320 * 240,
                .colorspace = V4L2_COLORSPACE_SRGB,
                .priv = 2},
        {352, 288, V4L2_PIX_FMT_SBGGR8, V4L2_FIELD_NONE,
                .bytesperline = 352,
                .sizeimage = 352 * 288,
                .colorspace = V4L2_COLORSPACE_SRGB,
                .priv = 0},
};
static const struct v4l2_pix_format ovfx2_ov2610_mode[] = {
        {800, 600, V4L2_PIX_FMT_SBGGR8, V4L2_FIELD_NONE,
                .bytesperline = 800,
                .sizeimage = 800 * 600,
                .colorspace = V4L2_COLORSPACE_SRGB,
                .priv = 1},
        {1600, 1200, V4L2_PIX_FMT_SBGGR8, V4L2_FIELD_NONE,
                .bytesperline = 1600,
                .sizeimage = 1600 * 1200,
                .colorspace = V4L2_COLORSPACE_SRGB},
};
static const struct v4l2_pix_format ovfx2_ov3610_mode[] = {
        {640, 480, V4L2_PIX_FMT_SBGGR8, V4L2_FIELD_NONE,
                .bytesperline = 640,
                .sizeimage = 640 * 480,
                .colorspace = V4L2_COLORSPACE_SRGB,
                .priv = 1},
        {800, 600, V4L2_PIX_FMT_SBGGR8, V4L2_FIELD_NONE,
                .bytesperline = 800,
                .sizeimage = 800 * 600,
                .colorspace = V4L2_COLORSPACE_SRGB,
                .priv = 1},
        {1024, 768, V4L2_PIX_FMT_SBGGR8, V4L2_FIELD_NONE,
                .bytesperline = 1024,
                .sizeimage = 1024 * 768,
                .colorspace = V4L2_COLORSPACE_SRGB,
                .priv = 1},
        {1600, 1200, V4L2_PIX_FMT_SBGGR8, V4L2_FIELD_NONE,
                .bytesperline = 1600,
                .sizeimage = 1600 * 1200,
                .colorspace = V4L2_COLORSPACE_SRGB,
                .priv = 0},
        {2048, 1536, V4L2_PIX_FMT_SBGGR8, V4L2_FIELD_NONE,
                .bytesperline = 2048,
                .sizeimage = 2048 * 1536,
                .colorspace = V4L2_COLORSPACE_SRGB,
                .priv = 0},
};
static const struct v4l2_pix_format ovfx2_ov9600_mode[] = {
        {640, 480, V4L2_PIX_FMT_SBGGR8, V4L2_FIELD_NONE,
                .bytesperline = 640,
                .sizeimage = 640 * 480,
                .colorspace = V4L2_COLORSPACE_SRGB,
                .priv = 1},
        {1280, 1024, V4L2_PIX_FMT_SBGGR8, V4L2_FIELD_NONE,
                .bytesperline = 1280,
                .sizeimage = 1280 * 1024,
                .colorspace = V4L2_COLORSPACE_SRGB},
};

/* Registers common to OV511 / OV518 */
#define R51x_FIFO_PSIZE                 0x30    /* 2 bytes wide w/ OV518(+) */
#define R51x_SYS_RESET                  0x50
        /* Reset type flags */
        #define OV511_RESET_OMNICE      0x08
#define R51x_SYS_INIT                   0x53
#define R51x_SYS_SNAP                   0x52
#define R51x_SYS_CUST_ID                0x5f
#define R51x_COMP_LUT_BEGIN             0x80

/* OV511 Camera interface register numbers */
#define R511_CAM_DELAY                  0x10
#define R511_CAM_EDGE                   0x11
#define R511_CAM_PXCNT                  0x12
#define R511_CAM_LNCNT                  0x13
#define R511_CAM_PXDIV                  0x14
#define R511_CAM_LNDIV                  0x15
#define R511_CAM_UV_EN                  0x16
#define R511_CAM_LINE_MODE              0x17
#define R511_CAM_OPTS                   0x18

#define R511_SNAP_FRAME                 0x19
#define R511_SNAP_PXCNT                 0x1a
#define R511_SNAP_LNCNT                 0x1b
#define R511_SNAP_PXDIV                 0x1c
#define R511_SNAP_LNDIV                 0x1d
#define R511_SNAP_UV_EN                 0x1e
#define R511_SNAP_OPTS                  0x1f

#define R511_DRAM_FLOW_CTL              0x20
#define R511_FIFO_OPTS                  0x31
#define R511_I2C_CTL                    0x40
#define R511_SYS_LED_CTL                0x55    /* OV511+ only */
#define R511_COMP_EN                    0x78
#define R511_COMP_LUT_EN                0x79

/* OV518 Camera interface register numbers */
#define R518_GPIO_OUT                   0x56    /* OV518(+) only */
#define R518_GPIO_CTL                   0x57    /* OV518(+) only */

/* OV519 Camera interface register numbers */
#define OV519_R10_H_SIZE                0x10
#define OV519_R11_V_SIZE                0x11
#define OV519_R12_X_OFFSETL             0x12
#define OV519_R13_X_OFFSETH             0x13
#define OV519_R14_Y_OFFSETL             0x14
#define OV519_R15_Y_OFFSETH             0x15
#define OV519_R16_DIVIDER               0x16
#define OV519_R20_DFR                   0x20
#define OV519_R25_FORMAT                0x25

/* OV519 System Controller register numbers */
#define OV519_R51_RESET1                0x51
#define OV519_R54_EN_CLK1               0x54
#define OV519_R57_SNAPSHOT              0x57

#define OV519_GPIO_DATA_OUT0            0x71
#define OV519_GPIO_IO_CTRL0             0x72

/*#define OV511_ENDPOINT_ADDRESS 1       * Isoc endpoint number */

/*
 * The FX2 chip does not give us a zero length read at end of frame.
 * It does, however, give a short read at the end of a frame, if
 * necessary, rather than run two frames together.
 *
 * By choosing the right bulk transfer size, we are guaranteed to always
 * get a short read for the last read of each frame.  Frame sizes are
 * always a composite number (width * height, or a multiple) so if we
 * choose a prime number, we are guaranteed that the last read of a
 * frame will be short.
 *
 * But it isn't that easy: the 2.6 kernel requires a multiple of 4KB,
 * otherwise EOVERFLOW "babbling" errors occur.  I have not been able
 * to figure out why.  [PMiller]
 *
 * The constant (13 * 4096) is the largest "prime enough" number less than 64KB.
 *
 * It isn't enough to know the number of bytes per frame, in case we
 * have data dropouts or buffer overruns (even though the FX2 double
 * buffers, there are some pretty strict real time constraints for
 * isochronous transfer for larger frame sizes).
 */
/*jfm: this value does not work for 800x600 - see isoc_init */
#define OVFX2_BULK_SIZE (13 * 4096)

/* I2C registers */
#define R51x_I2C_W_SID          0x41
#define R51x_I2C_SADDR_3        0x42
#define R51x_I2C_SADDR_2        0x43
#define R51x_I2C_R_SID          0x44
#define R51x_I2C_DATA           0x45
#define R518_I2C_CTL            0x47    /* OV518(+) only */
#define OVFX2_I2C_ADDR          0x00

/* I2C ADDRESSES */
#define OV7xx0_SID   0x42
#define OV_HIRES_SID 0x60               /* OV9xxx / OV2xxx / OV3xxx */
#define OV8xx0_SID   0xa0
#define OV6xx0_SID   0xc0

/* OV7610 registers */
#define OV7610_REG_GAIN         0x00    /* gain setting (5:0) */
#define OV7610_REG_BLUE         0x01    /* blue channel balance */
#define OV7610_REG_RED          0x02    /* red channel balance */
#define OV7610_REG_SAT          0x03    /* saturation */
#define OV8610_REG_HUE          0x04    /* 04 reserved */
#define OV7610_REG_CNT          0x05    /* Y contrast */
#define OV7610_REG_BRT          0x06    /* Y brightness */
#define OV7610_REG_COM_C        0x14    /* misc common regs */
#define OV7610_REG_ID_HIGH      0x1c    /* manufacturer ID MSB */
#define OV7610_REG_ID_LOW       0x1d    /* manufacturer ID LSB */
#define OV7610_REG_COM_I        0x29    /* misc settings */

/* OV7660 and OV7670 registers */
#define OV7670_R00_GAIN         0x00    /* Gain lower 8 bits (rest in vref) */
#define OV7670_R01_BLUE         0x01    /* blue gain */
#define OV7670_R02_RED          0x02    /* red gain */
#define OV7670_R03_VREF         0x03    /* Pieces of GAIN, VSTART, VSTOP */
#define OV7670_R04_COM1         0x04    /* Control 1 */
/*#define OV7670_R07_AECHH      0x07     * AEC MS 5 bits */
#define OV7670_R0C_COM3         0x0c    /* Control 3 */
#define OV7670_R0D_COM4         0x0d    /* Control 4 */
#define OV7670_R0E_COM5         0x0e    /* All "reserved" */
#define OV7670_R0F_COM6         0x0f    /* Control 6 */
#define OV7670_R10_AECH         0x10    /* More bits of AEC value */
#define OV7670_R11_CLKRC        0x11    /* Clock control */
#define OV7670_R12_COM7         0x12    /* Control 7 */
#define   OV7670_COM7_FMT_VGA    0x00
/*#define   OV7670_COM7_YUV      0x00    * YUV */
#define   OV7670_COM7_FMT_QVGA   0x10   /* QVGA format */
#define   OV7670_COM7_FMT_MASK   0x38
#define   OV7670_COM7_RESET      0x80   /* Register reset */
#define OV7670_R13_COM8         0x13    /* Control 8 */
#define   OV7670_COM8_AEC        0x01   /* Auto exposure enable */
#define   OV7670_COM8_AWB        0x02   /* White balance enable */
#define   OV7670_COM8_AGC        0x04   /* Auto gain enable */
#define   OV7670_COM8_BFILT      0x20   /* Band filter enable */
#define   OV7670_COM8_AECSTEP    0x40   /* Unlimited AEC step size */
#define   OV7670_COM8_FASTAEC    0x80   /* Enable fast AGC/AEC */
#define OV7670_R14_COM9         0x14    /* Control 9 - gain ceiling */
#define OV7670_R15_COM10        0x15    /* Control 10 */
#define OV7670_R17_HSTART       0x17    /* Horiz start high bits */
#define OV7670_R18_HSTOP        0x18    /* Horiz stop high bits */
#define OV7670_R19_VSTART       0x19    /* Vert start high bits */
#define OV7670_R1A_VSTOP        0x1a    /* Vert stop high bits */
#define OV7670_R1E_MVFP         0x1e    /* Mirror / vflip */
#define   OV7670_MVFP_VFLIP      0x10   /* vertical flip */
#define   OV7670_MVFP_MIRROR     0x20   /* Mirror image */
#define OV7670_R24_AEW          0x24    /* AGC upper limit */
#define OV7670_R25_AEB          0x25    /* AGC lower limit */
#define OV7670_R26_VPT          0x26    /* AGC/AEC fast mode op region */
#define OV7670_R32_HREF         0x32    /* HREF pieces */
#define OV7670_R3A_TSLB         0x3a    /* lots of stuff */
#define OV7670_R3B_COM11        0x3b    /* Control 11 */
#define   OV7670_COM11_EXP       0x02
#define   OV7670_COM11_HZAUTO    0x10   /* Auto detect 50/60 Hz */
#define OV7670_R3C_COM12        0x3c    /* Control 12 */
#define OV7670_R3D_COM13        0x3d    /* Control 13 */
#define   OV7670_COM13_GAMMA     0x80   /* Gamma enable */
#define   OV7670_COM13_UVSAT     0x40   /* UV saturation auto adjustment */
#define OV7670_R3E_COM14        0x3e    /* Control 14 */
#define OV7670_R3F_EDGE         0x3f    /* Edge enhancement factor */
#define OV7670_R40_COM15        0x40    /* Control 15 */
/*#define   OV7670_COM15_R00FF   0xc0    *      00 to FF */
#define OV7670_R41_COM16        0x41    /* Control 16 */
#define   OV7670_COM16_AWBGAIN   0x08   /* AWB gain enable */
/* end of ov7660 common registers */
#define OV7670_R55_BRIGHT       0x55    /* Brightness */
#define OV7670_R56_CONTRAS      0x56    /* Contrast control */
#define OV7670_R69_GFIX         0x69    /* Fix gain control */
/*#define OV7670_R8C_RGB444     0x8c     * RGB 444 control */
#define OV7670_R9F_HAECC1       0x9f    /* Hist AEC/AGC control 1 */
#define OV7670_RA0_HAECC2       0xa0    /* Hist AEC/AGC control 2 */
#define OV7670_RA5_BD50MAX      0xa5    /* 50hz banding step limit */
#define OV7670_RA6_HAECC3       0xa6    /* Hist AEC/AGC control 3 */
#define OV7670_RA7_HAECC4       0xa7    /* Hist AEC/AGC control 4 */
#define OV7670_RA8_HAECC5       0xa8    /* Hist AEC/AGC control 5 */
#define OV7670_RA9_HAECC6       0xa9    /* Hist AEC/AGC control 6 */
#define OV7670_RAA_HAECC7       0xaa    /* Hist AEC/AGC control 7 */
#define OV7670_RAB_BD60MAX      0xab    /* 60hz banding step limit */

struct ov_regvals {
        u8 reg;
        u8 val;
};
struct ov_i2c_regvals {
        u8 reg;
        u8 val;
};

/* Settings for OV2610 camera chip */
static const struct ov_i2c_regvals norm_2610[] = {
        { 0x12, 0x80 }, /* reset */
};

static const struct ov_i2c_regvals norm_2610ae[] = {
        {0x12, 0x80},   /* reset */
        {0x13, 0xcd},
        {0x09, 0x01},
        {0x0d, 0x00},
        {0x11, 0x80},
        {0x12, 0x20},   /* 1600x1200 */
        {0x33, 0x0c},
        {0x35, 0x90},
        {0x36, 0x37},
/* ms-win traces */
        {0x11, 0x83},   /* clock / 3 ? */
        {0x2d, 0x00},   /* 60 Hz filter */
        {0x24, 0xb0},   /* normal colors */
        {0x25, 0x90},
        {0x10, 0x43},
};

static const struct ov_i2c_regvals norm_3620b[] = {
        /*
         * From the datasheet: "Note that after writing to register COMH
         * (0x12) to change the sensor mode, registers related to the
         * sensor’s cropping window will be reset back to their default
         * values."
         *
         * "wait 4096 external clock ... to make sure the sensor is
         * stable and ready to access registers" i.e. 160us at 24MHz
         */
        { 0x12, 0x80 }, /* COMH reset */
        { 0x12, 0x00 }, /* QXGA, master */

        /*
         * 11 CLKRC "Clock Rate Control"
         * [7] internal frequency doublers: on
         * [6] video port mode: master
         * [5:0] clock divider: 1
         */
        { 0x11, 0x80 },

        /*
         * 13 COMI "Common Control I"
         *                  = 192 (0xC0) 11000000
         *    COMI[7] "AEC speed selection"
         *                  =   1 (0x01) 1....... "Faster AEC correction"
         *    COMI[6] "AEC speed step selection"
         *                  =   1 (0x01) .1...... "Big steps, fast"
         *    COMI[5] "Banding filter on off"
         *                  =   0 (0x00) ..0..... "Off"
         *    COMI[4] "Banding filter option"
         *                  =   0 (0x00) ...0.... "Main clock is 48 MHz and
         *                                         the PLL is ON"
         *    COMI[3] "Reserved"
         *                  =   0 (0x00) ....0...
         *    COMI[2] "AGC auto manual control selection"
         *                  =   0 (0x00) .....0.. "Manual"
         *    COMI[1] "AWB auto manual control selection"
         *                  =   0 (0x00) ......0. "Manual"
         *    COMI[0] "Exposure control"
         *                  =   0 (0x00) .......0 "Manual"
         */
        { 0x13, 0xc0 },

        /*
         * 09 COMC "Common Control C"
         *                  =   8 (0x08) 00001000
         *    COMC[7:5] "Reserved"
         *                  =   0 (0x00) 000.....
         *    COMC[4] "Sleep Mode Enable"
         *                  =   0 (0x00) ...0.... "Normal mode"
         *    COMC[3:2] "Sensor sampling reset timing selection"
         *                  =   2 (0x02) ....10.. "Longer reset time"
         *    COMC[1:0] "Output drive current select"
         *                  =   0 (0x00) ......00 "Weakest"
         */
        { 0x09, 0x08 },

        /*
         * 0C COMD "Common Control D"
         *                  =   8 (0x08) 00001000
         *    COMD[7] "Reserved"
         *                  =   0 (0x00) 0.......
         *    COMD[6] "Swap MSB and LSB at the output port"
         *                  =   0 (0x00) .0...... "False"
         *    COMD[5:3] "Reserved"
         *                  =   1 (0x01) ..001...
         *    COMD[2] "Output Average On Off"
         *                  =   0 (0x00) .....0.. "Output Normal"
         *    COMD[1] "Sensor precharge voltage selection"
         *                  =   0 (0x00) ......0. "Selects internal
         *                                         reference precharge
         *                                         voltage"
         *    COMD[0] "Snapshot option"
         *                  =   0 (0x00) .......0 "Enable live video output
         *                                         after snapshot sequence"
         */
        { 0x0c, 0x08 },

        /*
         * 0D COME "Common Control E"
         *                  = 161 (0xA1) 10100001
         *    COME[7] "Output average option"
         *                  =   1 (0x01) 1....... "Output average of 4 pixels"
         *    COME[6] "Anti-blooming control"
         *                  =   0 (0x00) .0...... "Off"
         *    COME[5:3] "Reserved"
         *                  =   4 (0x04) ..100...
         *    COME[2] "Clock output power down pin status"
         *                  =   0 (0x00) .....0.. "Tri-state data output pin
         *                                         on power down"
         *    COME[1] "Data output pin status selection at power down"
         *                  =   0 (0x00) ......0. "Tri-state VSYNC, PCLK,
         *                                         HREF, and CHSYNC pins on
         *                                         power down"
         *    COME[0] "Auto zero circuit select"
         *                  =   1 (0x01) .......1 "On"
         */
        { 0x0d, 0xa1 },

        /*
         * 0E COMF "Common Control F"
         *                  = 112 (0x70) 01110000
         *    COMF[7] "System clock selection"
         *                  =   0 (0x00) 0....... "Use 24 MHz system clock"
         *    COMF[6:4] "Reserved"
         *                  =   7 (0x07) .111....
         *    COMF[3] "Manual auto negative offset canceling selection"
         *                  =   0 (0x00) ....0... "Auto detect negative
         *                                         offset and cancel it"
         *    COMF[2:0] "Reserved"
         *                  =   0 (0x00) .....000
         */
        { 0x0e, 0x70 },

        /*
         * 0F COMG "Common Control G"
         *                  =  66 (0x42) 01000010
         *    COMG[7] "Optical black output selection"
         *                  =   0 (0x00) 0....... "Disable"
         *    COMG[6] "Black level calibrate selection"
         *                  =   1 (0x01) .1...... "Use optical black pixels
         *                                         to calibrate"
         *    COMG[5:4] "Reserved"
         *                  =   0 (0x00) ..00....
         *    COMG[3] "Channel offset adjustment"
         *                  =   0 (0x00) ....0... "Disable offset adjustment"
         *    COMG[2] "ADC black level calibration option"
         *                  =   0 (0x00) .....0.. "Use B/G line and G/R
         *                                         line to calibrate each
         *                                         channel's black level"
         *    COMG[1] "Reserved"
         *                  =   1 (0x01) ......1.
         *    COMG[0] "ADC black level calibration enable"
         *                  =   0 (0x00) .......0 "Disable"
         */
        { 0x0f, 0x42 },

        /*
         * 14 COMJ "Common Control J"
         *                  = 198 (0xC6) 11000110
         *    COMJ[7:6] "AGC gain ceiling"
         *                  =   3 (0x03) 11...... "8x"
         *    COMJ[5:4] "Reserved"
         *                  =   0 (0x00) ..00....
         *    COMJ[3] "Auto banding filter"
         *                  =   0 (0x00) ....0... "Banding filter is always
         *                                         on off depending on
         *                                         COMI[5] setting"
         *    COMJ[2] "VSYNC drop option"
         *                  =   1 (0x01) .....1.. "SYNC is dropped if frame
         *                                         data is dropped"
         *    COMJ[1] "Frame data drop"
         *                  =   1 (0x01) ......1. "Drop frame data if
         *                                         exposure is not within
         *                                         tolerance.  In AEC mode,
         *                                         data is normally dropped
         *                                         when data is out of
         *                                         range."
         *    COMJ[0] "Reserved"
         *                  =   0 (0x00) .......0
         */
        { 0x14, 0xc6 },

        /*
         * 15 COMK "Common Control K"
         *                  =   2 (0x02) 00000010
         *    COMK[7] "CHSYNC pin output swap"
         *                  =   0 (0x00) 0....... "CHSYNC"
         *    COMK[6] "HREF pin output swap"
         *                  =   0 (0x00) .0...... "HREF"
         *    COMK[5] "PCLK output selection"
         *                  =   0 (0x00) ..0..... "PCLK always output"
         *    COMK[4] "PCLK edge selection"
         *                  =   0 (0x00) ...0.... "Data valid on falling edge"
         *    COMK[3] "HREF output polarity"
         *                  =   0 (0x00) ....0... "positive"
         *    COMK[2] "Reserved"
         *                  =   0 (0x00) .....0..
         *    COMK[1] "VSYNC polarity"
         *                  =   1 (0x01) ......1. "negative"
         *    COMK[0] "HSYNC polarity"
         *                  =   0 (0x00) .......0 "positive"
         */
        { 0x15, 0x02 },

        /*
         * 33 CHLF "Current Control"
         *                  =   9 (0x09) 00001001
         *    CHLF[7:6] "Sensor current control"
         *                  =   0 (0x00) 00......
         *    CHLF[5] "Sensor current range control"
         *                  =   0 (0x00) ..0..... "normal range"
         *    CHLF[4] "Sensor current"
         *                  =   0 (0x00) ...0.... "normal current"
         *    CHLF[3] "Sensor buffer current control"
         *                  =   1 (0x01) ....1... "half current"
         *    CHLF[2] "Column buffer current control"
         *                  =   0 (0x00) .....0.. "normal current"
         *    CHLF[1] "Analog DSP current control"
         *                  =   0 (0x00) ......0. "normal current"
         *    CHLF[1] "ADC current control"
         *                  =   0 (0x00) ......0. "normal current"
         */
        { 0x33, 0x09 },

        /*
         * 34 VBLM "Blooming Control"
         *                  =  80 (0x50) 01010000
         *    VBLM[7] "Hard soft reset switch"
         *                  =   0 (0x00) 0....... "Hard reset"
         *    VBLM[6:4] "Blooming voltage selection"
         *                  =   5 (0x05) .101....
         *    VBLM[3:0] "Sensor current control"
         *                  =   0 (0x00) ....0000
         */
        { 0x34, 0x50 },

        /*
         * 36 VCHG "Sensor Precharge Voltage Control"
         *                  =   0 (0x00) 00000000
         *    VCHG[7] "Reserved"
         *                  =   0 (0x00) 0.......
         *    VCHG[6:4] "Sensor precharge voltage control"
         *                  =   0 (0x00) .000....
         *    VCHG[3:0] "Sensor array common reference"
         *                  =   0 (0x00) ....0000
         */
        { 0x36, 0x00 },

        /*
         * 37 ADC "ADC Reference Control"
         *                  =   4 (0x04) 00000100
         *    ADC[7:4] "Reserved"
         *                  =   0 (0x00) 0000....
         *    ADC[3] "ADC input signal range"
         *                  =   0 (0x00) ....0... "Input signal 1.0x"
         *    ADC[2:0] "ADC range control"
         *                  =   4 (0x04) .....100

         */
        { 0x37, 0x04 },

        /*
         * 38 ACOM "Analog Common Ground"
         *                  =  82 (0x52) 01010010
         *    ACOM[7] "Analog gain control"
         *                  =   0 (0x00) 0....... "Gain 1x"
         *    ACOM[6] "Analog black level calibration"
         *                  =   1 (0x01) .1...... "On"
         *    ACOM[5:0] "Reserved"
         *                  =  18 (0x12) ..010010
         */
        { 0x38, 0x52 },

        /*
         * 3A FREFA "Internal Reference Adjustment"
         *                  =   0 (0x00) 00000000
         *    FREFA[7:0] "Range"
         *                  =   0 (0x00) 00000000
         */
        { 0x3a, 0x00 },

        /*
         * 3C FVOPT "Internal Reference Adjustment"
         *                  =  31 (0x1F) 00011111
         *    FVOPT[7:0] "Range"
         *                  =  31 (0x1F) 00011111
         */
        { 0x3c, 0x1f },

        /*
         * 44 Undocumented  =   0 (0x00) 00000000
         *    44[7:0] "It's a secret"
         *                  =   0 (0x00) 00000000
         */
        { 0x44, 0x00 },

        /*
         * 40 Undocumented  =   0 (0x00) 00000000
         *    40[7:0] "It's a secret"
         *                  =   0 (0x00) 00000000
         */
        { 0x40, 0x00 },

        /*
         * 41 Undocumented  =   0 (0x00) 00000000
         *    41[7:0] "It's a secret"
         *                  =   0 (0x00) 00000000
         */
        { 0x41, 0x00 },

        /*
         * 42 Undocumented  =   0 (0x00) 00000000
         *    42[7:0] "It's a secret"
         *                  =   0 (0x00) 00000000
         */
        { 0x42, 0x00 },

        /*
         * 43 Undocumented  =   0 (0x00) 00000000
         *    43[7:0] "It's a secret"
         *                  =   0 (0x00) 00000000
         */
        { 0x43, 0x00 },

        /*
         * 45 Undocumented  = 128 (0x80) 10000000
         *    45[7:0] "It's a secret"
         *                  = 128 (0x80) 10000000
         */
        { 0x45, 0x80 },

        /*
         * 48 Undocumented  = 192 (0xC0) 11000000
         *    48[7:0] "It's a secret"
         *                  = 192 (0xC0) 11000000
         */
        { 0x48, 0xc0 },

        /*
         * 49 Undocumented  =  25 (0x19) 00011001
         *    49[7:0] "It's a secret"
         *                  =  25 (0x19) 00011001
         */
        { 0x49, 0x19 },

        /*
         * 4B Undocumented  = 128 (0x80) 10000000
         *    4B[7:0] "It's a secret"
         *                  = 128 (0x80) 10000000
         */
        { 0x4b, 0x80 },

        /*
         * 4D Undocumented  = 196 (0xC4) 11000100
         *    4D[7:0] "It's a secret"
         *                  = 196 (0xC4) 11000100
         */
        { 0x4d, 0xc4 },

        /*
         * 35 VREF "Reference Voltage Control"
         *                  =  76 (0x4c) 01001100
         *    VREF[7:5] "Column high reference control"
         *                  =   2 (0x02) 010..... "higher voltage"
         *    VREF[4:2] "Column low reference control"
         *                  =   3 (0x03) ...011.. "Highest voltage"
         *    VREF[1:0] "Reserved"
         *                  =   0 (0x00) ......00
         */
        { 0x35, 0x4c },

        /*
         * 3D Undocumented  =   0 (0x00) 00000000
         *    3D[7:0] "It's a secret"
         *                  =   0 (0x00) 00000000
         */
        { 0x3d, 0x00 },

        /*
         * 3E Undocumented  =   0 (0x00) 00000000
         *    3E[7:0] "It's a secret"
         *                  =   0 (0x00) 00000000
         */
        { 0x3e, 0x00 },

        /*
         * 3B FREFB "Internal Reference Adjustment"
         *                  =  24 (0x18) 00011000
         *    FREFB[7:0] "Range"
         *                  =  24 (0x18) 00011000
         */
        { 0x3b, 0x18 },

        /*
         * 33 CHLF "Current Control"
         *                  =  25 (0x19) 00011001
         *    CHLF[7:6] "Sensor current control"
         *                  =   0 (0x00) 00......
         *    CHLF[5] "Sensor current range control"
         *                  =   0 (0x00) ..0..... "normal range"
         *    CHLF[4] "Sensor current"
         *                  =   1 (0x01) ...1.... "double current"
         *    CHLF[3] "Sensor buffer current control"
         *                  =   1 (0x01) ....1... "half current"
         *    CHLF[2] "Column buffer current control"
         *                  =   0 (0x00) .....0.. "normal current"
         *    CHLF[1] "Analog DSP current control"
         *                  =   0 (0x00) ......0. "normal current"
         *    CHLF[1] "ADC current control"
         *                  =   0 (0x00) ......0. "normal current"
         */
        { 0x33, 0x19 },

        /*
         * 34 VBLM "Blooming Control"
         *                  =  90 (0x5A) 01011010
         *    VBLM[7] "Hard soft reset switch"
         *                  =   0 (0x00) 0....... "Hard reset"
         *    VBLM[6:4] "Blooming voltage selection"
         *                  =   5 (0x05) .101....
         *    VBLM[3:0] "Sensor current control"
         *                  =  10 (0x0A) ....1010
         */
        { 0x34, 0x5a },

        /*
         * 3B FREFB "Internal Reference Adjustment"
         *                  =   0 (0x00) 00000000
         *    FREFB[7:0] "Range"
         *                  =   0 (0x00) 00000000
         */
        { 0x3b, 0x00 },

        /*
         * 33 CHLF "Current Control"
         *                  =   9 (0x09) 00001001
         *    CHLF[7:6] "Sensor current control"
         *                  =   0 (0x00) 00......
         *    CHLF[5] "Sensor current range control"
         *                  =   0 (0x00) ..0..... "normal range"
         *    CHLF[4] "Sensor current"
         *                  =   0 (0x00) ...0.... "normal current"
         *    CHLF[3] "Sensor buffer current control"
         *                  =   1 (0x01) ....1... "half current"
         *    CHLF[2] "Column buffer current control"
         *                  =   0 (0x00) .....0.. "normal current"
         *    CHLF[1] "Analog DSP current control"
         *                  =   0 (0x00) ......0. "normal current"
         *    CHLF[1] "ADC current control"
         *                  =   0 (0x00) ......0. "normal current"
         */
        { 0x33, 0x09 },

        /*
         * 34 VBLM "Blooming Control"
         *                  =  80 (0x50) 01010000
         *    VBLM[7] "Hard soft reset switch"
         *                  =   0 (0x00) 0....... "Hard reset"
         *    VBLM[6:4] "Blooming voltage selection"
         *                  =   5 (0x05) .101....
         *    VBLM[3:0] "Sensor current control"
         *                  =   0 (0x00) ....0000
         */
        { 0x34, 0x50 },

        /*
         * 12 COMH "Common Control H"
         *                  =  64 (0x40) 01000000
         *    COMH[7] "SRST"
         *                  =   0 (0x00) 0....... "No-op"
         *    COMH[6:4] "Resolution selection"
         *                  =   4 (0x04) .100.... "XGA"
         *    COMH[3] "Master slave selection"
         *                  =   0 (0x00) ....0... "Master mode"
         *    COMH[2] "Internal B/R channel option"
         *                  =   0 (0x00) .....0.. "B/R use same channel"
         *    COMH[1] "Color bar test pattern"
         *                  =   0 (0x00) ......0. "Off"
         *    COMH[0] "Reserved"
         *                  =   0 (0x00) .......0
         */
        { 0x12, 0x40 },

        /*
         * 17 HREFST "Horizontal window start"
         *                  =  31 (0x1F) 00011111
         *    HREFST[7:0] "Horizontal window start, 8 MSBs"
         *                  =  31 (0x1F) 00011111
         */
        { 0x17, 0x1f },

        /*
         * 18 HREFEND "Horizontal window end"
         *                  =  95 (0x5F) 01011111
         *    HREFEND[7:0] "Horizontal Window End, 8 MSBs"
         *                  =  95 (0x5F) 01011111
         */
        { 0x18, 0x5f },

        /*
         * 19 VSTRT "Vertical window start"
         *                  =   0 (0x00) 00000000
         *    VSTRT[7:0] "Vertical Window Start, 8 MSBs"
         *                  =   0 (0x00) 00000000
         */
        { 0x19, 0x00 },

        /*
         * 1A VEND "Vertical window end"
         *                  =  96 (0x60) 01100000
         *    VEND[7:0] "Vertical Window End, 8 MSBs"
         *                  =  96 (0x60) 01100000
         */
        { 0x1a, 0x60 },

        /*
         * 32 COMM "Common Control M"
         *                  =  18 (0x12) 00010010
         *    COMM[7:6] "Pixel clock divide option"
         *                  =   0 (0x00) 00...... "/1"
         *    COMM[5:3] "Horizontal window end position, 3 LSBs"
         *                  =   2 (0x02) ..010...
         *    COMM[2:0] "Horizontal window start position, 3 LSBs"
         *                  =   2 (0x02) .....010
         */
        { 0x32, 0x12 },

        /*
         * 03 COMA "Common Control A"
         *                  =  74 (0x4A) 01001010
         *    COMA[7:4] "AWB Update Threshold"
         *                  =   4 (0x04) 0100....
         *    COMA[3:2] "Vertical window end line control 2 LSBs"
         *                  =   2 (0x02) ....10..
         *    COMA[1:0] "Vertical window start line control 2 LSBs"
         *                  =   2 (0x02) ......10
         */
        { 0x03, 0x4a },

        /*
         * 11 CLKRC "Clock Rate Control"
         *                  = 128 (0x80) 10000000
         *    CLKRC[7] "Internal frequency doublers on off seclection"
         *                  =   1 (0x01) 1....... "On"
         *    CLKRC[6] "Digital video master slave selection"
         *                  =   0 (0x00) .0...... "Master mode, sensor
         *                                         provides PCLK"
         *    CLKRC[5:0] "Clock divider { CLK = PCLK/(1+CLKRC[5:0]) }"
         *                  =   0 (0x00) ..000000
         */
        { 0x11, 0x80 },

        /*
         * 12 COMH "Common Control H"
         *                  =   0 (0x00) 00000000
         *    COMH[7] "SRST"
         *                  =   0 (0x00) 0....... "No-op"
         *    COMH[6:4] "Resolution selection"
         *                  =   0 (0x00) .000.... "QXGA"
         *    COMH[3] "Master slave selection"
         *                  =   0 (0x00) ....0... "Master mode"
         *    COMH[2] "Internal B/R channel option"
         *                  =   0 (0x00) .....0.. "B/R use same channel"
         *    COMH[1] "Color bar test pattern"
         *                  =   0 (0x00) ......0. "Off"
         *    COMH[0] "Reserved"
         *                  =   0 (0x00) .......0
         */
        { 0x12, 0x00 },

        /*
         * 12 COMH "Common Control H"
         *                  =  64 (0x40) 01000000
         *    COMH[7] "SRST"
         *                  =   0 (0x00) 0....... "No-op"
         *    COMH[6:4] "Resolution selection"
         *                  =   4 (0x04) .100.... "XGA"
         *    COMH[3] "Master slave selection"
         *                  =   0 (0x00) ....0... "Master mode"
         *    COMH[2] "Internal B/R channel option"
         *                  =   0 (0x00) .....0.. "B/R use same channel"
         *    COMH[1] "Color bar test pattern"
         *                  =   0 (0x00) ......0. "Off"
         *    COMH[0] "Reserved"
         *                  =   0 (0x00) .......0
         */
        { 0x12, 0x40 },

        /*
         * 17 HREFST "Horizontal window start"
         *                  =  31 (0x1F) 00011111
         *    HREFST[7:0] "Horizontal window start, 8 MSBs"
         *                  =  31 (0x1F) 00011111
         */
        { 0x17, 0x1f },

        /*
         * 18 HREFEND "Horizontal window end"
         *                  =  95 (0x5F) 01011111
         *    HREFEND[7:0] "Horizontal Window End, 8 MSBs"
         *                  =  95 (0x5F) 01011111
         */
        { 0x18, 0x5f },

        /*
         * 19 VSTRT "Vertical window start"
         *                  =   0 (0x00) 00000000
         *    VSTRT[7:0] "Vertical Window Start, 8 MSBs"
         *                  =   0 (0x00) 00000000
         */
        { 0x19, 0x00 },

        /*
         * 1A VEND "Vertical window end"
         *                  =  96 (0x60) 01100000
         *    VEND[7:0] "Vertical Window End, 8 MSBs"
         *                  =  96 (0x60) 01100000
         */
        { 0x1a, 0x60 },

        /*
         * 32 COMM "Common Control M"
         *                  =  18 (0x12) 00010010
         *    COMM[7:6] "Pixel clock divide option"
         *                  =   0 (0x00) 00...... "/1"
         *    COMM[5:3] "Horizontal window end position, 3 LSBs"
         *                  =   2 (0x02) ..010...
         *    COMM[2:0] "Horizontal window start position, 3 LSBs"
         *                  =   2 (0x02) .....010
         */
        { 0x32, 0x12 },

        /*
         * 03 COMA "Common Control A"
         *                  =  74 (0x4A) 01001010
         *    COMA[7:4] "AWB Update Threshold"
         *                  =   4 (0x04) 0100....
         *    COMA[3:2] "Vertical window end line control 2 LSBs"
         *                  =   2 (0x02) ....10..
         *    COMA[1:0] "Vertical window start line control 2 LSBs"
         *                  =   2 (0x02) ......10
         */
        { 0x03, 0x4a },

        /*
         * 02 RED "Red Gain Control"
         *                  = 175 (0xAF) 10101111
         *    RED[7] "Action"
         *                  =   1 (0x01) 1....... "gain = 1/(1+bitrev([6:0]))"
         *    RED[6:0] "Value"
         *                  =  47 (0x2F) .0101111
         */
        { 0x02, 0xaf },

        /*
         * 2D ADDVSL "VSYNC Pulse Width"
         *                  = 210 (0xD2) 11010010
         *    ADDVSL[7:0] "VSYNC pulse width, LSB"
         *                  = 210 (0xD2) 11010010
         */
        { 0x2d, 0xd2 },

        /*
         * 00 GAIN          =  24 (0x18) 00011000
         *    GAIN[7:6] "Reserved"
         *                  =   0 (0x00) 00......
         *    GAIN[5] "Double"
         *                  =   0 (0x00) ..0..... "False"
         *    GAIN[4] "Double"
         *                  =   1 (0x01) ...1.... "True"
         *    GAIN[3:0] "Range"
         *                  =   8 (0x08) ....1000
         */
        { 0x00, 0x18 },

        /*
         * 01 BLUE "Blue Gain Control"
         *                  = 240 (0xF0) 11110000
         *    BLUE[7] "Action"
         *                  =   1 (0x01) 1....... "gain = 1/(1+bitrev([6:0]))"
         *    BLUE[6:0] "Value"
         *                  = 112 (0x70) .1110000
         */
        { 0x01, 0xf0 },

        /*
         * 10 AEC "Automatic Exposure Control"
         *                  =  10 (0x0A) 00001010
         *    AEC[7:0] "Automatic Exposure Control, 8 MSBs"
         *                  =  10 (0x0A) 00001010
         */
        { 0x10, 0x0a },

        { 0xe1, 0x67 },
        { 0xe3, 0x03 },
        { 0xe4, 0x26 },
        { 0xe5, 0x3e },
        { 0xf8, 0x01 },
        { 0xff, 0x01 },
};

static const struct ov_i2c_regvals norm_6x20[] = {
        { 0x12, 0x80 }, /* reset */
        { 0x11, 0x01 },
        { 0x03, 0x60 },
        { 0x05, 0x7f }, /* For when autoadjust is off */
        { 0x07, 0xa8 },
        /* The ratio of 0x0c and 0x0d controls the white point */
        { 0x0c, 0x24 },
        { 0x0d, 0x24 },
        { 0x0f, 0x15 }, /* COMS */
        { 0x10, 0x75 }, /* AEC Exposure time */
        { 0x12, 0x24 }, /* Enable AGC */
        { 0x14, 0x04 },
        /* 0x16: 0x06 helps frame stability with moving objects */
        { 0x16, 0x06 },
/*      { 0x20, 0x30 },  * Aperture correction enable */
        { 0x26, 0xb2 }, /* BLC enable */
        /* 0x28: 0x05 Selects RGB format if RGB on */
        { 0x28, 0x05 },
        { 0x2a, 0x04 }, /* Disable framerate adjust */
/*      { 0x2b, 0xac },  * Framerate; Set 2a[7] first */
        { 0x2d, 0x85 },
        { 0x33, 0xa0 }, /* Color Processing Parameter */
        { 0x34, 0xd2 }, /* Max A/D range */
        { 0x38, 0x8b },
        { 0x39, 0x40 },

        { 0x3c, 0x39 }, /* Enable AEC mode changing */
        { 0x3c, 0x3c }, /* Change AEC mode */
        { 0x3c, 0x24 }, /* Disable AEC mode changing */

        { 0x3d, 0x80 },
        /* These next two registers (0x4a, 0x4b) are undocumented.
         * They control the color balance */
        { 0x4a, 0x80 },
        { 0x4b, 0x80 },
        { 0x4d, 0xd2 }, /* This reduces noise a bit */
        { 0x4e, 0xc1 },
        { 0x4f, 0x04 },
/* Do 50-53 have any effect? */
/* Toggle 0x12[2] off and on here? */
};

static const struct ov_i2c_regvals norm_6x30[] = {
        { 0x12, 0x80 }, /* Reset */
        { 0x00, 0x1f }, /* Gain */
        { 0x01, 0x99 }, /* Blue gain */
        { 0x02, 0x7c }, /* Red gain */
        { 0x03, 0xc0 }, /* Saturation */
        { 0x05, 0x0a }, /* Contrast */
        { 0x06, 0x95 }, /* Brightness */
        { 0x07, 0x2d }, /* Sharpness */
        { 0x0c, 0x20 },
        { 0x0d, 0x20 },
        { 0x0e, 0xa0 }, /* Was 0x20, bit7 enables a 2x gain which we need */
        { 0x0f, 0x05 },
        { 0x10, 0x9a },
        { 0x11, 0x00 }, /* Pixel clock = fastest */
        { 0x12, 0x24 }, /* Enable AGC and AWB */
        { 0x13, 0x21 },
        { 0x14, 0x80 },
        { 0x15, 0x01 },
        { 0x16, 0x03 },
        { 0x17, 0x38 },
        { 0x18, 0xea },
        { 0x19, 0x04 },
        { 0x1a, 0x93 },
        { 0x1b, 0x00 },
        { 0x1e, 0xc4 },
        { 0x1f, 0x04 },
        { 0x20, 0x20 },
        { 0x21, 0x10 },
        { 0x22, 0x88 },
        { 0x23, 0xc0 }, /* Crystal circuit power level */
        { 0x25, 0x9a }, /* Increase AEC black ratio */
        { 0x26, 0xb2 }, /* BLC enable */
        { 0x27, 0xa2 },
        { 0x28, 0x00 },
        { 0x29, 0x00 },
        { 0x2a, 0x84 }, /* 60 Hz power */
        { 0x2b, 0xa8 }, /* 60 Hz power */
        { 0x2c, 0xa0 },
        { 0x2d, 0x95 }, /* Enable auto-brightness */
        { 0x2e, 0x88 },
        { 0x33, 0x26 },
        { 0x34, 0x03 },
        { 0x36, 0x8f },
        { 0x37, 0x80 },
        { 0x38, 0x83 },
        { 0x39, 0x80 },
        { 0x3a, 0x0f },
        { 0x3b, 0x3c },
        { 0x3c, 0x1a },
        { 0x3d, 0x80 },
        { 0x3e, 0x80 },
        { 0x3f, 0x0e },
        { 0x40, 0x00 }, /* White bal */
        { 0x41, 0x00 }, /* White bal */
        { 0x42, 0x80 },
        { 0x43, 0x3f }, /* White bal */
        { 0x44, 0x80 },
        { 0x45, 0x20 },
        { 0x46, 0x20 },
        { 0x47, 0x80 },
        { 0x48, 0x7f },
        { 0x49, 0x00 },
        { 0x4a, 0x00 },
        { 0x4b, 0x80 },
        { 0x4c, 0xd0 },
        { 0x4d, 0x10 }, /* U = 0.563u, V = 0.714v */
        { 0x4e, 0x40 },
        { 0x4f, 0x07 }, /* UV avg., col. killer: max */
        { 0x50, 0xff },
        { 0x54, 0x23 }, /* Max AGC gain: 18dB */
        { 0x55, 0xff },
        { 0x56, 0x12 },
        { 0x57, 0x81 },
        { 0x58, 0x75 },
        { 0x59, 0x01 }, /* AGC dark current comp.: +1 */
        { 0x5a, 0x2c },
        { 0x5b, 0x0f }, /* AWB chrominance levels */
        { 0x5c, 0x10 },
        { 0x3d, 0x80 },
        { 0x27, 0xa6 },
        { 0x12, 0x20 }, /* Toggle AWB */
        { 0x12, 0x24 },
};

/* Lawrence Glaister <lg@jfm.bc.ca> reports:
 *
 * Register 0x0f in the 7610 has the following effects:
 *
 * 0x85 (AEC method 1): Best overall, good contrast range
 * 0x45 (AEC method 2): Very overexposed
 * 0xa5 (spec sheet default): Ok, but the black level is
 *      shifted resulting in loss of contrast
 * 0x05 (old driver setting): very overexposed, too much
 *      contrast
 */
static const struct ov_i2c_regvals norm_7610[] = {
        { 0x10, 0xff },
        { 0x16, 0x06 },
        { 0x28, 0x24 },
        { 0x2b, 0xac },
        { 0x12, 0x00 },
        { 0x38, 0x81 },
        { 0x28, 0x24 }, /* 0c */
        { 0x0f, 0x85 }, /* lg's setting */
        { 0x15, 0x01 },
        { 0x20, 0x1c },
        { 0x23, 0x2a },
        { 0x24, 0x10 },
        { 0x25, 0x8a },
        { 0x26, 0xa2 },
        { 0x27, 0xc2 },
        { 0x2a, 0x04 },
        { 0x2c, 0xfe },
        { 0x2d, 0x93 },
        { 0x30, 0x71 },
        { 0x31, 0x60 },
        { 0x32, 0x26 },
        { 0x33, 0x20 },
        { 0x34, 0x48 },
        { 0x12, 0x24 },
        { 0x11, 0x01 },
        { 0x0c, 0x24 },
        { 0x0d, 0x24 },
};

static const struct ov_i2c_regvals norm_7620[] = {
        { 0x12, 0x80 },         /* reset */
        { 0x00, 0x00 },         /* gain */
        { 0x01, 0x80 },         /* blue gain */
        { 0x02, 0x80 },         /* red gain */
        { 0x03, 0xc0 },         /* OV7670_R03_VREF */
        { 0x06, 0x60 },
        { 0x07, 0x00 },
        { 0x0c, 0x24 },
        { 0x0c, 0x24 },
        { 0x0d, 0x24 },
        { 0x11, 0x01 },
        { 0x12, 0x24 },
        { 0x13, 0x01 },
        { 0x14, 0x84 },
        { 0x15, 0x01 },
        { 0x16, 0x03 },
        { 0x17, 0x2f },
        { 0x18, 0xcf },
        { 0x19, 0x06 },
        { 0x1a, 0xf5 },
        { 0x1b, 0x00 },
        { 0x20, 0x18 },
        { 0x21, 0x80 },
        { 0x22, 0x80 },
        { 0x23, 0x00 },
        { 0x26, 0xa2 },
        { 0x27, 0xea },
        { 0x28, 0x22 }, /* Was 0x20, bit1 enables a 2x gain which we need */
        { 0x29, 0x00 },
        { 0x2a, 0x10 },
        { 0x2b, 0x00 },
        { 0x2c, 0x88 },
        { 0x2d, 0x91 },
        { 0x2e, 0x80 },
        { 0x2f, 0x44 },
        { 0x60, 0x27 },
        { 0x61, 0x02 },
        { 0x62, 0x5f },
        { 0x63, 0xd5 },
        { 0x64, 0x57 },
        { 0x65, 0x83 },
        { 0x66, 0x55 },
        { 0x67, 0x92 },
        { 0x68, 0xcf },
        { 0x69, 0x76 },
        { 0x6a, 0x22 },
        { 0x6b, 0x00 },
        { 0x6c, 0x02 },
        { 0x6d, 0x44 },
        { 0x6e, 0x80 },
        { 0x6f, 0x1d },
        { 0x70, 0x8b },
        { 0x71, 0x00 },
        { 0x72, 0x14 },
        { 0x73, 0x54 },
        { 0x74, 0x00 },
        { 0x75, 0x8e },
        { 0x76, 0x00 },
        { 0x77, 0xff },
        { 0x78, 0x80 },
        { 0x79, 0x80 },
        { 0x7a, 0x80 },
        { 0x7b, 0xe2 },
        { 0x7c, 0x00 },
};

/* 7640 and 7648. The defaults should be OK for most registers. */
static const struct ov_i2c_regvals norm_7640[] = {
        { 0x12, 0x80 },
        { 0x12, 0x14 },
};

static const struct ov_regvals init_519_ov7660[] = {
        { 0x5d, 0x03 }, /* Turn off suspend mode */
        { 0x53, 0x9b }, /* 0x9f enables the (unused) microcontroller */
        { 0x54, 0x0f }, /* bit2 (jpeg enable) */
        { 0xa2, 0x20 }, /* a2-a5 are undocumented */
        { 0xa3, 0x18 },
        { 0xa4, 0x04 },
        { 0xa5, 0x28 },
        { 0x37, 0x00 }, /* SetUsbInit */
        { 0x55, 0x02 }, /* 4.096 Mhz audio clock */
        /* Enable both fields, YUV Input, disable defect comp (why?) */
        { 0x20, 0x0c }, /* 0x0d does U <-> V swap */
        { 0x21, 0x38 },
        { 0x22, 0x1d },
        { 0x17, 0x50 }, /* undocumented */
        { 0x37, 0x00 }, /* undocumented */
        { 0x40, 0xff }, /* I2C timeout counter */
        { 0x46, 0x00 }, /* I2C clock prescaler */
};
static const struct ov_i2c_regvals norm_7660[] = {
        {OV7670_R12_COM7, OV7670_COM7_RESET},
        {OV7670_R11_CLKRC, 0x81},
        {0x92, 0x00},                   /* DM_LNL */
        {0x93, 0x00},                   /* DM_LNH */
        {0x9d, 0x4c},                   /* BD50ST */
        {0x9e, 0x3f},                   /* BD60ST */
        {OV7670_R3B_COM11, 0x02},
        {OV7670_R13_COM8, 0xf5},
        {OV7670_R10_AECH, 0x00},
        {OV7670_R00_GAIN, 0x00},
        {OV7670_R01_BLUE, 0x7c},
        {OV7670_R02_RED, 0x9d},
        {OV7670_R12_COM7, 0x00},
        {OV7670_R04_COM1, 00},
        {OV7670_R18_HSTOP, 0x01},
        {OV7670_R17_HSTART, 0x13},
        {OV7670_R32_HREF, 0x92},
        {OV7670_R19_VSTART, 0x02},
        {OV7670_R1A_VSTOP, 0x7a},
        {OV7670_R03_VREF, 0x00},
        {OV7670_R0E_COM5, 0x04},
        {OV7670_R0F_COM6, 0x62},
        {OV7670_R15_COM10, 0x00},
        {0x16, 0x02},                   /* RSVD */
        {0x1b, 0x00},                   /* PSHFT */
        {OV7670_R1E_MVFP, 0x01},
        {0x29, 0x3c},                   /* RSVD */
        {0x33, 0x00},                   /* CHLF */
        {0x34, 0x07},                   /* ARBLM */
        {0x35, 0x84},                   /* RSVD */
        {0x36, 0x00},                   /* RSVD */
        {0x37, 0x04},                   /* ADC */
        {0x39, 0x43},                   /* OFON */
        {OV7670_R3A_TSLB, 0x00},
        {OV7670_R3C_COM12, 0x6c},
        {OV7670_R3D_COM13, 0x98},
        {OV7670_R3F_EDGE, 0x23},
        {OV7670_R40_COM15, 0xc1},
        {OV7670_R41_COM16, 0x22},
        {0x6b, 0x0a},                   /* DBLV */
        {0xa1, 0x08},                   /* RSVD */
        {0x69, 0x80},                   /* HV */
        {0x43, 0xf0},                   /* RSVD.. */
        {0x44, 0x10},
        {0x45, 0x78},
        {0x46, 0xa8},
        {0x47, 0x60},
        {0x48, 0x80},
        {0x59, 0xba},
        {0x5a, 0x9a},
        {0x5b, 0x22},
        {0x5c, 0xb9},
        {0x5d, 0x9b},
        {0x5e, 0x10},
        {0x5f, 0xe0},
        {0x60, 0x85},
        {0x61, 0x60},
        {0x9f, 0x9d},                   /* RSVD */
        {0xa0, 0xa0},                   /* DSPC2 */
        {0x4f, 0x60},                   /* matrix */
        {0x50, 0x64},
        {0x51, 0x04},
        {0x52, 0x18},
        {0x53, 0x3c},
        {0x54, 0x54},
        {0x55, 0x40},
        {0x56, 0x40},
        {0x57, 0x40},
        {0x58, 0x0d},                   /* matrix sign */
        {0x8b, 0xcc},                   /* RSVD */
        {0x8c, 0xcc},
        {0x8d, 0xcf},
        {0x6c, 0x40},                   /* gamma curve */
        {0x6d, 0xe0},
        {0x6e, 0xa0},
        {0x6f, 0x80},
        {0x70, 0x70},
        {0x71, 0x80},
        {0x72, 0x60},
        {0x73, 0x60},
        {0x74, 0x50},
        {0x75, 0x40},
        {0x76, 0x38},
        {0x77, 0x3c},
        {0x78, 0x32},
        {0x79, 0x1a},
        {0x7a, 0x28},
        {0x7b, 0x24},
        {0x7c, 0x04},                   /* gamma curve */
        {0x7d, 0x12},
        {0x7e, 0x26},
        {0x7f, 0x46},
        {0x80, 0x54},
        {0x81, 0x64},
        {0x82, 0x70},
        {0x83, 0x7c},
        {0x84, 0x86},
        {0x85, 0x8e},
        {0x86, 0x9c},
        {0x87, 0xab},
        {0x88, 0xc4},
        {0x89, 0xd1},
        {0x8a, 0xe5},
        {OV7670_R14_COM9, 0x1e},
        {OV7670_R24_AEW, 0x80},
        {OV7670_R25_AEB, 0x72},
        {OV7670_R26_VPT, 0xb3},
        {0x62, 0x80},                   /* LCC1 */
        {0x63, 0x80},                   /* LCC2 */
        {0x64, 0x06},                   /* LCC3 */
        {0x65, 0x00},                   /* LCC4 */
        {0x66, 0x01},                   /* LCC5 */
        {0x94, 0x0e},                   /* RSVD.. */
        {0x95, 0x14},
        {OV7670_R13_COM8, OV7670_COM8_FASTAEC
                        | OV7670_COM8_AECSTEP
                        | OV7670_COM8_BFILT
                        | 0x10
                        | OV7670_COM8_AGC
                        | OV7670_COM8_AWB
                        | OV7670_COM8_AEC},
        {0xa1, 0xc8}
};
static const struct ov_i2c_regvals norm_9600[] = {
        {0x12, 0x80},
        {0x0c, 0x28},
        {0x11, 0x80},
        {0x13, 0xb5},
        {0x14, 0x3e},
        {0x1b, 0x04},
        {0x24, 0xb0},
        {0x25, 0x90},
        {0x26, 0x94},
        {0x35, 0x90},
        {0x37, 0x07},
        {0x38, 0x08},
        {0x01, 0x8e},
        {0x02, 0x85}
};

/* 7670. Defaults taken from OmniVision provided data,
*  as provided by Jonathan Corbet of OLPC               */
static const struct ov_i2c_regvals norm_7670[] = {
        { OV7670_R12_COM7, OV7670_COM7_RESET },
        { OV7670_R3A_TSLB, 0x04 },              /* OV */
        { OV7670_R12_COM7, OV7670_COM7_FMT_VGA }, /* VGA */
        { OV7670_R11_CLKRC, 0x01 },
/*
 * Set the hardware window.  These values from OV don't entirely
 * make sense - hstop is less than hstart.  But they work...
 */
        { OV7670_R17_HSTART, 0x13 },
        { OV7670_R18_HSTOP, 0x01 },
        { OV7670_R32_HREF, 0xb6 },
        { OV7670_R19_VSTART, 0x02 },
        { OV7670_R1A_VSTOP, 0x7a },
        { OV7670_R03_VREF, 0x0a },

        { OV7670_R0C_COM3, 0x00 },
        { OV7670_R3E_COM14, 0x00 },
/* Mystery scaling numbers */
        { 0x70, 0x3a },
        { 0x71, 0x35 },
        { 0x72, 0x11 },
        { 0x73, 0xf0 },
        { 0xa2, 0x02 },
/*      { OV7670_R15_COM10, 0x0 }, */

/* Gamma curve values */
        { 0x7a, 0x20 },
        { 0x7b, 0x10 },
        { 0x7c, 0x1e },
        { 0x7d, 0x35 },
        { 0x7e, 0x5a },
        { 0x7f, 0x69 },
        { 0x80, 0x76 },
        { 0x81, 0x80 },
        { 0x82, 0x88 },
        { 0x83, 0x8f },
        { 0x84, 0x96 },
        { 0x85, 0xa3 },
        { 0x86, 0xaf },
        { 0x87, 0xc4 },
        { 0x88, 0xd7 },
        { 0x89, 0xe8 },

/* AGC and AEC parameters.  Note we start by disabling those features,
   then turn them only after tweaking the values. */
        { OV7670_R13_COM8, OV7670_COM8_FASTAEC
                         | OV7670_COM8_AECSTEP
                         | OV7670_COM8_BFILT },
        { OV7670_R00_GAIN, 0x00 },
        { OV7670_R10_AECH, 0x00 },
        { OV7670_R0D_COM4, 0x40 }, /* magic reserved bit */
        { OV7670_R14_COM9, 0x18 }, /* 4x gain + magic rsvd bit */
        { OV7670_RA5_BD50MAX, 0x05 },
        { OV7670_RAB_BD60MAX, 0x07 },
        { OV7670_R24_AEW, 0x95 },
        { OV7670_R25_AEB, 0x33 },
        { OV7670_R26_VPT, 0xe3 },
        { OV7670_R9F_HAECC1, 0x78 },
        { OV7670_RA0_HAECC2, 0x68 },
        { 0xa1, 0x03 }, /* magic */
        { OV7670_RA6_HAECC3, 0xd8 },
        { OV7670_RA7_HAECC4, 0xd8 },
        { OV7670_RA8_HAECC5, 0xf0 },
        { OV7670_RA9_HAECC6, 0x90 },
        { OV7670_RAA_HAECC7, 0x94 },
        { OV7670_R13_COM8, OV7670_COM8_FASTAEC
                        | OV7670_COM8_AECSTEP
                        | OV7670_COM8_BFILT
                        | OV7670_COM8_AGC
                        | OV7670_COM8_AEC },

/* Almost all of these are magic "reserved" values.  */
        { OV7670_R0E_COM5, 0x61 },
        { OV7670_R0F_COM6, 0x4b },
        { 0x16, 0x02 },
        { OV7670_R1E_MVFP, 0x07 },
        { 0x21, 0x02 },
        { 0x22, 0x91 },
        { 0x29, 0x07 },
        { 0x33, 0x0b },
        { 0x35, 0x0b },
        { 0x37, 0x1d },
        { 0x38, 0x71 },
        { 0x39, 0x2a },
        { OV7670_R3C_COM12, 0x78 },
        { 0x4d, 0x40 },
        { 0x4e, 0x20 },
        { OV7670_R69_GFIX, 0x00 },
        { 0x6b, 0x4a },
        { 0x74, 0x10 },
        { 0x8d, 0x4f },
        { 0x8e, 0x00 },
        { 0x8f, 0x00 },
        { 0x90, 0x00 },
        { 0x91, 0x00 },
        { 0x96, 0x00 },
        { 0x9a, 0x00 },
        { 0xb0, 0x84 },
        { 0xb1, 0x0c },
        { 0xb2, 0x0e },
        { 0xb3, 0x82 },
        { 0xb8, 0x0a },

/* More reserved magic, some of which tweaks white balance */
        { 0x43, 0x0a },
        { 0x44, 0xf0 },
        { 0x45, 0x34 },
        { 0x46, 0x58 },
        { 0x47, 0x28 },
        { 0x48, 0x3a },
        { 0x59, 0x88 },
        { 0x5a, 0x88 },
        { 0x5b, 0x44 },
        { 0x5c, 0x67 },
        { 0x5d, 0x49 },
        { 0x5e, 0x0e },
        { 0x6c, 0x0a },
        { 0x6d, 0x55 },
        { 0x6e, 0x11 },
        { 0x6f, 0x9f },                 /* "9e for advance AWB" */
        { 0x6a, 0x40 },
        { OV7670_R01_BLUE, 0x40 },
        { OV7670_R02_RED, 0x60 },
        { OV7670_R13_COM8, OV7670_COM8_FASTAEC
                        | OV7670_COM8_AECSTEP
                        | OV7670_COM8_BFILT
                        | OV7670_COM8_AGC
                        | OV7670_COM8_AEC
                        | OV7670_COM8_AWB },

/* Matrix coefficients */
        { 0x4f, 0x80 },
        { 0x50, 0x80 },
        { 0x51, 0x00 },
        { 0x52, 0x22 },
        { 0x53, 0x5e },
        { 0x54, 0x80 },
        { 0x58, 0x9e },

        { OV7670_R41_COM16, OV7670_COM16_AWBGAIN },
        { OV7670_R3F_EDGE, 0x00 },
        { 0x75, 0x05 },
        { 0x76, 0xe1 },
        { 0x4c, 0x00 },
        { 0x77, 0x01 },
        { OV7670_R3D_COM13, OV7670_COM13_GAMMA
                          | OV7670_COM13_UVSAT
                          | 2},         /* was 3 */
        { 0x4b, 0x09 },
        { 0xc9, 0x60 },
        { OV7670_R41_COM16, 0x38 },
        { 0x56, 0x40 },

        { 0x34, 0x11 },
        { OV7670_R3B_COM11, OV7670_COM11_EXP|OV7670_COM11_HZAUTO },
        { 0xa4, 0x88 },
        { 0x96, 0x00 },
        { 0x97, 0x30 },
        { 0x98, 0x20 },
        { 0x99, 0x30 },
        { 0x9a, 0x84 },
        { 0x9b, 0x29 },
        { 0x9c, 0x03 },
        { 0x9d, 0x4c },
        { 0x9e, 0x3f },
        { 0x78, 0x04 },

/* Extra-weird stuff.  Some sort of multiplexor register */
        { 0x79, 0x01 },
        { 0xc8, 0xf0 },
        { 0x79, 0x0f },
        { 0xc8, 0x00 },
        { 0x79, 0x10 },
        { 0xc8, 0x7e },
        { 0x79, 0x0a },
        { 0xc8, 0x80 },
        { 0x79, 0x0b },
        { 0xc8, 0x01 },
        { 0x79, 0x0c },
        { 0xc8, 0x0f },
        { 0x79, 0x0d },
        { 0xc8, 0x20 },
        { 0x79, 0x09 },
        { 0xc8, 0x80 },
        { 0x79, 0x02 },
        { 0xc8, 0xc0 },
        { 0x79, 0x03 },
        { 0xc8, 0x40 },
        { 0x79, 0x05 },
        { 0xc8, 0x30 },
        { 0x79, 0x26 },
};

static const struct ov_i2c_regvals norm_8610[] = {
        { 0x12, 0x80 },
        { 0x00, 0x00 },
        { 0x01, 0x80 },
        { 0x02, 0x80 },
        { 0x03, 0xc0 },
        { 0x04, 0x30 },
        { 0x05, 0x30 }, /* was 0x10, new from windrv 090403 */
        { 0x06, 0x70 }, /* was 0x80, new from windrv 090403 */
        { 0x0a, 0x86 },
        { 0x0b, 0xb0 },
        { 0x0c, 0x20 },
        { 0x0d, 0x20 },
        { 0x11, 0x01 },
        { 0x12, 0x25 },
        { 0x13, 0x01 },
        { 0x14, 0x04 },
        { 0x15, 0x01 }, /* Lin and Win think different about UV order */
        { 0x16, 0x03 },
        { 0x17, 0x38 }, /* was 0x2f, new from windrv 090403 */
        { 0x18, 0xea }, /* was 0xcf, new from windrv 090403 */
        { 0x19, 0x02 }, /* was 0x06, new from windrv 090403 */
        { 0x1a, 0xf5 },
        { 0x1b, 0x00 },
        { 0x20, 0xd0 }, /* was 0x90, new from windrv 090403 */
        { 0x23, 0xc0 }, /* was 0x00, new from windrv 090403 */
        { 0x24, 0x30 }, /* was 0x1d, new from windrv 090403 */
        { 0x25, 0x50 }, /* was 0x57, new from windrv 090403 */
        { 0x26, 0xa2 },
        { 0x27, 0xea },
        { 0x28, 0x00 },
        { 0x29, 0x00 },
        { 0x2a, 0x80 },
        { 0x2b, 0xc8 }, /* was 0xcc, new from windrv 090403 */
        { 0x2c, 0xac },
        { 0x2d, 0x45 }, /* was 0xd5, new from windrv 090403 */
        { 0x2e, 0x80 },
        { 0x2f, 0x14 }, /* was 0x01, new from windrv 090403 */
        { 0x4c, 0x00 },
        { 0x4d, 0x30 }, /* was 0x10, new from windrv 090403 */
        { 0x60, 0x02 }, /* was 0x01, new from windrv 090403 */
        { 0x61, 0x00 }, /* was 0x09, new from windrv 090403 */
        { 0x62, 0x5f }, /* was 0xd7, new from windrv 090403 */
        { 0x63, 0xff },
        { 0x64, 0x53 }, /* new windrv 090403 says 0x57,
                         * maybe thats wrong */
        { 0x65, 0x00 },
        { 0x66, 0x55 },
        { 0x67, 0xb0 },
        { 0x68, 0xc0 }, /* was 0xaf, new from windrv 090403 */
        { 0x69, 0x02 },
        { 0x6a, 0x22 },
        { 0x6b, 0x00 },
        { 0x6c, 0x99 }, /* was 0x80, old windrv says 0x00, but
                         * deleting bit7 colors the first images red */
        { 0x6d, 0x11 }, /* was 0x00, new from windrv 090403 */
        { 0x6e, 0x11 }, /* was 0x00, new from windrv 090403 */
        { 0x6f, 0x01 },
        { 0x70, 0x8b },
        { 0x71, 0x00 },
        { 0x72, 0x14 },
        { 0x73, 0x54 },
        { 0x74, 0x00 },/* 0x60? - was 0x00, new from windrv 090403 */
        { 0x75, 0x0e },
        { 0x76, 0x02 }, /* was 0x02, new from windrv 090403 */
        { 0x77, 0xff },
        { 0x78, 0x80 },
        { 0x79, 0x80 },
        { 0x7a, 0x80 },
        { 0x7b, 0x10 }, /* was 0x13, new from windrv 090403 */
        { 0x7c, 0x00 },
        { 0x7d, 0x08 }, /* was 0x09, new from windrv 090403 */
        { 0x7e, 0x08 }, /* was 0xc0, new from windrv 090403 */
        { 0x7f, 0xfb },
        { 0x80, 0x28 },
        { 0x81, 0x00 },
        { 0x82, 0x23 },
        { 0x83, 0x0b },
        { 0x84, 0x00 },
        { 0x85, 0x62 }, /* was 0x61, new from windrv 090403 */
        { 0x86, 0xc9 },
        { 0x87, 0x00 },
        { 0x88, 0x00 },
        { 0x89, 0x01 },
        { 0x12, 0x20 },
        { 0x12, 0x25 }, /* was 0x24, new from windrv 090403 */
};

static unsigned char ov7670_abs_to_sm(unsigned char v)
{
        if (v > 127)
                return v & 0x7f;
        return (128 - v) | 0x80;
}

/* Write a OV519 register */
static void reg_w(struct sd *sd, u16 index, u16 value)
{
        int ret, req = 0;

        if (sd->gspca_dev.usb_err < 0)
                return;

        switch (sd->bridge) {
        case BRIDGE_OV511:
        case BRIDGE_OV511PLUS:
                req = 2;
                break;
        case BRIDGE_OVFX2:
                req = 0x0a;
                /* fall through */
        case BRIDGE_W9968CF:
                PDEBUG(D_USBO, "SET %02x %04x %04x",
                                req, value, index);
                ret = usb_control_msg(sd->gspca_dev.dev,
                        usb_sndctrlpipe(sd->gspca_dev.dev, 0),
                        req,
                        USB_DIR_OUT | USB_TYPE_VENDOR | USB_RECIP_DEVICE,
                        value, index, NULL, 0, 500);
                goto leave;
        default:
                req = 1;
        }

        PDEBUG(D_USBO, "SET %02x 0000 %04x %02x",
                        req, index, value);
        sd->gspca_dev.usb_buf[0] = value;
        ret = usb_control_msg(sd->gspca_dev.dev,
                        usb_sndctrlpipe(sd->gspca_dev.dev, 0),
                        req,
                        USB_DIR_OUT | USB_TYPE_VENDOR | USB_RECIP_DEVICE,
                        0, index,
                        sd->gspca_dev.usb_buf, 1, 500);
leave:
        if (ret < 0) {
                pr_err("reg_w %02x failed %d\n", index, ret);
                sd->gspca_dev.usb_err = ret;
                return;
        }
}

/* Read from a OV519 register, note not valid for the w9968cf!! */
/* returns: negative is error, pos or zero is data */
static int reg_r(struct sd *sd, u16 index)
{
        int ret;
        int req;

        if (sd->gspca_dev.usb_err < 0)
                return -1;

        switch (sd->bridge) {
        case BRIDGE_OV511:
        case BRIDGE_OV511PLUS:
                req = 3;
                break;
        case BRIDGE_OVFX2:
                req = 0x0b;
                break;
        default:
                req = 1;
        }

        ret = usb_control_msg(sd->gspca_dev.dev,
                        usb_rcvctrlpipe(sd->gspca_dev.dev, 0),
                        req,
                        USB_DIR_IN | USB_TYPE_VENDOR | USB_RECIP_DEVICE,
                        0, index, sd->gspca_dev.usb_buf, 1, 500);

        if (ret >= 0) {
                ret = sd->gspca_dev.usb_buf[0];
                PDEBUG(D_USBI, "GET %02x 0000 %04x %02x",
                        req, index, ret);
        } else {
                pr_err("reg_r %02x failed %d\n", index, ret);
                sd->gspca_dev.usb_err = ret;
        }

        return ret;
}

/* Read 8 values from a OV519 register */
static int reg_r8(struct sd *sd,
                  u16 index)
{
        int ret;

        if (sd->gspca_dev.usb_err < 0)
                return -1;

        ret = usb_control_msg(sd->gspca_dev.dev,
                        usb_rcvctrlpipe(sd->gspca_dev.dev, 0),
                        1,                      /* REQ_IO */
                        USB_DIR_IN | USB_TYPE_VENDOR | USB_RECIP_DEVICE,
                        0, index, sd->gspca_dev.usb_buf, 8, 500);

        if (ret >= 0) {
                ret = sd->gspca_dev.usb_buf[0];
        } else {
                pr_err("reg_r8 %02x failed %d\n", index, ret);
                sd->gspca_dev.usb_err = ret;
        }

        return ret;
}

/*
 * Writes bits at positions specified by mask to an OV51x reg. Bits that are in
 * the same position as 1's in "mask" are cleared and set to "value". Bits
 * that are in the same position as 0's in "mask" are preserved, regardless
 * of their respective state in "value".
 */
static void reg_w_mask(struct sd *sd,
                        u16 index,
                        u8 value,
                        u8 mask)
{
        int ret;
        u8 oldval;

        if (mask != 0xff) {
                value &= mask;                  /* Enforce mask on value */
                ret = reg_r(sd, index);
                if (ret < 0)
                        return;

                oldval = ret & ~mask;           /* Clear the masked bits */
                value |= oldval;                /* Set the desired bits */
        }
        reg_w(sd, index, value);
}

/*
 * Writes multiple (n) byte value to a single register. Only valid with certain
 * registers (0x30 and 0xc4 - 0xce).
 */
static void ov518_reg_w32(struct sd *sd, u16 index, u32 value, int n)
{
        int ret;

        if (sd->gspca_dev.usb_err < 0)
                return;

        *((__le32 *) sd->gspca_dev.usb_buf) = __cpu_to_le32(value);

        ret = usb_control_msg(sd->gspca_dev.dev,
                        usb_sndctrlpipe(sd->gspca_dev.dev, 0),
                        1 /* REG_IO */,
                        USB_DIR_OUT | USB_TYPE_VENDOR | USB_RECIP_DEVICE,
                        0, index,
                        sd->gspca_dev.usb_buf, n, 500);
        if (ret < 0) {
                pr_err("reg_w32 %02x failed %d\n", index, ret);
                sd->gspca_dev.usb_err = ret;
        }
}

static void ov511_i2c_w(struct sd *sd, u8 reg, u8 value)
{
        int rc, retries;

        PDEBUG(D_USBO, "ov511_i2c_w %02x %02x", reg, value);

        /* Three byte write cycle */
        for (retries = 6; ; ) {
                /* Select camera register */
                reg_w(sd, R51x_I2C_SADDR_3, reg);

                /* Write "value" to I2C data port of OV511 */
                reg_w(sd, R51x_I2C_DATA, value);

                /* Initiate 3-byte write cycle */
                reg_w(sd, R511_I2C_CTL, 0x01);

                do {
                        rc = reg_r(sd, R511_I2C_CTL);
                } while (rc > 0 && ((rc & 1) == 0)); /* Retry until idle */

                if (rc < 0)
                        return;

                if ((rc & 2) == 0) /* Ack? */
                        break;
                if (--retries < 0) {
                        PDEBUG(D_USBO, "i2c write retries exhausted");
                        return;
                }
        }
}

static int ov511_i2c_r(struct sd *sd, u8 reg)
{
        int rc, value, retries;

        /* Two byte write cycle */
        for (retries = 6; ; ) {
                /* Select camera register */
                reg_w(sd, R51x_I2C_SADDR_2, reg);

                /* Initiate 2-byte write cycle */
                reg_w(sd, R511_I2C_CTL, 0x03);

                do {
                        rc = reg_r(sd, R511_I2C_CTL);
                } while (rc > 0 && ((rc & 1) == 0)); /* Retry until idle */

                if (rc < 0)
                        return rc;

                if ((rc & 2) == 0) /* Ack? */
                        break;

                /* I2C abort */
                reg_w(sd, R511_I2C_CTL, 0x10);

                if (--retries < 0) {
                        PDEBUG(D_USBI, "i2c write retries exhausted");
                        return -1;
                }
        }

        /* Two byte read cycle */
        for (retries = 6; ; ) {
                /* Initiate 2-byte read cycle */
                reg_w(sd, R511_I2C_CTL, 0x05);

                do {
                        rc = reg_r(sd, R511_I2C_CTL);
                } while (rc > 0 && ((rc & 1) == 0)); /* Retry until idle */

                if (rc < 0)
                        return rc;

                if ((rc & 2) == 0) /* Ack? */
                        break;

                /* I2C abort */
                reg_w(sd, R511_I2C_CTL, 0x10);

                if (--retries < 0) {
                        PDEBUG(D_USBI, "i2c read retries exhausted");
                        return -1;
                }
        }

        value = reg_r(sd, R51x_I2C_DATA);

        PDEBUG(D_USBI, "ov511_i2c_r %02x %02x", reg, value);

        /* This is needed to make i2c_w() work */
        reg_w(sd, R511_I2C_CTL, 0x05);

        return value;
}

/*
 * The OV518 I2C I/O procedure is different, hence, this function.
 * This is normally only called from i2c_w(). Note that this function
 * always succeeds regardless of whether the sensor is present and working.
 */
static void ov518_i2c_w(struct sd *sd,
                u8 reg,
                u8 value)
{
        PDEBUG(D_USBO, "ov518_i2c_w %02x %02x", reg, value);

        /* Select camera register */
        reg_w(sd, R51x_I2C_SADDR_3, reg);

        /* Write "value" to I2C data port of OV511 */
        reg_w(sd, R51x_I2C_DATA, value);

        /* Initiate 3-byte write cycle */
        reg_w(sd, R518_I2C_CTL, 0x01);

        /* wait for write complete */
        msleep(4);
        reg_r8(sd, R518_I2C_CTL);
}

/*
 * returns: negative is error, pos or zero is data
 *
 * The OV518 I2C I/O procedure is different, hence, this function.
 * This is normally only called from i2c_r(). Note that this function
 * always succeeds regardless of whether the sensor is present and working.
 */
static int ov518_i2c_r(struct sd *sd, u8 reg)
{
        int value;

        /* Select camera register */
        reg_w(sd, R51x_I2C_SADDR_2, reg);

        /* Initiate 2-byte write cycle */
        reg_w(sd, R518_I2C_CTL, 0x03);
        reg_r8(sd, R518_I2C_CTL);

        /* Initiate 2-byte read cycle */
        reg_w(sd, R518_I2C_CTL, 0x05);
        reg_r8(sd, R518_I2C_CTL);

        value = reg_r(sd, R51x_I2C_DATA);
        PDEBUG(D_USBI, "ov518_i2c_r %02x %02x", reg, value);
        return value;
}

static void ovfx2_i2c_w(struct sd *sd, u8 reg, u8 value)
{
        int ret;

        if (sd->gspca_dev.usb_err < 0)
                return;

        ret = usb_control_msg(sd->gspca_dev.dev,
                        usb_sndctrlpipe(sd->gspca_dev.dev, 0),
                        0x02,
                        USB_DIR_OUT | USB_TYPE_VENDOR | USB_RECIP_DEVICE,
                        (u16) value, (u16) reg, NULL, 0, 500);

        if (ret < 0) {
                pr_err("ovfx2_i2c_w %02x failed %d\n", reg, ret);
                sd->gspca_dev.usb_err = ret;
        }

        PDEBUG(D_USBO, "ovfx2_i2c_w %02x %02x", reg, value);
}

static int ovfx2_i2c_r(struct sd *sd, u8 reg)
{
        int ret;

        if (sd->gspca_dev.usb_err < 0)
                return -1;

        ret = usb_control_msg(sd->gspca_dev.dev,
                        usb_rcvctrlpipe(sd->gspca_dev.dev, 0),
                        0x03,
                        USB_DIR_IN | USB_TYPE_VENDOR | USB_RECIP_DEVICE,
                        0, (u16) reg, sd->gspca_dev.usb_buf, 1, 500);

        if (ret >= 0) {
                ret = sd->gspca_dev.usb_buf[0];
                PDEBUG(D_USBI, "ovfx2_i2c_r %02x %02x", reg, ret);
        } else {
                pr_err("ovfx2_i2c_r %02x failed %d\n", reg, ret);
                sd->gspca_dev.usb_err = ret;
        }

        return ret;
}

static void i2c_w(struct sd *sd, u8 reg, u8 value)
{
        if (sd->sensor_reg_cache[reg] == value)
                return;

        switch (sd->bridge) {
        case BRIDGE_OV511:
        case BRIDGE_OV511PLUS:
                ov511_i2c_w(sd, reg, value);
                break;
        case BRIDGE_OV518:
        case BRIDGE_OV518PLUS:
        case BRIDGE_OV519:
                ov518_i2c_w(sd, reg, value);
                break;
        case BRIDGE_OVFX2:
                ovfx2_i2c_w(sd, reg, value);
                break;
        case BRIDGE_W9968CF:
                w9968cf_i2c_w(sd, reg, value);
                break;
        }

        if (sd->gspca_dev.usb_err >= 0) {
                /* Up on sensor reset empty the register cache */
                if (reg == 0x12 && (value & 0x80))
                        memset(sd->sensor_reg_cache, -1,
                                sizeof(sd->sensor_reg_cache));
                else
                        sd->sensor_reg_cache[reg] = value;
        }
}

static int i2c_r(struct sd *sd, u8 reg)
{
        int ret = -1;

        if (sd->sensor_reg_cache[reg] != -1)
                return sd->sensor_reg_cache[reg];

        switch (sd->bridge) {
        case BRIDGE_OV511:
        case BRIDGE_OV511PLUS:
                ret = ov511_i2c_r(sd, reg);
                break;
        case BRIDGE_OV518:
        case BRIDGE_OV518PLUS:
        case BRIDGE_OV519:
                ret = ov518_i2c_r(sd, reg);
                break;
        case BRIDGE_OVFX2:
                ret = ovfx2_i2c_r(sd, reg);
                break;
        case BRIDGE_W9968CF:
                ret = w9968cf_i2c_r(sd, reg);
                break;
        }

        if (ret >= 0)
                sd->sensor_reg_cache[reg] = ret;

        return ret;
}

/* Writes bits at positions specified by mask to an I2C reg. Bits that are in
 * the same position as 1's in "mask" are cleared and set to "value". Bits
 * that are in the same position as 0's in "mask" are preserved, regardless
 * of their respective state in "value".
 */
static void i2c_w_mask(struct sd *sd,
                        u8 reg,
                        u8 value,
                        u8 mask)
{
        int rc;
        u8 oldval;

        value &= mask;                  /* Enforce mask on value */
        rc = i2c_r(sd, reg);
        if (rc < 0)
                return;
        oldval = rc & ~mask;            /* Clear the masked bits */
        value |= oldval;                /* Set the desired bits */
        i2c_w(sd, reg, value);
}

/* Temporarily stops OV511 from functioning. Must do this before changing
 * registers while the camera is streaming */
static inline void ov51x_stop(struct sd *sd)
{
        PDEBUG(D_STREAM, "stopping");
        sd->stopped = 1;
        switch (sd->bridge) {
        case BRIDGE_OV511:
        case BRIDGE_OV511PLUS:
                reg_w(sd, R51x_SYS_RESET, 0x3d);
                break;
        case BRIDGE_OV518:
        case BRIDGE_OV518PLUS:
                reg_w_mask(sd, R51x_SYS_RESET, 0x3a, 0x3a);
                break;
        case BRIDGE_OV519:
                reg_w(sd, OV519_R51_RESET1, 0x0f);
                reg_w(sd, OV519_R51_RESET1, 0x00);
                reg_w(sd, 0x22, 0x00);          /* FRAR */
                break;
        case BRIDGE_OVFX2:
                reg_w_mask(sd, 0x0f, 0x00, 0x02);
                break;
        case BRIDGE_W9968CF:
                reg_w(sd, 0x3c, 0x0a05); /* stop USB transfer */
                break;
        }
}

/* Restarts OV511 after ov511_stop() is called. Has no effect if it is not
 * actually stopped (for performance). */
static inline void ov51x_restart(struct sd *sd)
{
        PDEBUG(D_STREAM, "restarting");
        if (!sd->stopped)
                return;
        sd->stopped = 0;

        /* Reinitialize the stream */
        switch (sd->bridge) {
        case BRIDGE_OV511:
        case BRIDGE_OV511PLUS:
                reg_w(sd, R51x_SYS_RESET, 0x00);
                break;
        case BRIDGE_OV518:
        case BRIDGE_OV518PLUS:
                reg_w(sd, 0x2f, 0x80);
                reg_w(sd, R51x_SYS_RESET, 0x00);
                break;
        case BRIDGE_OV519:
                reg_w(sd, OV519_R51_RESET1, 0x0f);
                reg_w(sd, OV519_R51_RESET1, 0x00);
                reg_w(sd, 0x22, 0x1d);          /* FRAR */
                break;
        case BRIDGE_OVFX2:
                reg_w_mask(sd, 0x0f, 0x02, 0x02);
                break;
        case BRIDGE_W9968CF:
                reg_w(sd, 0x3c, 0x8a05); /* USB FIFO enable */
                break;
        }
}

static void ov51x_set_slave_ids(struct sd *sd, u8 slave);

/* This does an initial reset of an OmniVision sensor and ensures that I2C
 * is synchronized. Returns <0 on failure.
 */
static int init_ov_sensor(struct sd *sd, u8 slave)
{
        int i;

        ov51x_set_slave_ids(sd, slave);

        /* Reset the sensor */
        i2c_w(sd, 0x12, 0x80);

        /* Wait for it to initialize */
        msleep(150);

        for (i = 0; i < i2c_detect_tries; i++) {
                if (i2c_r(sd, OV7610_REG_ID_HIGH) == 0x7f &&
                    i2c_r(sd, OV7610_REG_ID_LOW) == 0xa2) {
                        PDEBUG(D_PROBE, "I2C synced in %d attempt(s)", i);
                        return 0;
                }

                /* Reset the sensor */
                i2c_w(sd, 0x12, 0x80);

                /* Wait for it to initialize */
                msleep(150);

                /* Dummy read to sync I2C */
                if (i2c_r(sd, 0x00) < 0)
                        return -1;
        }
        return -1;
}

/* Set the read and write slave IDs. The "slave" argument is the write slave,
 * and the read slave will be set to (slave + 1).
 * This should not be called from outside the i2c I/O functions.
 * Sets I2C read and write slave IDs. Returns <0 for error
 */
static void ov51x_set_slave_ids(struct sd *sd,
                                u8 slave)
{
        switch (sd->bridge) {
        case BRIDGE_OVFX2:
                reg_w(sd, OVFX2_I2C_ADDR, slave);
                return;
        case BRIDGE_W9968CF:
                sd->sensor_addr = slave;
                return;
        }

        reg_w(sd, R51x_I2C_W_SID, slave);
        reg_w(sd, R51x_I2C_R_SID, slave + 1);
}

static void write_regvals(struct sd *sd,
                         const struct ov_regvals *regvals,
                         int n)
{
        while (--n >= 0) {
                reg_w(sd, regvals->reg, regvals->val);
                regvals++;
        }
}

static void write_i2c_regvals(struct sd *sd,
                        const struct ov_i2c_regvals *regvals,
                        int n)
{
        while (--n >= 0) {
                i2c_w(sd, regvals->reg, regvals->val);
                regvals++;
        }
}

/****************************************************************************
 *
 * OV511 and sensor configuration
 *
 ***************************************************************************/

/* This initializes the OV2x10 / OV3610 / OV3620 / OV9600 */
static void ov_hires_configure(struct sd *sd)
{
        int high, low;

        if (sd->bridge != BRIDGE_OVFX2) {
                pr_err("error hires sensors only supported with ovfx2\n");
                return;
        }

        PDEBUG(D_PROBE, "starting ov hires configuration");

        /* Detect sensor (sub)type */
        high = i2c_r(sd, 0x0a);
        low = i2c_r(sd, 0x0b);
        /* info("%x, %x", high, low); */
        switch (high) {
        case 0x96:
                switch (low) {
                case 0x40:
                        PDEBUG(D_PROBE, "Sensor is a OV2610");
                        sd->sensor = SEN_OV2610;
                        return;
                case 0x41:
                        PDEBUG(D_PROBE, "Sensor is a OV2610AE");
                        sd->sensor = SEN_OV2610AE;
                        return;
                case 0xb1:
                        PDEBUG(D_PROBE, "Sensor is a OV9600");
                        sd->sensor = SEN_OV9600;
                        return;
                }
                break;
        case 0x36:
                if ((low & 0x0f) == 0x00) {
                        PDEBUG(D_PROBE, "Sensor is a OV3610");
                        sd->sensor = SEN_OV3610;
                        return;
                }
                break;
        }
        pr_err("Error unknown sensor type: %02x%02x\n", high, low);
}

/* This initializes the OV8110, OV8610 sensor. The OV8110 uses
 * the same register settings as the OV8610, since they are very similar.
 */
static void ov8xx0_configure(struct sd *sd)
{
        int rc;

        PDEBUG(D_PROBE, "starting ov8xx0 configuration");

        /* Detect sensor (sub)type */
        rc = i2c_r(sd, OV7610_REG_COM_I);
        if (rc < 0) {
                PDEBUG(D_ERR, "Error detecting sensor type");
                return;
        }
        if ((rc & 3) == 1)
                sd->sensor = SEN_OV8610;
        else
                pr_err("Unknown image sensor version: %d\n", rc & 3);
}

/* This initializes the OV7610, OV7620, or OV76BE sensor. The OV76BE uses
 * the same register settings as the OV7610, since they are very similar.
 */
static void ov7xx0_configure(struct sd *sd)
{
        int rc, high, low;

        PDEBUG(D_PROBE, "starting OV7xx0 configuration");

        /* Detect sensor (sub)type */
        rc = i2c_r(sd, OV7610_REG_COM_I);

        /* add OV7670 here
         * it appears to be wrongly detected as a 7610 by default */
        if (rc < 0) {
                PDEBUG(D_ERR, "Error detecting sensor type");
                return;
        }
        if ((rc & 3) == 3) {
                /* quick hack to make OV7670s work */
                high = i2c_r(sd, 0x0a);
                low = i2c_r(sd, 0x0b);
                /* info("%x, %x", high, low); */
                if (high == 0x76 && (low & 0xf0) == 0x70) {
                        PDEBUG(D_PROBE, "Sensor is an OV76%02x", low);
                        sd->sensor = SEN_OV7670;
                } else {
                        PDEBUG(D_PROBE, "Sensor is an OV7610");
                        sd->sensor = SEN_OV7610;
                }
        } else if ((rc & 3) == 1) {
                /* I don't know what's different about the 76BE yet. */
                if (i2c_r(sd, 0x15) & 1) {
                        PDEBUG(D_PROBE, "Sensor is an OV7620AE");
                        sd->sensor = SEN_OV7620AE;
                } else {
                        PDEBUG(D_PROBE, "Sensor is an OV76BE");
                        sd->sensor = SEN_OV76BE;
                }
        } else if ((rc & 3) == 0) {
                /* try to read product id registers */
                high = i2c_r(sd, 0x0a);
                if (high < 0) {
                        PDEBUG(D_ERR, "Error detecting camera chip PID");
                        return;
                }
                low = i2c_r(sd, 0x0b);
                if (low < 0) {
                        PDEBUG(D_ERR, "Error detecting camera chip VER");
                        return;
                }
                if (high == 0x76) {
                        switch (low) {
                        case 0x30:
                                pr_err("Sensor is an OV7630/OV7635\n");
                                pr_err("7630 is not supported by this driver\n");
                                return;
                        case 0x40:
                                PDEBUG(D_PROBE, "Sensor is an OV7645");
                                sd->sensor = SEN_OV7640; /* FIXME */
                                break;
                        case 0x45:
                                PDEBUG(D_PROBE, "Sensor is an OV7645B");
                                sd->sensor = SEN_OV7640; /* FIXME */
                                break;
                        case 0x48:
                                PDEBUG(D_PROBE, "Sensor is an OV7648");
                                sd->sensor = SEN_OV7648;
                                break;
                        case 0x60:
                                PDEBUG(D_PROBE, "Sensor is a OV7660");
                                sd->sensor = SEN_OV7660;
                                break;
                        default:
                                PDEBUG(D_PROBE, "Unknown sensor: 0x76%x", low);
                                return;
                        }
                } else {
                        PDEBUG(D_PROBE, "Sensor is an OV7620");
                        sd->sensor = SEN_OV7620;
                }
        } else {
                pr_err("Unknown image sensor version: %d\n", rc & 3);
        }
}

/* This initializes the OV6620, OV6630, OV6630AE, or OV6630AF sensor. */
static void ov6xx0_configure(struct sd *sd)
{
        int rc;
        PDEBUG(D_PROBE, "starting OV6xx0 configuration");

        /* Detect sensor (sub)type */
        rc = i2c_r(sd, OV7610_REG_COM_I);
        if (rc < 0) {
                PDEBUG(D_ERR, "Error detecting sensor type");
                return;
        }

        /* Ugh. The first two bits are the version bits, but
         * the entire register value must be used. I guess OVT
         * underestimated how many variants they would make. */
        switch (rc) {
        case 0x00:
                sd->sensor = SEN_OV6630;
                pr_warn("WARNING: Sensor is an OV66308. Your camera may have been misdetected in previous driver versions.\n");
                break;
        case 0x01:
                sd->sensor = SEN_OV6620;
                PDEBUG(D_PROBE, "Sensor is an OV6620");
                break;
        case 0x02:
                sd->sensor = SEN_OV6630;
                PDEBUG(D_PROBE, "Sensor is an OV66308AE");
                break;
        case 0x03:
                sd->sensor = SEN_OV66308AF;
                PDEBUG(D_PROBE, "Sensor is an OV66308AF");
                break;
        case 0x90:
                sd->sensor = SEN_OV6630;
                pr_warn("WARNING: Sensor is an OV66307. Your camera may have been misdetected in previous driver versions.\n");
                break;
        default:
                pr_err("FATAL: Unknown sensor version: 0x%02x\n", rc);
                return;
        }

        /* Set sensor-specific vars */
        sd->sif = 1;
}

/* Turns on or off the LED. Only has an effect with OV511+/OV518(+)/OV519 */
static void ov51x_led_control(struct sd *sd, int on)
{
        if (sd->invert_led)
                on = !on;

        switch (sd->bridge) {
        /* OV511 has no LED control */
        case BRIDGE_OV511PLUS:
                reg_w(sd, R511_SYS_LED_CTL, on);
                break;
        case BRIDGE_OV518:
        case BRIDGE_OV518PLUS:
                reg_w_mask(sd, R518_GPIO_OUT, 0x02 * on, 0x02);
                break;
        case BRIDGE_OV519:
                reg_w_mask(sd, OV519_GPIO_DATA_OUT0, on, 1);
                break;
        }
}

static void sd_reset_snapshot(struct gspca_dev *gspca_dev)
{
        struct sd *sd = (struct sd *) gspca_dev;

        if (!sd->snapshot_needs_reset)
                return;

        /* Note it is important that we clear sd->snapshot_needs_reset,
           before actually clearing the snapshot state in the bridge
           otherwise we might race with the pkt_scan interrupt handler */
        sd->snapshot_needs_reset = 0;

        switch (sd->bridge) {
        case BRIDGE_OV511:
        case BRIDGE_OV511PLUS:
                reg_w(sd, R51x_SYS_SNAP, 0x02);
                reg_w(sd, R51x_SYS_SNAP, 0x00);
                break;
        case BRIDGE_OV518:
        case BRIDGE_OV518PLUS:
                reg_w(sd, R51x_SYS_SNAP, 0x02); /* Reset */
                reg_w(sd, R51x_SYS_SNAP, 0x01); /* Enable */
                break;
        case BRIDGE_OV519:
                reg_w(sd, R51x_SYS_RESET, 0x40);
                reg_w(sd, R51x_SYS_RESET, 0x00);
                break;
        }
}

static void ov51x_upload_quan_tables(struct sd *sd)
{
        const unsigned char yQuanTable511[] = {
                0, 1, 1, 2, 2, 3, 3, 4,
                1, 1, 1, 2, 2, 3, 4, 4,
                1, 1, 2, 2, 3, 4, 4, 4,
                2, 2, 2, 3, 4, 4, 4, 4,
                2, 2, 3, 4, 4, 5, 5, 5,
                3, 3, 4, 4, 5, 5, 5, 5,
                3, 4, 4, 4, 5, 5, 5, 5,
                4, 4, 4, 4, 5, 5, 5, 5
        };

        const unsigned char uvQuanTable511[] = {
                0, 2, 2, 3, 4, 4, 4, 4,
                2, 2, 2, 4, 4, 4, 4, 4,
                2, 2, 3, 4, 4, 4, 4, 4,
                3, 4, 4, 4, 4, 4, 4, 4,
                4, 4, 4, 4, 4, 4, 4, 4,
                4, 4, 4, 4, 4, 4, 4, 4,
                4, 4, 4, 4, 4, 4, 4, 4,
                4, 4, 4, 4, 4, 4, 4, 4
        };

        /* OV518 quantization tables are 8x4 (instead of 8x8) */
        const unsigned char yQuanTable518[] = {

                5, 4, 5, 6, 6, 7, 7, 7,
                5, 5, 5, 5, 6, 7, 7, 7,
                6, 6, 6, 6, 7, 7, 7, 8,
                7, 7, 6, 7, 7, 7, 8, 8
        };
        const unsigned char uvQuanTable518[] = {
                6, 6, 6, 7, 7, 7, 7, 7,
                6, 6, 6, 7, 7, 7, 7, 7,
                6, 6, 6, 7, 7, 7, 7, 8,
                7, 7, 7, 7, 7, 7, 8, 8
        };

        const unsigned char *pYTable, *pUVTable;
        unsigned char val0, val1;
        int i, size, reg = R51x_COMP_LUT_BEGIN;

        PDEBUG(D_PROBE, "Uploading quantization tables");

        if (sd->bridge == BRIDGE_OV511 || sd->bridge == BRIDGE_OV511PLUS) {
                pYTable = yQuanTable511;
                pUVTable = uvQuanTable511;
                size = 32;
        } else {
                pYTable = yQuanTable518;
                pUVTable = uvQuanTable518;
                size = 16;
        }

        for (i = 0; i < size; i++) {
                val0 = *pYTable++;
                val1 = *pYTable++;
                val0 &= 0x0f;
                val1 &= 0x0f;
                val0 |= val1 << 4;
                reg_w(sd, reg, val0);

                val0 = *pUVTable++;
                val1 = *pUVTable++;
                val0 &= 0x0f;
                val1 &= 0x0f;
                val0 |= val1 << 4;
                reg_w(sd, reg + size, val0);

                reg++;
        }
}

/* This initializes the OV511/OV511+ and the sensor */
static void ov511_configure(struct gspca_dev *gspca_dev)
{
        struct sd *sd = (struct sd *) gspca_dev;

        /* For 511 and 511+ */
        const struct ov_regvals init_511[] = {
                { R51x_SYS_RESET,       0x7f },
                { R51x_SYS_INIT,        0x01 },
                { R51x_SYS_RESET,       0x7f },
                { R51x_SYS_INIT,        0x01 },
                { R51x_SYS_RESET,       0x3f },
                { R51x_SYS_INIT,        0x01 },
                { R51x_SYS_RESET,       0x3d },
        };

        const struct ov_regvals norm_511[] = {
                { R511_DRAM_FLOW_CTL,   0x01 },
                { R51x_SYS_SNAP,        0x00 },
                { R51x_SYS_SNAP,        0x02 },
                { R51x_SYS_SNAP,        0x00 },
                { R511_FIFO_OPTS,       0x1f },
                { R511_COMP_EN,         0x00 },
                { R511_COMP_LUT_EN,     0x03 },
        };

        const struct ov_regvals norm_511_p[] = {
                { R511_DRAM_FLOW_CTL,   0xff },
                { R51x_SYS_SNAP,        0x00 },
                { R51x_SYS_SNAP,        0x02 },
                { R51x_SYS_SNAP,        0x00 },
                { R511_FIFO_OPTS,       0xff },
                { R511_COMP_EN,         0x00 },
                { R511_COMP_LUT_EN,     0x03 },
        };

        const struct ov_regvals compress_511[] = {
                { 0x70, 0x1f },
                { 0x71, 0x05 },
                { 0x72, 0x06 },
                { 0x73, 0x06 },
                { 0x74, 0x14 },
                { 0x75, 0x03 },
                { 0x76, 0x04 },
                { 0x77, 0x04 },
        };

        PDEBUG(D_PROBE, "Device custom id %x", reg_r(sd, R51x_SYS_CUST_ID));

        write_regvals(sd, init_511, ARRAY_SIZE(init_511));

        switch (sd->bridge) {
        case BRIDGE_OV511:
                write_regvals(sd, norm_511, ARRAY_SIZE(norm_511));
                break;
        case BRIDGE_OV511PLUS:
                write_regvals(sd, norm_511_p, ARRAY_SIZE(norm_511_p));
                break;
        }

        /* Init compression */
        write_regvals(sd, compress_511, ARRAY_SIZE(compress_511));

        ov51x_upload_quan_tables(sd);
}

/* This initializes the OV518/OV518+ and the sensor */
static void ov518_configure(struct gspca_dev *gspca_dev)
{
        struct sd *sd = (struct sd *) gspca_dev;

        /* For 518 and 518+ */
        const struct ov_regvals init_518[] = {
                { R51x_SYS_RESET,       0x40 },
                { R51x_SYS_INIT,        0xe1 },
                { R51x_SYS_RESET,       0x3e },
                { R51x_SYS_INIT,        0xe1 },
                { R51x_SYS_RESET,       0x00 },
                { R51x_SYS_INIT,        0xe1 },
                { 0x46,                 0x00 },
                { 0x5d,                 0x03 },
        };

        const struct ov_regvals norm_518[] = {
                { R51x_SYS_SNAP,        0x02 }, /* Reset */
                { R51x_SYS_SNAP,        0x01 }, /* Enable */
                { 0x31,                 0x0f },
                { 0x5d,                 0x03 },
                { 0x24,                 0x9f },
                { 0x25,                 0x90 },
                { 0x20,                 0x00 },
                { 0x51,                 0x04 },
                { 0x71,                 0x19 },
                { 0x2f,                 0x80 },
        };

        const struct ov_regvals norm_518_p[] = {
                { R51x_SYS_SNAP,        0x02 }, /* Reset */
                { R51x_SYS_SNAP,        0x01 }, /* Enable */
                { 0x31,                 0x0f },
                { 0x5d,                 0x03 },
                { 0x24,                 0x9f },
                { 0x25,                 0x90 },
                { 0x20,                 0x60 },
                { 0x51,                 0x02 },
                { 0x71,                 0x19 },
                { 0x40,                 0xff },
                { 0x41,                 0x42 },
                { 0x46,                 0x00 },
                { 0x33,                 0x04 },
                { 0x21,                 0x19 },
                { 0x3f,                 0x10 },
                { 0x2f,                 0x80 },
        };

        /* First 5 bits of custom ID reg are a revision ID on OV518 */
        PDEBUG(D_PROBE, "Device revision %d",
                0x1f & reg_r(sd, R51x_SYS_CUST_ID));

        write_regvals(sd, init_518, ARRAY_SIZE(init_518));

        /* Set LED GPIO pin to output mode */
        reg_w_mask(sd, R518_GPIO_CTL, 0x00, 0x02);

        switch (sd->bridge) {
        case BRIDGE_OV518:
                write_regvals(sd, norm_518, ARRAY_SIZE(norm_518));
                break;
        case BRIDGE_OV518PLUS:
                write_regvals(sd, norm_518_p, ARRAY_SIZE(norm_518_p));
                break;
        }

        ov51x_upload_quan_tables(sd);

        reg_w(sd, 0x2f, 0x80);
}

static void ov519_configure(struct sd *sd)
{
        static const struct ov_regvals init_519[] = {
                { 0x5a, 0x6d }, /* EnableSystem */
                { 0x53, 0x9b }, /* don't enable the microcontroller */
                { OV519_R54_EN_CLK1, 0xff }, /* set bit2 to enable jpeg */
                { 0x5d, 0x03 },
                { 0x49, 0x01 },
                { 0x48, 0x00 },
                /* Set LED pin to output mode. Bit 4 must be cleared or sensor
                 * detection will fail. This deserves further investigation. */
                { OV519_GPIO_IO_CTRL0,   0xee },
                { OV519_R51_RESET1, 0x0f },
                { OV519_R51_RESET1, 0x00 },
                { 0x22, 0x00 },
                /* windows reads 0x55 at this point*/
        };

        write_regvals(sd, init_519, ARRAY_SIZE(init_519));
}

static void ovfx2_configure(struct sd *sd)
{
        static const struct ov_regvals init_fx2[] = {
                { 0x00, 0x60 },
                { 0x02, 0x01 },
                { 0x0f, 0x1d },
                { 0xe9, 0x82 },
                { 0xea, 0xc7 },
                { 0xeb, 0x10 },
                { 0xec, 0xf6 },
        };

        sd->stopped = 1;

        write_regvals(sd, init_fx2, ARRAY_SIZE(init_fx2));
}

/* set the mode */
/* This function works for ov7660 only */
static void ov519_set_mode(struct sd *sd)
{
        static const struct ov_regvals bridge_ov7660[2][10] = {
                {{0x10, 0x14}, {0x11, 0x1e}, {0x12, 0x00}, {0x13, 0x00},
                 {0x14, 0x00}, {0x15, 0x00}, {0x16, 0x00}, {0x20, 0x0c},
                 {0x25, 0x01}, {0x26, 0x00}},
                {{0x10, 0x28}, {0x11, 0x3c}, {0x12, 0x00}, {0x13, 0x00},
                 {0x14, 0x00}, {0x15, 0x00}, {0x16, 0x00}, {0x20, 0x0c},
                 {0x25, 0x03}, {0x26, 0x00}}
        };
        static const struct ov_i2c_regvals sensor_ov7660[2][3] = {
                {{0x12, 0x00}, {0x24, 0x00}, {0x0c, 0x0c}},
                {{0x12, 0x00}, {0x04, 0x00}, {0x0c, 0x00}}
        };
        static const struct ov_i2c_regvals sensor_ov7660_2[] = {
                {OV7670_R17_HSTART, 0x13},
                {OV7670_R18_HSTOP, 0x01},
                {OV7670_R32_HREF, 0x92},
                {OV7670_R19_VSTART, 0x02},
                {OV7670_R1A_VSTOP, 0x7a},
                {OV7670_R03_VREF, 0x00},
/*              {0x33, 0x00}, */
/*              {0x34, 0x07}, */
/*              {0x36, 0x00}, */
/*              {0x6b, 0x0a}, */
        };

        write_regvals(sd, bridge_ov7660[sd->gspca_dev.curr_mode],
                        ARRAY_SIZE(bridge_ov7660[0]));
        write_i2c_regvals(sd, sensor_ov7660[sd->gspca_dev.curr_mode],
                        ARRAY_SIZE(sensor_ov7660[0]));
        write_i2c_regvals(sd, sensor_ov7660_2,
                        ARRAY_SIZE(sensor_ov7660_2));
}

/* set the frame rate */
/* This function works for sensors ov7640, ov7648 ov7660 and ov7670 only */
static void ov519_set_fr(struct sd *sd)
{
        int fr;
        u8 clock;
        /* frame rate table with indices:
         *      - mode = 0: 320x240, 1: 640x480
         *      - fr rate = 0: 30, 1: 25, 2: 20, 3: 15, 4: 10, 5: 5
         *      - reg = 0: bridge a4, 1: bridge 23, 2: sensor 11 (clock)
         */
        static const u8 fr_tb[2][6][3] = {
                {{0x04, 0xff, 0x00},
                 {0x04, 0x1f, 0x00},
                 {0x04, 0x1b, 0x00},
                 {0x04, 0x15, 0x00},
                 {0x04, 0x09, 0x00},
                 {0x04, 0x01, 0x00}},
                {{0x0c, 0xff, 0x00},
                 {0x0c, 0x1f, 0x00},
                 {0x0c, 0x1b, 0x00},
                 {0x04, 0xff, 0x01},
                 {0x04, 0x1f, 0x01},
                 {0x04, 0x1b, 0x01}},
        };

        if (frame_rate > 0)
                sd->frame_rate = frame_rate;
        if (sd->frame_rate >= 30)
                fr = 0;
        else if (sd->frame_rate >= 25)
                fr = 1;
        else if (sd->frame_rate >= 20)
                fr = 2;
        else if (sd->frame_rate >= 15)
                fr = 3;
        else if (sd->frame_rate >= 10)
                fr = 4;
        else
                fr = 5;
        reg_w(sd, 0xa4, fr_tb[sd->gspca_dev.curr_mode][fr][0]);
        reg_w(sd, 0x23, fr_tb[sd->gspca_dev.curr_mode][fr][1]);
        clock = fr_tb[sd->gspca_dev.curr_mode][fr][2];
        if (sd->sensor == SEN_OV7660)
                clock |= 0x80;          /* enable double clock */
        ov518_i2c_w(sd, OV7670_R11_CLKRC, clock);
}

static void setautogain(struct gspca_dev *gspca_dev)
{
        struct sd *sd = (struct sd *) gspca_dev;

        i2c_w_mask(sd, 0x13, sd->ctrls[AUTOGAIN].val ? 0x05 : 0x00, 0x05);
}

/* this function is called at probe time */
static int sd_config(struct gspca_dev *gspca_dev,
                        const struct usb_device_id *id)
{
        struct sd *sd = (struct sd *) gspca_dev;
        struct cam *cam = &gspca_dev->cam;

        sd->bridge = id->driver_info & BRIDGE_MASK;
        sd->invert_led = (id->driver_info & BRIDGE_INVERT_LED) != 0;

        switch (sd->bridge) {
        case BRIDGE_OV511:
        case BRIDGE_OV511PLUS:
                cam->cam_mode = ov511_vga_mode;
                cam->nmodes = ARRAY_SIZE(ov511_vga_mode);
                break;
        case BRIDGE_OV518:
        case BRIDGE_OV518PLUS:
                cam->cam_mode = ov518_vga_mode;
                cam->nmodes = ARRAY_SIZE(ov518_vga_mode);
                break;
        case BRIDGE_OV519:
                cam->cam_mode = ov519_vga_mode;
                cam->nmodes = ARRAY_SIZE(ov519_vga_mode);
                break;
        case BRIDGE_OVFX2:
                cam->cam_mode = ov519_vga_mode;
                cam->nmodes = ARRAY_SIZE(ov519_vga_mode);
                cam->bulk_size = OVFX2_BULK_SIZE;
                cam->bulk_nurbs = MAX_NURBS;
                cam->bulk = 1;
                break;
        case BRIDGE_W9968CF:
                cam->cam_mode = w9968cf_vga_mode;
                cam->nmodes = ARRAY_SIZE(w9968cf_vga_mode);
                cam->reverse_alts = 1;
                break;
        }

        gspca_dev->cam.ctrls = sd->ctrls;
        sd->quality = QUALITY_DEF;
        sd->frame_rate = 15;

        return 0;
}

/* this function is called at probe and resume time */
static int sd_init(struct gspca_dev *gspca_dev)
{
        struct sd *sd = (struct sd *) gspca_dev;
        struct cam *cam = &gspca_dev->cam;

        switch (sd->bridge) {
        case BRIDGE_OV511:
        case BRIDGE_OV511PLUS:
                ov511_configure(gspca_dev);
                break;
        case BRIDGE_OV518:
        case BRIDGE_OV518PLUS:
                ov518_configure(gspca_dev);
                break;
        case BRIDGE_OV519:
                ov519_configure(sd);
                break;
        case BRIDGE_OVFX2:
                ovfx2_configure(sd);
                break;
        case BRIDGE_W9968CF:
                w9968cf_configure(sd);
                break;
        }

        /* The OV519 must be more aggressive about sensor detection since
         * I2C write will never fail if the sensor is not present. We have
         * to try to initialize the sensor to detect its presence */
        sd->sensor = -1;

        /* Test for 76xx */
        if (init_ov_sensor(sd, OV7xx0_SID) >= 0) {
                ov7xx0_configure(sd);

        /* Test for 6xx0 */
        } else if (init_ov_sensor(sd, OV6xx0_SID) >= 0) {
                ov6xx0_configure(sd);

        /* Test for 8xx0 */
        } else if (init_ov_sensor(sd, OV8xx0_SID) >= 0) {
                ov8xx0_configure(sd);

        /* Test for 3xxx / 2xxx */
        } else if (init_ov_sensor(sd, OV_HIRES_SID) >= 0) {
                ov_hires_configure(sd);
        } else {
                pr_err("Can't determine sensor slave IDs\n");
                goto error;
        }

        if (sd->sensor < 0)
                goto error;

        ov51x_led_control(sd, 0);       /* turn LED off */

        switch (sd->bridge) {
        case BRIDGE_OV511:
        case BRIDGE_OV511PLUS:
                if (sd->sif) {
                        cam->cam_mode = ov511_sif_mode;
                        cam->nmodes = ARRAY_SIZE(ov511_sif_mode);
                }
                break;
        case BRIDGE_OV518:
        case BRIDGE_OV518PLUS:
                if (sd->sif) {
                        cam->cam_mode = ov518_sif_mode;
                        cam->nmodes = ARRAY_SIZE(ov518_sif_mode);
                }
                break;
        case BRIDGE_OV519:
                if (sd->sif) {
                        cam->cam_mode = ov519_sif_mode;
                        cam->nmodes = ARRAY_SIZE(ov519_sif_mode);
                }
                break;
        case BRIDGE_OVFX2:
                switch (sd->sensor) {
                case SEN_OV2610:
                case SEN_OV2610AE:
                        cam->cam_mode = ovfx2_ov2610_mode;
                        cam->nmodes = ARRAY_SIZE(ovfx2_ov2610_mode);
                        break;
                case SEN_OV3610:
                        cam->cam_mode = ovfx2_ov3610_mode;
                        cam->nmodes = ARRAY_SIZE(ovfx2_ov3610_mode);
                        break;
                case SEN_OV9600:
                        cam->cam_mode = ovfx2_ov9600_mode;
                        cam->nmodes = ARRAY_SIZE(ovfx2_ov9600_mode);
                        break;
                default:
                        if (sd->sif) {
                                cam->cam_mode = ov519_sif_mode;
                                cam->nmodes = ARRAY_SIZE(ov519_sif_mode);
                        }
                        break;
                }
                break;
        case BRIDGE_W9968CF:
                if (sd->sif)
                        cam->nmodes = ARRAY_SIZE(w9968cf_vga_mode) - 1;

                /* w9968cf needs initialisation once the sensor is known */
                w9968cf_init(sd);
                break;
        }

        gspca_dev->ctrl_dis = ctrl_dis[sd->sensor];

        /* initialize the sensor */
        switch (sd->sensor) {
        case SEN_OV2610:
                write_i2c_regvals(sd, norm_2610, ARRAY_SIZE(norm_2610));

                /* Enable autogain, autoexpo, awb, bandfilter */
                i2c_w_mask(sd, 0x13, 0x27, 0x27);
                break;
        case SEN_OV2610AE:
                write_i2c_regvals(sd, norm_2610ae, ARRAY_SIZE(norm_2610ae));

                /* enable autoexpo */
                i2c_w_mask(sd, 0x13, 0x05, 0x05);
                break;
        case SEN_OV3610:
                write_i2c_regvals(sd, norm_3620b, ARRAY_SIZE(norm_3620b));

                /* Enable autogain, autoexpo, awb, bandfilter */
                i2c_w_mask(sd, 0x13, 0x27, 0x27);
                break;
        case SEN_OV6620:
                write_i2c_regvals(sd, norm_6x20, ARRAY_SIZE(norm_6x20));
                break;
        case SEN_OV6630:
        case SEN_OV66308AF:
                sd->ctrls[CONTRAST].def = 200;
                                 /* The default is too low for the ov6630 */
                write_i2c_regvals(sd, norm_6x30, ARRAY_SIZE(norm_6x30));
                break;
        default:
/*      case SEN_OV7610: */
/*      case SEN_OV76BE: */
                write_i2c_regvals(sd, norm_7610, ARRAY_SIZE(norm_7610));
                i2c_w_mask(sd, 0x0e, 0x00, 0x40);
                break;
        case SEN_OV7620:
        case SEN_OV7620AE:
                write_i2c_regvals(sd, norm_7620, ARRAY_SIZE(norm_7620));
                break;
        case SEN_OV7640:
        case SEN_OV7648:
                write_i2c_regvals(sd, norm_7640, ARRAY_SIZE(norm_7640));
                break;
        case SEN_OV7660:
                i2c_w(sd, OV7670_R12_COM7, OV7670_COM7_RESET);
                msleep(14);
                reg_w(sd, OV519_R57_SNAPSHOT, 0x23);
                write_regvals(sd, init_519_ov7660,
                                ARRAY_SIZE(init_519_ov7660));
                write_i2c_regvals(sd, norm_7660, ARRAY_SIZE(norm_7660));
                sd->gspca_dev.curr_mode = 1;    /* 640x480 */
                ov519_set_mode(sd);
                ov519_set_fr(sd);
                sd->ctrls[COLORS].max = 4;      /* 0..4 */
                sd->ctrls[COLORS].val =
                        sd->ctrls[COLORS].def = 2;
                setcolors(gspca_dev);
                sd->ctrls[CONTRAST].max = 6;    /* 0..6 */
                sd->ctrls[CONTRAST].val =
                        sd->ctrls[CONTRAST].def = 3;
                setcontrast(gspca_dev);
                sd->ctrls[BRIGHTNESS].max = 6;  /* 0..6 */
                sd->ctrls[BRIGHTNESS].val =
                        sd->ctrls[BRIGHTNESS].def = 3;
                setbrightness(gspca_dev);
                sd_reset_snapshot(gspca_dev);
                ov51x_restart(sd);
                ov51x_stop(sd);                 /* not in win traces */
                ov51x_led_control(sd, 0);
                break;
        case SEN_OV7670:
                sd->ctrls[FREQ].max = 3;        /* auto */
                sd->ctrls[FREQ].def = 3;
                write_i2c_regvals(sd, norm_7670, ARRAY_SIZE(norm_7670));
                break;
        case SEN_OV8610:
                write_i2c_regvals(sd, norm_8610, ARRAY_SIZE(norm_8610));
                break;
        case SEN_OV9600:
                write_i2c_regvals(sd, norm_9600, ARRAY_SIZE(norm_9600));

                /* enable autoexpo */
/*              i2c_w_mask(sd, 0x13, 0x05, 0x05); */
                break;
        }
        return gspca_dev->usb_err;
error:
        PDEBUG(D_ERR, "OV519 Config failed");
        return -EINVAL;
}

/* function called at start time before URB creation */
static int sd_isoc_init(struct gspca_dev *gspca_dev)
{
        struct sd *sd = (struct sd *) gspca_dev;

        switch (sd->bridge) {
        case BRIDGE_OVFX2:
                if (gspca_dev->width != 800)
                        gspca_dev->cam.bulk_size = OVFX2_BULK_SIZE;
                else
                        gspca_dev->cam.bulk_size = 7 * 4096;
                break;
        }
        return 0;
}

/* Set up the OV511/OV511+ with the given image parameters.
 *
 * Do not put any sensor-specific code in here (including I2C I/O functions)
 */
static void ov511_mode_init_regs(struct sd *sd)
{
        int hsegs, vsegs, packet_size, fps, needed;
        int interlaced = 0;
        struct usb_host_interface *alt;
        struct usb_interface *intf;

        intf = usb_ifnum_to_if(sd->gspca_dev.dev, sd->gspca_dev.iface);
        alt = usb_altnum_to_altsetting(intf, sd->gspca_dev.alt);
        if (!alt) {
                pr_err("Couldn't get altsetting\n");
                sd->gspca_dev.usb_err = -EIO;
                return;
        }

        packet_size = le16_to_cpu(alt->endpoint[0].desc.wMaxPacketSize);
        reg_w(sd, R51x_FIFO_PSIZE, packet_size >> 5);

        reg_w(sd, R511_CAM_UV_EN, 0x01);
        reg_w(sd, R511_SNAP_UV_EN, 0x01);
        reg_w(sd, R511_SNAP_OPTS, 0x03);

        /* Here I'm assuming that snapshot size == image size.
         * I hope that's always true. --claudio
         */
        hsegs = (sd->gspca_dev.width >> 3) - 1;
        vsegs = (sd->gspca_dev.height >> 3) - 1;

        reg_w(sd, R511_CAM_PXCNT, hsegs);
        reg_w(sd, R511_CAM_LNCNT, vsegs);
        reg_w(sd, R511_CAM_PXDIV, 0x00);
        reg_w(sd, R511_CAM_LNDIV, 0x00);

        /* YUV420, low pass filter on */
        reg_w(sd, R511_CAM_OPTS, 0x03);

        /* Snapshot additions */
        reg_w(sd, R511_SNAP_PXCNT, hsegs);
        reg_w(sd, R511_SNAP_LNCNT, vsegs);
        reg_w(sd, R511_SNAP_PXDIV, 0x00);
        reg_w(sd, R511_SNAP_LNDIV, 0x00);

        /******** Set the framerate ********/
        if (frame_rate > 0)
                sd->frame_rate = frame_rate;

        switch (sd->sensor) {
        case SEN_OV6620:
                /* No framerate control, doesn't like higher rates yet */
                sd->clockdiv = 3;
                break;

        /* Note once the FIXME's in mode_init_ov_sensor_regs() are fixed
           for more sensors we need to do this for them too */
        case SEN_OV7620:
        case SEN_OV7620AE:
        case SEN_OV7640:
        case SEN_OV7648:
        case SEN_OV76BE:
                if (sd->gspca_dev.width == 320)
                        interlaced = 1;
                /* Fall through */
        case SEN_OV6630:
        case SEN_OV7610:
        case SEN_OV7670:
                switch (sd->frame_rate) {
                case 30:
                case 25:
                        /* Not enough bandwidth to do 640x480 @ 30 fps */
                        if (sd->gspca_dev.width != 640) {
                                sd->clockdiv = 0;
                                break;
                        }
                        /* Fall through for 640x480 case */
                default:
/*              case 20: */
/*              case 15: */
                        sd->clockdiv = 1;
                        break;
                case 10:
                        sd->clockdiv = 2;
                        break;
                case 5:
                        sd->clockdiv = 5;
                        break;
                }
                if (interlaced) {
                        sd->clockdiv = (sd->clockdiv + 1) * 2 - 1;
                        /* Higher then 10 does not work */
                        if (sd->clockdiv > 10)
                                sd->clockdiv = 10;
                }
                break;

        case SEN_OV8610:
                /* No framerate control ?? */
                sd->clockdiv = 0;
                break;
        }

        /* Check if we have enough bandwidth to disable compression */
        fps = (interlaced ? 60 : 30) / (sd->clockdiv + 1) + 1;
        needed = fps * sd->gspca_dev.width * sd->gspca_dev.height * 3 / 2;
        /* 1400 is a conservative estimate of the max nr of isoc packets/sec */
        if (needed > 1400 * packet_size) {
                /* Enable Y and UV quantization and compression */
                reg_w(sd, R511_COMP_EN, 0x07);
                reg_w(sd, R511_COMP_LUT_EN, 0x03);
        } else {
                reg_w(sd, R511_COMP_EN, 0x06);
                reg_w(sd, R511_COMP_LUT_EN, 0x00);
        }

        reg_w(sd, R51x_SYS_RESET, OV511_RESET_OMNICE);
        reg_w(sd, R51x_SYS_RESET, 0);
}

/* Sets up the OV518/OV518+ with the given image parameters
 *
 * OV518 needs a completely different approach, until we can figure out what
 * the individual registers do. Also, only 15 FPS is supported now.
 *
 * Do not put any sensor-specific code in here (including I2C I/O functions)
 */
static void ov518_mode_init_regs(struct sd *sd)
{
        int hsegs, vsegs, packet_size;
        struct usb_host_interface *alt;
        struct usb_interface *intf;

        intf = usb_ifnum_to_if(sd->gspca_dev.dev, sd->gspca_dev.iface);
        alt = usb_altnum_to_altsetting(intf, sd->gspca_dev.alt);
        if (!alt) {
                pr_err("Couldn't get altsetting\n");
                sd->gspca_dev.usb_err = -EIO;
                return;
        }

        packet_size = le16_to_cpu(alt->endpoint[0].desc.wMaxPacketSize);
        ov518_reg_w32(sd, R51x_FIFO_PSIZE, packet_size & ~7, 2);

        /******** Set the mode ********/
        reg_w(sd, 0x2b, 0);
        reg_w(sd, 0x2c, 0);
        reg_w(sd, 0x2d, 0);
        reg_w(sd, 0x2e, 0);
        reg_w(sd, 0x3b, 0);
        reg_w(sd, 0x3c, 0);
        reg_w(sd, 0x3d, 0);
        reg_w(sd, 0x3e, 0);

        if (sd->bridge == BRIDGE_OV518) {
                /* Set 8-bit (YVYU) input format */
                reg_w_mask(sd, 0x20, 0x08, 0x08);

                /* Set 12-bit (4:2:0) output format */
                reg_w_mask(sd, 0x28, 0x80, 0xf0);
                reg_w_mask(sd, 0x38, 0x80, 0xf0);
        } else {
                reg_w(sd, 0x28, 0x80);
                reg_w(sd, 0x38, 0x80);
        }

        hsegs = sd->gspca_dev.width / 16;
        vsegs = sd->gspca_dev.height / 4;

        reg_w(sd, 0x29, hsegs);
        reg_w(sd, 0x2a, vsegs);

        reg_w(sd, 0x39, hsegs);
        reg_w(sd, 0x3a, vsegs);

        /* Windows driver does this here; who knows why */
        reg_w(sd, 0x2f, 0x80);

        /******** Set the framerate ********/
        sd->clockdiv = 1;

        /* Mode independent, but framerate dependent, regs */
        /* 0x51: Clock divider; Only works on some cams which use 2 crystals */
        reg_w(sd, 0x51, 0x04);
        reg_w(sd, 0x22, 0x18);
        reg_w(sd, 0x23, 0xff);

        if (sd->bridge == BRIDGE_OV518PLUS) {
                switch (sd->sensor) {
                case SEN_OV7620AE:
                        if (sd->gspca_dev.width == 320) {
                                reg_w(sd, 0x20, 0x00);
                                reg_w(sd, 0x21, 0x19);
                        } else {
                                reg_w(sd, 0x20, 0x60);
                                reg_w(sd, 0x21, 0x1f);
                        }
                        break;
                case SEN_OV7620:
                        reg_w(sd, 0x20, 0x00);
                        reg_w(sd, 0x21, 0x19);
                        break;
                default:
                        reg_w(sd, 0x21, 0x19);
                }
        } else
                reg_w(sd, 0x71, 0x17);  /* Compression-related? */

        /* FIXME: Sensor-specific */
        /* Bit 5 is what matters here. Of course, it is "reserved" */
        i2c_w(sd, 0x54, 0x23);

        reg_w(sd, 0x2f, 0x80);

        if (sd->bridge == BRIDGE_OV518PLUS) {
                reg_w(sd, 0x24, 0x94);
                reg_w(sd, 0x25, 0x90);
                ov518_reg_w32(sd, 0xc4,    400, 2);     /* 190h   */
                ov518_reg_w32(sd, 0xc6,    540, 2);     /* 21ch   */
                ov518_reg_w32(sd, 0xc7,    540, 2);     /* 21ch   */
                ov518_reg_w32(sd, 0xc8,    108, 2);     /* 6ch    */
                ov518_reg_w32(sd, 0xca, 131098, 3);     /* 2001ah */
                ov518_reg_w32(sd, 0xcb,    532, 2);     /* 214h   */
                ov518_reg_w32(sd, 0xcc,   2400, 2);     /* 960h   */
                ov518_reg_w32(sd, 0xcd,     32, 2);     /* 20h    */
                ov518_reg_w32(sd, 0xce,    608, 2);     /* 260h   */
        } else {
                reg_w(sd, 0x24, 0x9f);
                reg_w(sd, 0x25, 0x90);
                ov518_reg_w32(sd, 0xc4,    400, 2);     /* 190h   */
                ov518_reg_w32(sd, 0xc6,    381, 2);     /* 17dh   */
                ov518_reg_w32(sd, 0xc7,    381, 2);     /* 17dh   */
                ov518_reg_w32(sd, 0xc8,    128, 2);     /* 80h    */
                ov518_reg_w32(sd, 0xca, 183331, 3);     /* 2cc23h */
                ov518_reg_w32(sd, 0xcb,    746, 2);     /* 2eah   */
                ov518_reg_w32(sd, 0xcc,   1750, 2);     /* 6d6h   */
                ov518_reg_w32(sd, 0xcd,     45, 2);     /* 2dh    */
                ov518_reg_w32(sd, 0xce,    851, 2);     /* 353h   */
        }

        reg_w(sd, 0x2f, 0x80);
}

/* Sets up the OV519 with the given image parameters
 *
 * OV519 needs a completely different approach, until we can figure out what
 * the individual registers do.
 *
 * Do not put any sensor-specific code in here (including I2C I/O functions)
 */
static void ov519_mode_init_regs(struct sd *sd)
{
        static const struct ov_regvals mode_init_519_ov7670[] = {
                { 0x5d, 0x03 }, /* Turn off suspend mode */
                { 0x53, 0x9f }, /* was 9b in 1.65-1.08 */
                { OV519_R54_EN_CLK1, 0x0f }, /* bit2 (jpeg enable) */
                { 0xa2, 0x20 }, /* a2-a5 are undocumented */
                { 0xa3, 0x18 },
                { 0xa4, 0x04 },
                { 0xa5, 0x28 },
                { 0x37, 0x00 }, /* SetUsbInit */
                { 0x55, 0x02 }, /* 4.096 Mhz audio clock */
                /* Enable both fields, YUV Input, disable defect comp (why?) */
                { 0x20, 0x0c },
                { 0x21, 0x38 },
                { 0x22, 0x1d },
                { 0x17, 0x50 }, /* undocumented */
                { 0x37, 0x00 }, /* undocumented */
                { 0x40, 0xff }, /* I2C timeout counter */
                { 0x46, 0x00 }, /* I2C clock prescaler */
                { 0x59, 0x04 }, /* new from windrv 090403 */
                { 0xff, 0x00 }, /* undocumented */
                /* windows reads 0x55 at this point, why? */
        };

        static const struct ov_regvals mode_init_519[] = {
                { 0x5d, 0x03 }, /* Turn off suspend mode */
                { 0x53, 0x9f }, /* was 9b in 1.65-1.08 */
                { OV519_R54_EN_CLK1, 0x0f }, /* bit2 (jpeg enable) */
                { 0xa2, 0x20 }, /* a2-a5 are undocumented */
                { 0xa3, 0x18 },
                { 0xa4, 0x04 },
                { 0xa5, 0x28 },
                { 0x37, 0x00 }, /* SetUsbInit */
                { 0x55, 0x02 }, /* 4.096 Mhz audio clock */
                /* Enable both fields, YUV Input, disable defect comp (why?) */
                { 0x22, 0x1d },
                { 0x17, 0x50 }, /* undocumented */
                { 0x37, 0x00 }, /* undocumented */
                { 0x40, 0xff }, /* I2C timeout counter */
                { 0x46, 0x00 }, /* I2C clock prescaler */
                { 0x59, 0x04 }, /* new from windrv 090403 */
                { 0xff, 0x00 }, /* undocumented */
                /* windows reads 0x55 at this point, why? */
        };

        /******** Set the mode ********/
        switch (sd->sensor) {
        default:
                write_regvals(sd, mode_init_519, ARRAY_SIZE(mode_init_519));
                if (sd->sensor == SEN_OV7640 ||
                    sd->sensor == SEN_OV7648) {
                        /* Select 8-bit input mode */
                        reg_w_mask(sd, OV519_R20_DFR, 0x10, 0x10);
                }
                break;
        case SEN_OV7660:
                return;         /* done by ov519_set_mode/fr() */
        case SEN_OV7670:
                write_regvals(sd, mode_init_519_ov7670,
                                ARRAY_SIZE(mode_init_519_ov7670));
                break;
        }

        reg_w(sd, OV519_R10_H_SIZE,     sd->gspca_dev.width >> 4);
        reg_w(sd, OV519_R11_V_SIZE,     sd->gspca_dev.height >> 3);
        if (sd->sensor == SEN_OV7670 &&
            sd->gspca_dev.cam.cam_mode[sd->gspca_dev.curr_mode].priv)
                reg_w(sd, OV519_R12_X_OFFSETL, 0x04);
        else if (sd->sensor == SEN_OV7648 &&
            sd->gspca_dev.cam.cam_mode[sd->gspca_dev.curr_mode].priv)
                reg_w(sd, OV519_R12_X_OFFSETL, 0x01);
        else
                reg_w(sd, OV519_R12_X_OFFSETL, 0x00);
        reg_w(sd, OV519_R13_X_OFFSETH,  0x00);
        reg_w(sd, OV519_R14_Y_OFFSETL,  0x00);
        reg_w(sd, OV519_R15_Y_OFFSETH,  0x00);
        reg_w(sd, OV519_R16_DIVIDER,    0x00);
        reg_w(sd, OV519_R25_FORMAT,     0x03); /* YUV422 */
        reg_w(sd, 0x26,                 0x00); /* Undocumented */

        /******** Set the framerate ********/
        if (frame_rate > 0)
                sd->frame_rate = frame_rate;

/* FIXME: These are only valid at the max resolution. */
        sd->clockdiv = 0;
        switch (sd->sensor) {
        case SEN_OV7640:
        case SEN_OV7648:
                switch (sd->frame_rate) {
                default:
/*              case 30: */
                        reg_w(sd, 0xa4, 0x0c);
                        reg_w(sd, 0x23, 0xff);
                        break;
                case 25:
                        reg_w(sd, 0xa4, 0x0c);
                        reg_w(sd, 0x23, 0x1f);
                        break;
                case 20:
                        reg_w(sd, 0xa4, 0x0c);
                        reg_w(sd, 0x23, 0x1b);
                        break;
                case 15:
                        reg_w(sd, 0xa4, 0x04);
                        reg_w(sd, 0x23, 0xff);
                        sd->clockdiv = 1;
                        break;
                case 10:
                        reg_w(sd, 0xa4, 0x04);
                        reg_w(sd, 0x23, 0x1f);
                        sd->clockdiv = 1;
                        break;
                case 5:
                        reg_w(sd, 0xa4, 0x04);
                        reg_w(sd, 0x23, 0x1b);
                        sd->clockdiv = 1;
                        break;
                }
                break;
        case SEN_OV8610:
                switch (sd->frame_rate) {
                default:        /* 15 fps */
/*              case 15: */
                        reg_w(sd, 0xa4, 0x06);
                        reg_w(sd, 0x23, 0xff);
                        break;
                case 10:
                        reg_w(sd, 0xa4, 0x06);
                        reg_w(sd, 0x23, 0x1f);
                        break;
                case 5:
                        reg_w(sd, 0xa4, 0x06);
                        reg_w(sd, 0x23, 0x1b);
                        break;
                }
                break;
        case SEN_OV7670:                /* guesses, based on 7640 */
                PDEBUG(D_STREAM, "Setting framerate to %d fps",
                                 (sd->frame_rate == 0) ? 15 : sd->frame_rate);
                reg_w(sd, 0xa4, 0x10);
                switch (sd->frame_rate) {
                case 30:
                        reg_w(sd, 0x23, 0xff);
                        break;
                case 20:
                        reg_w(sd, 0x23, 0x1b);
                        break;
                default:
/*              case 15: */
                        reg_w(sd, 0x23, 0xff);
                        sd->clockdiv = 1;
                        break;
                }
                break;
        }
}

static void mode_init_ov_sensor_regs(struct sd *sd)
{
        struct gspca_dev *gspca_dev;
        int qvga, xstart, xend, ystart, yend;
        u8 v;

        gspca_dev = &sd->gspca_dev;
        qvga = gspca_dev->cam.cam_mode[gspca_dev->curr_mode].priv & 1;

        /******** Mode (VGA/QVGA) and sensor specific regs ********/
        switch (sd->sensor) {