aboutsummaryrefslogblamecommitdiffstats
path: root/drivers/media/video/gspca/ov519.c
blob: 83139efc462935de68331cd5b8c0a1959425d746 (plain) (tree)
1464
1465
1466
1467
1468
1469
1470
1471
1472
1473
1474
1475
1476
1477
1478
1479
1480
1481
1482
1483
1484
1485
1486
1487
1488
1489
1490
1491
1492
1493
1494
1495
1496
1497
1498
1499
1500
1501
1502
1503
1504
1505
1506
1507
1508
1509
1510
1511
1512
1513
1514
1515
1516
1517
1518
1519
1520
1521
1522
1523
1524
1525
1526
1527
1528
1529
1530
1531
1532
1533
1534
1535
1536
1537
1538
1539
1540
1541
1542
1543
1544
1545
1546
1547
1548
1549
1550
1551
1552
1553
1554
1555
1556
1557
1558
1559
1560
1561
1562
1563
1564
1565
1566
1567
1568
1569
1570
1571
1572
1573
1574
1575
1576
1577
1578
1579
1580
1581
1582
1583
1584
1585
1586
1587
1588
1589
1590
1591
1592
1593
1594
1595
1596
1597
1598
1599
1600
1601
1602
1603
1604
1605
1606
1607
1608
1609
1610
1611
1612
1613
1614
1615
1616
1617
1618
1619
1620
1621
1622
1623
1624
1625
1626
1627
1628
1629
1630
1631
1632
1633
1634
1635
1636
1637
1638
1639
1640
1641
1642
1643
1644
1645
1646
1647
1648
1649
1650
1651
1652
1653
1654
1655
1656
1657
1658
1659
1660
1661
1662
1663
1664
1665
1666
1667
1668
1669
1670
1671
1672
1673
1674
1675
1676
1677
1678
1679
1680
1681
1682
1683
1684
1685
1686
1687
1688
1689
1690
1691
1692
1693
1694
1695
1696
1697
1698
1699
1700
1701
1702
1703
1704
1705
1706
1707
1708
1709
1710
1711
1712
1713
1714
1715
1716
1717
1718
1719
1720
1721
1722
1723
1724
1725
1726
1727
1728
1729
1730
1731
1732
1733
1734
1735
1736
1737
1738
1739
1740
1741
1742
1743
1744
1745
1746
1747
1748
1749
1750
1751
1752
1753
1754
1755
1756
1757
1758
1759
1760
1761
1762
1763
1764
1765
1766
1767
1768
1769
1770
1771
1772
1773
1774
1775
1776
1777
1778
1779
1780
1781
1782
1783
1784
1785
1786
1787
1788
1789
1790
1791
1792
1793
1794
1795
1796
1797
1798
1799
1800
1801
1802
1803
1804
1805
1806
1807
1808
1809
1810
1811
1812
1813
1814
1815
1816
1817
1818
1819
1820
1821
1822
1823
1824
1825
1826
1827
1828
1829
1830
1831
1832
1833
1834
1835
1836
1837
1838
1839
1840
1841
1842
1843
1844
1845
1846
1847
1848
1849
1850
1851
1852
1853
1854
1855
1856
1857
1858
1859
1860
1861
1862
1863
1864
1865
1866
1867
1868
1869
1870
1871
1872
1873
1874
1875
1876
1877
1878
1879
1880
1881
1882
1883
1884
1885
1886
1887
1888
1889
1890
1891
1892

























                                                                          


































































































                                                                               










                                                      
  










                                                      




































































































































                                                                             
 
                                         




                                                                         
                                                       









                                                                            




                                                                        
                                                                 

                     
                                               






                                                                 
                              

                




                                                                        
                                                                 

                     
                                               




















































































































































































































                                                                               
                                                          













                                                           
                                                                  
























                                                                             
                                                          








































































































































                                                                                
                                                          




























                                                  
                                                          

































































                                                                          
                                                          





                                                                  
                                                          






































































































































































































































































                                                                               
                                                          









































                                                                          
                                                          




































































































































































                                                                               
                                                     
















                                                                               
                                                              


























































                                                                                
























                                                                            
                                                                 





















                                                                               
                                                          


































                                                                               
                                                             


                                                           
                                                                    












































































































































































































































































































































































































































                                                                               
                                                                         

































































































                                                                               
                                            







                                                                      
                                            















































































































                                                                            
                                       












                                       
                                                                  












                                     
























                                                                     
                                      












                                                                     
 
/**
 * OV519 driver
 *
 * Copyright (C) 2008 Jean-Francois Moine (http://moinejf.free.fr)
 *
 * (This module is adapted from the ov51x-jpeg package)
 *
 * 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 MODULE_NAME "ov519"

#include "gspca.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;

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

	/* Determined by sensor type */
	short maxwidth;
	short maxheight;

	unsigned char primary_i2c_slave;	/* I2C write id of sensor */

	unsigned char brightness;
	unsigned char contrast;
	unsigned char colors;

	char compress;		/* Should the next frame be compressed? */
	char compress_inited;	/* Are compression params uploaded? */
	char stopped;		/* Streaming is temporarily paused */

	char frame_rate;	/* current Framerate (OV519 only) */
	char clockdiv;		/* clockdiv override for OV519 only */

	char sensor;		/* Type of image sensor chip (SEN_*) */
#define SEN_UNKNOWN 0
#define SEN_OV6620 1
#define SEN_OV6630 2
#define SEN_OV7610 3
#define SEN_OV7620 4
#define SEN_OV7630 5
#define SEN_OV7640 6
#define SEN_OV7670 7
#define SEN_OV76BE 8
#define SEN_OV8610 9

};

/* V4L2 controls supported by the driver */
static int sd_setbrightness(struct gspca_dev *gspca_dev, __s32 val);
static int sd_getbrightness(struct gspca_dev *gspca_dev, __s32 *val);
static int sd_setcontrast(struct gspca_dev *gspca_dev, __s32 val);
static int sd_getcontrast(struct gspca_dev *gspca_dev, __s32 *val);
static int sd_setcolors(struct gspca_dev *gspca_dev, __s32 val);
static int sd_getcolors(struct gspca_dev *gspca_dev, __s32 *val);

static struct ctrl sd_ctrls[] = {
#define SD_BRIGHTNESS 0
	{
	    {
		.id      = V4L2_CID_BRIGHTNESS,
		.type    = V4L2_CTRL_TYPE_INTEGER,
		.name    = "Brightness",
		.minimum = 0,
		.maximum = 255,
		.step    = 1,
		.default_value = 127,
	    },
	    .set = sd_setbrightness,
	    .get = sd_getbrightness,
	},
#define SD_CONTRAST 1
	{
	    {
		.id      = V4L2_CID_CONTRAST,
		.type    = V4L2_CTRL_TYPE_INTEGER,
		.name    = "Contrast",
		.minimum = 0,
		.maximum = 255,
		.step    = 1,
		.default_value = 127,
	    },
	    .set = sd_setcontrast,
	    .get = sd_getcontrast,
	},
#define SD_COLOR 2
	{
	    {
		.id      = V4L2_CID_SATURATION,
		.type    = V4L2_CTRL_TYPE_INTEGER,
		.name    = "Saturation",
		.minimum = 0,
		.maximum = 255,
		.step    = 1,
		.default_value = 127,
	    },
	    .set = sd_setcolors,
	    .get = sd_getcolors,
	},
};

static struct v4l2_pix_format vga_mode[] = {
	{320, 240, V4L2_PIX_FMT_JPEG, V4L2_FIELD_NONE,
		.bytesperline = 320,
		.sizeimage = 320 * 240 * 3 / 8 + 589,
		.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 struct v4l2_pix_format sif_mode[] = {
	{176, 144, V4L2_PIX_FMT_JPEG, V4L2_FIELD_NONE,
		.bytesperline = 176,
		.sizeimage = 176 * 144 * 3 / 8 + 589,
		.colorspace = V4L2_COLORSPACE_JPEG,
		.priv = 1},
	{352, 288, V4L2_PIX_FMT_JPEG, V4L2_FIELD_NONE,
		.bytesperline = 352,
		.sizeimage = 352 * 288 * 3 / 8 + 589,
		.colorspace = V4L2_COLORSPACE_JPEG,
		.priv = 0},
};

/* OV519 Camera interface register numbers */
#define OV519_CAM_H_SIZE		0x10
#define OV519_CAM_V_SIZE		0x11
#define OV519_CAM_X_OFFSETL		0x12
#define OV519_CAM_X_OFFSETH		0x13
#define OV519_CAM_Y_OFFSETL		0x14
#define OV519_CAM_Y_OFFSETH		0x15
#define OV519_CAM_DIVIDER		0x16
#define OV519_CAM_DFR			0x20
#define OV519_CAM_FORMAT		0x25

/* OV519 System Controller register numbers */
#define OV519_SYS_RESET1 0x51
#define OV519_SYS_EN_CLK1 0x54

#define OV519_GPIO_DATA_OUT0		0x71
#define OV519_GPIO_IO_CTRL0		0x72

#define OV511_ENDPOINT_ADDRESS  1	/* Isoc endpoint number */

/* 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 */

/* I2C ADDRESSES */
#define OV7xx0_SID   0x42
#define OV8xx0_SID   0xa0
#define OV6xx0_SID   0xc0

/* OV7610 registers */
#define OV7610_REG_GAIN		0x00	/* gain setting (5:0) */
#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 */

/* OV7670 registers */
#define OV7670_REG_GAIN        0x00    /* Gain lower 8 bits (rest in vref) */
#define OV7670_REG_BLUE        0x01    /* blue gain */
#define OV7670_REG_RED         0x02    /* red gain */
#define OV7670_REG_VREF        0x03    /* Pieces of GAIN, VSTART, VSTOP */
#define OV7670_REG_COM1        0x04    /* Control 1 */
#define OV7670_REG_AECHH       0x07    /* AEC MS 5 bits */
#define OV7670_REG_COM3        0x0c    /* Control 3 */
#define OV7670_REG_COM4        0x0d    /* Control 4 */
#define OV7670_REG_COM5        0x0e    /* All "reserved" */
#define OV7670_REG_COM6        0x0f    /* Control 6 */
#define OV7670_REG_AECH        0x10    /* More bits of AEC value */
#define OV7670_REG_CLKRC       0x11    /* Clock control */
#define OV7670_REG_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_REG_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_REG_COM9        0x14    /* Control 9  - gain ceiling */
#define OV7670_REG_COM10       0x15    /* Control 10 */
#define OV7670_REG_HSTART      0x17    /* Horiz start high bits */
#define OV7670_REG_HSTOP       0x18    /* Horiz stop high bits */
#define OV7670_REG_VSTART      0x19    /* Vert start high bits */
#define OV7670_REG_VSTOP       0x1a    /* Vert stop high bits */
#define OV7670_REG_MVFP        0x1e    /* Mirror / vflip */
#define   OV7670_MVFP_MIRROR     0x20    /* Mirror image */
#define OV7670_REG_AEW         0x24    /* AGC upper limit */
#define OV7670_REG_AEB         0x25    /* AGC lower limit */
#define OV7670_REG_VPT         0x26    /* AGC/AEC fast mode op region */
#define OV7670_REG_HREF        0x32    /* HREF pieces */
#define OV7670_REG_TSLB        0x3a    /* lots of stuff */
#define OV7670_REG_COM11       0x3b    /* Control 11 */
#define   OV7670_COM11_EXP       0x02
#define   OV7670_COM11_HZAUTO    0x10    /* Auto detect 50/60 Hz */
#define OV7670_REG_COM12       0x3c    /* Control 12 */
#define OV7670_REG_COM13       0x3d    /* Control 13 */
#define   OV7670_COM13_GAMMA     0x80    /* Gamma enable */
#define   OV7670_COM13_UVSAT     0x40    /* UV saturation auto adjustment */
#define OV7670_REG_COM14       0x3e    /* Control 14 */
#define OV7670_REG_EDGE        0x3f    /* Edge enhancement factor */
#define OV7670_REG_COM15       0x40    /* Control 15 */
#define   OV7670_COM15_R00FF     0xc0    /*            00 to FF */
#define OV7670_REG_COM16       0x41    /* Control 16 */
#define   OV7670_COM16_AWBGAIN   0x08    /* AWB gain enable */
#define OV7670_REG_BRIGHT      0x55    /* Brightness */
#define OV7670_REG_CONTRAS     0x56    /* Contrast control */
#define OV7670_REG_GFIX        0x69    /* Fix gain control */
#define OV7670_REG_RGB444      0x8c    /* RGB 444 control */
#define OV7670_REG_HAECC1      0x9f    /* Hist AEC/AGC control 1 */
#define OV7670_REG_HAECC2      0xa0    /* Hist AEC/AGC control 2 */
#define OV7670_REG_BD50MAX     0xa5    /* 50hz banding step limit */
#define OV7670_REG_HAECC3      0xa6    /* Hist AEC/AGC control 3 */
#define OV7670_REG_HAECC4      0xa7    /* Hist AEC/AGC control 4 */
#define OV7670_REG_HAECC5      0xa8    /* Hist AEC/AGC control 5 */
#define OV7670_REG_HAECC6      0xa9    /* Hist AEC/AGC control 6 */
#define OV7670_REG_HAECC7      0xaa    /* Hist AEC/AGC control 7 */
#define OV7670_REG_BD60MAX     0xab    /* 60hz banding step limit */

struct ovsensor_window {
	short x;
	short y;
	short width;
	short height;
/*	int format; */
	short quarter;		/* Scale width and height down 2x */
	short clockdiv;		/* Clock divisor setting */
};

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 int reg_w(struct sd *sd, __u16 index, __u8 value)
{
	int ret;

	sd->gspca_dev.usb_buf[0] = value;
	ret = usb_control_msg(sd->gspca_dev.dev,
			usb_sndctrlpipe(sd->gspca_dev.dev, 0),
			1,			/* REQ_IO (ov518/519) */
			USB_DIR_OUT | USB_TYPE_VENDOR | USB_RECIP_DEVICE,
			0, index,
			sd->gspca_dev.usb_buf, 1, 500);
	if (ret < 0)
		PDEBUG(D_ERR, "Write reg [%02x] %02x failed", index, value);
	return ret;
}

/* Read from a OV519 register */
/* returns: negative is error, pos or zero is data */
static int reg_r(struct sd *sd, __u16 index)
{
	int ret;

	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, 1, 500);

	if (ret >= 0)
		ret = sd->gspca_dev.usb_buf[0];
	else
		PDEBUG(D_ERR, "Read reg [0x%02x] failed", index);
	return ret;
}

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

	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
		PDEBUG(D_ERR, "Read reg 8 [0x%02x] failed", index);
	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 int 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 ret;

		oldval = ret & ~mask;		/* Clear the masked bits */
		value |= oldval;		/* Set the desired bits */
	}
	return reg_w(sd, index, 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 int i2c_w(struct sd *sd,
		__u8 reg,
		__u8 value)
{
	int rc;

	PDEBUG(D_USBO, "i2c 0x%02x -> [0x%02x]", value, reg);

	/* Select camera register */
	rc = reg_w(sd, R51x_I2C_SADDR_3, reg);
	if (rc < 0)
		return rc;

	/* Write "value" to I2C data port of OV511 */
	rc = reg_w(sd, R51x_I2C_DATA, value);
	if (rc < 0)
		return rc;

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

	/* wait for write complete */
	msleep(4);
	if (rc < 0)
		return rc;
	return 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 i2c_r(struct sd *sd, __u8 reg)
{
	int rc, value;

	/* Select camera register */
	rc = reg_w(sd, R51x_I2C_SADDR_2, reg);
	if (rc < 0)
		return rc;

	/* Initiate 2-byte write cycle */
	rc = reg_w(sd, R518_I2C_CTL, 0x03);
	if (rc < 0)
		return rc;

	/* Initiate 2-byte read cycle */
	rc = reg_w(sd, R518_I2C_CTL, 0x05);
	if (rc < 0)
		return rc;
	value = reg_r(sd, R51x_I2C_DATA);
	PDEBUG(D_USBI, "i2c [0x%02X] -> 0x%02X", reg, value);
	return value;
}

/* 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 int 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 rc;
	oldval = rc & ~mask;		/* Clear the masked bits */
	value |= oldval;		/* Set the desired bits */
	return i2c_w(sd, reg, value);
}

/* Temporarily stops OV511 from functioning. Must do this before changing
 * registers while the camera is streaming */
static inline int ov51x_stop(struct sd *sd)
{
	PDEBUG(D_STREAM, "stopping");
	sd->stopped = 1;
	return reg_w(sd, OV519_SYS_RESET1, 0x0f);
}

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

	/* Reinitialize the stream */
	return reg_w(sd, OV519_SYS_RESET1, 0x00);
}

/* 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)
{
	int i, success;

	/* Reset the sensor */
	if (i2c_w(sd, 0x12, 0x80) < 0)
		return -EIO;

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

	for (i = 0, success = 0; i < i2c_detect_tries && !success; i++) {
		if (i2c_r(sd, OV7610_REG_ID_HIGH) == 0x7f &&
		    i2c_r(sd, OV7610_REG_ID_LOW) == 0xa2) {
			success = 1;
			continue;
		}

		/* Reset the sensor */
		if (i2c_w(sd, 0x12, 0x80) < 0)
			return -EIO;
		/* Wait for it to initialize */
		msleep(150);
		/* Dummy read to sync I2C */
		if (i2c_r(sd, 0x00) < 0)
			return -EIO;
	}
	if (!success)
		return -EIO;
	PDEBUG(D_PROBE, "I2C synced in %d attempt(s)", i);
	return 0;
}

/* Switch on standard JPEG compression. Returns 0 for success. */
static int ov519_init_compression(struct sd *sd)
{
	if (!sd->compress_inited) {
		if (reg_w_mask(sd, OV519_SYS_EN_CLK1, 1 << 2, 1 << 2) < 0) {
			PDEBUG(D_ERR, "Error switching to compressed mode");
			return -EIO;
		}
		sd->compress_inited = 1;
	}
	return 0;
}

/* 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 int ov51x_set_slave_ids(struct sd *sd,
				__u8 slave)
{
	int rc;

	rc = reg_w(sd, R51x_I2C_W_SID, slave);
	if (rc < 0)
		return rc;
	return reg_w(sd, R51x_I2C_R_SID, slave + 1);
}

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

static int write_regvals(struct sd *sd,
			 const struct ov_regvals *regvals,
			 int n)
{
	int rc;

	while (--n >= 0) {
		rc = reg_w(sd, regvals->reg, regvals->val);
		if (rc < 0)
			return rc;
		regvals++;
	}
	return 0;
}

static int write_i2c_regvals(struct sd *sd,
			     const struct ov_i2c_regvals *regvals,
			     int n)
{
	int rc;

	while (--n >= 0) {
		rc = i2c_w(sd, regvals->reg, regvals->val);
		if (rc < 0)
			return rc;
		regvals++;
	}
	return 0;
}

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

/* This initializes the OV8110, OV8610 sensor. The OV8110 uses
 * the same register settings as the OV8610, since they are very similar.
 */
static int ov8xx0_configure(struct sd *sd)
{
	int rc;
	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 */
	};

	PDEBUG(D_PROBE, "starting ov8xx0 configuration");

	if (init_ov_sensor(sd) < 0)
		PDEBUG(D_ERR|D_PROBE, "Failed to read sensor ID");
	else
		PDEBUG(D_PROBE, "OV86x0 initialized");

	/* Detect sensor (sub)type */
	rc = i2c_r(sd, OV7610_REG_COM_I);
	if (rc < 0) {
		PDEBUG(D_ERR, "Error detecting sensor type");
		return -1;
	}
	if ((rc & 3) == 1) {
		PDEBUG(D_PROBE, "Sensor is an OV8610");
		sd->sensor = SEN_OV8610;
	} else {
		PDEBUG(D_ERR, "Unknown image sensor version: %d", rc & 3);
		return -1;
	}
	PDEBUG(D_PROBE, "Writing 8610 registers");
	if (write_i2c_regvals(sd,
			norm_8610,
			sizeof norm_8610 / sizeof norm_8610[0]))
		return -1;

	/* Set sensor-specific vars */
	sd->maxwidth = 640;
	sd->maxheight = 480;
	return 0;
}

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

	/* 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[] = {
		{ 0x00, 0x00 },		/* gain */
		{ 0x01, 0x80 },		/* blue gain */
		{ 0x02, 0x80 },		/* red gain */
		{ 0x03, 0xc0 },		/* OV7670_REG_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, 0x20 },
		{ 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 },
	};

	/* 7670. Defaults taken from OmniVision provided data,
	*  as provided by Jonathan Corbet of OLPC		*/
	static const struct ov_i2c_regvals norm_7670[] = {
		{ OV7670_REG_COM7, OV7670_COM7_RESET },
		{ OV7670_REG_TSLB, 0x04 },		/* OV */
		{ OV7670_REG_COM7, OV7670_COM7_FMT_VGA }, /* VGA */
		{ OV7670_REG_CLKRC, 0x1 },
	/*
	 * Set the hardware window.  These values from OV don't entirely
	 * make sense - hstop is less than hstart.  But they work...
	 */
		{ OV7670_REG_HSTART, 0x13 },	{ OV7670_REG_HSTOP, 0x01 },
		{ OV7670_REG_HREF, 0xb6 },	{ OV7670_REG_VSTART, 0x02 },
		{ OV7670_REG_VSTOP, 0x7a },	{ OV7670_REG_VREF, 0x0a },

		{ OV7670_REG_COM3, 0 },	{ OV7670_REG_COM14, 0 },
	/* Mystery scaling numbers */
		{ 0x70, 0x3a },		{ 0x71, 0x35 },
		{ 0x72, 0x11 },		{ 0x73, 0xf0 },
		{ 0xa2, 0x02 },
/* jfm */
/* { OV7670_REG_COM10, 0x0 }, */

	/* Gamma curve values */
		{ 0x7a, 0x20 },
/* jfm:win 7b=1c */
		{ 0x7b, 0x10 },
/* jfm:win 7c=28 */
		{ 0x7c, 0x1e },
/* jfm:win 7d=3c */
		{ 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_REG_COM8, OV7670_COM8_FASTAEC
				 | OV7670_COM8_AECSTEP
				 | OV7670_COM8_BFILT },
		{ OV7670_REG_GAIN, 0 },	{ OV7670_REG_AECH, 0 },
		{ OV7670_REG_COM4, 0x40 }, /* magic reserved bit */
/* jfm:win 14=38 */
		{ OV7670_REG_COM9, 0x18 }, /* 4x gain + magic rsvd bit */
		{ OV7670_REG_BD50MAX, 0x05 },	{ OV7670_REG_BD60MAX, 0x07 },
		{ OV7670_REG_AEW, 0x95 },	{ OV7670_REG_AEB, 0x33 },
		{ OV7670_REG_VPT, 0xe3 },	{ OV7670_REG_HAECC1, 0x78 },
		{ OV7670_REG_HAECC2, 0x68 },
/* jfm:win a1=0b */
		{ 0xa1, 0x03 }, /* magic */
		{ OV7670_REG_HAECC3, 0xd8 },	{ OV7670_REG_HAECC4, 0xd8 },
		{ OV7670_REG_HAECC5, 0xf0 },	{ OV7670_REG_HAECC6, 0x90 },
		{ OV7670_REG_HAECC7, 0x94 },
		{ OV7670_REG_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_REG_COM5, 0x61 },	{ OV7670_REG_COM6, 0x4b },
		{ 0x16, 0x02 },
/* jfm */
/*		{ OV7670_REG_MVFP, 0x07|OV7670_MVFP_MIRROR }, */
		{ OV7670_REG_MVFP, 0x07 },
		{ 0x21, 0x02 },		{ 0x22, 0x91 },
		{ 0x29, 0x07 },		{ 0x33, 0x0b },
		{ 0x35, 0x0b },		{ 0x37, 0x1d },
		{ 0x38, 0x71 },		{ 0x39, 0x2a },
		{ OV7670_REG_COM12, 0x78 },	{ 0x4d, 0x40 },
		{ 0x4e, 0x20 },		{ OV7670_REG_GFIX, 0 },
		{ 0x6b, 0x4a },		{ 0x74, 0x10 },
		{ 0x8d, 0x4f },		{ 0x8e, 0 },
		{ 0x8f, 0 },		{ 0x90, 0 },
		{ 0x91, 0 },		{ 0x96, 0 },
		{ 0x9a, 0 },		{ 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_REG_BLUE, 0x40 },
		{ OV7670_REG_RED, 0x60 },
		{ OV7670_REG_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, 0 },		{ 0x52, 0x22 },
		{ 0x53, 0x5e },		{ 0x54, 0x80 },
		{ 0x58, 0x9e },

		{ OV7670_REG_COM16, OV7670_COM16_AWBGAIN },
		{ OV7670_REG_EDGE, 0 },
		{ 0x75, 0x05 },		{ 0x76, 0xe1 },
		{ 0x4c, 0 },		{ 0x77, 0x01 },
		{ OV7670_REG_COM13, 0xc3 },	{ 0x4b, 0x09 },
		{ 0xc9, 0x60 },		{ OV7670_REG_COM16, 0x38 },
		{ 0x56, 0x40 },

		{ 0x34, 0x11 },
		{ OV7670_REG_COM11, OV7670_COM11_EXP|OV7670_COM11_HZAUTO },
		{ 0xa4, 0x88 },		{ 0x96, 0 },
		{ 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 },

	/* Format YUV422 */
		{ OV7670_REG_COM7, OV7670_COM7_YUV },  /* Selects YUV mode */
		{ OV7670_REG_RGB444, 0 },	/* No RGB444 please */
		{ OV7670_REG_COM1, 0 },
		{ OV7670_REG_COM15, OV7670_COM15_R00FF },
		{ OV7670_REG_COM9, 0x18 },
				/* 4x gain ceiling; 0x8 is reserved bit */
		{ 0x4f, 0x80 }, 	/* "matrix coefficient 1" */
		{ 0x50, 0x80 }, 	/* "matrix coefficient 2" */
		{ 0x52, 0x22 }, 	/* "matrix coefficient 4" */
		{ 0x53, 0x5e }, 	/* "matrix coefficient 5" */
		{ 0x54, 0x80 }, 	/* "matrix coefficient 6" */
		{ OV7670_REG_COM13, OV7670_COM13_GAMMA|OV7670_COM13_UVSAT },
};

	PDEBUG(D_PROBE, "starting OV7xx0 configuration");

/* jfm:already done? */
	if (init_ov_sensor(sd) < 0)
		PDEBUG(D_ERR, "Failed to read sensor ID");
	else
		PDEBUG(D_PROBE, "OV7xx0 initialized");

	/* 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 -1;
	}
	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 == 0x73) {
			PDEBUG(D_PROBE, "Sensor is an OV7670");
			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");
		else
			PDEBUG(D_PROBE, "Sensor is an OV76BE");

		/* OV511+ will return all zero isoc data unless we
		 * configure the sensor as a 7620. Someone needs to
		 * find the exact reg. setting that causes this. */
		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 high;
		}
		low = i2c_r(sd, 0x0b);
		if (low < 0) {
			PDEBUG(D_ERR, "Error detecting camera chip VER");
			return low;
		}
		if (high == 0x76) {
			if (low == 0x30) {
				PDEBUG(D_PROBE, "Sensor is an OV7630/OV7635");
				sd->sensor = SEN_OV7630;
			} else if (low == 0x40) {
				PDEBUG(D_PROBE, "Sensor is an OV7645");
				sd->sensor = SEN_OV7640; /* FIXME */
			} else if (low == 0x45) {
				PDEBUG(D_PROBE, "Sensor is an OV7645B");
				sd->sensor = SEN_OV7640; /* FIXME */
			} else if (low == 0x48) {
				PDEBUG(D_PROBE, "Sensor is an OV7648");
				sd->sensor = SEN_OV7640; /* FIXME */
			} else {
				PDEBUG(D_PROBE, "Unknown sensor: 0x76%X", low);
				return -1;
			}
		} else {
			PDEBUG(D_PROBE, "Sensor is an OV7620");
			sd->sensor = SEN_OV7620;
		}
	} else {
		PDEBUG(D_ERR, "Unknown image sensor version: %d", rc & 3);
		return -1;
	}

	if (sd->sensor == SEN_OV7620) {
		PDEBUG(D_PROBE, "Writing 7620 registers");
		if (write_i2c_regvals(sd, norm_7620,
				sizeof norm_7620 / sizeof norm_7620[0]))
			return -1;
	} else if (sd->sensor == SEN_OV7630) {
		PDEBUG(D_ERR, "7630 is not supported by this driver version");
		return -1;
	} else if (sd->sensor == SEN_OV7640) {
		PDEBUG(D_PROBE, "Writing 7640 registers");
		if (write_i2c_regvals(sd, norm_7640,
				sizeof norm_7640 / sizeof norm_7640[0]))
			return -1;
	} else if (sd->sensor == SEN_OV7670) {
		PDEBUG(D_PROBE, "Writing 7670 registers");
		if (write_i2c_regvals(sd, norm_7670,
				sizeof norm_7670 / sizeof norm_7670[0]))
			return -1;
	} else {
		PDEBUG(D_PROBE, "Writing 7610 registers");
		if (write_i2c_regvals(sd, norm_7610,
				sizeof norm_7610 / sizeof norm_7610[0]))
			return -1;
	}

	/* Set sensor-specific vars */
	sd->maxwidth = 640;
	sd->maxheight = 480;
	return 0;
}

/* This initializes the OV6620, OV6630, OV6630AE, or OV6630AF sensor. */
static int ov6xx0_configure(struct sd *sd)
{
	int rc;
	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, 0x99 },
		{ 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, 0x20 },
		{ 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 },
	};

	PDEBUG(D_PROBE, "starting sensor configuration");

	if (init_ov_sensor(sd) < 0) {
		PDEBUG(D_ERR, "Failed to read sensor ID.");
		return -1;
	}
	PDEBUG(D_PROBE, "OV6xx0 sensor detected");

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

	/* 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. */
	if (rc == 0x00) {
		sd->sensor = SEN_OV6630;
		PDEBUG(D_ERR,
			"WARNING: Sensor is an OV66308. Your camera may have");
		PDEBUG(D_ERR, "been misdetected in previous driver versions.");
	} else if (rc == 0x01) {
		sd->sensor = SEN_OV6620;
		PDEBUG(D_PROBE, "Sensor is an OV6620");
	} else if (rc == 0x02) {
		sd->sensor = SEN_OV6630;
		PDEBUG(D_PROBE, "Sensor is an OV66308AE");
	} else if (rc == 0x03) {
		sd->sensor = SEN_OV6630;
		PDEBUG(D_PROBE, "Sensor is an OV66308AF");
	} else if (rc == 0x90) {
		sd->sensor = SEN_OV6630;
		PDEBUG(D_ERR,
			"WARNING: Sensor is an OV66307. Your camera may have");
		PDEBUG(D_ERR, "been misdetected in previous driver versions.");
	} else {
		PDEBUG(D_ERR, "FATAL: Unknown sensor version: 0x%02x", rc);
		return -1;
	}

	/* Set sensor-specific vars */
	sd->maxwidth = 352;
	sd->maxheight = 288;

	if (sd->sensor == SEN_OV6620) {
		PDEBUG(D_PROBE, "Writing 6x20 registers");
		if (write_i2c_regvals(sd, norm_6x20,
				sizeof norm_6x20 / sizeof norm_6x20[0]))
			return -1;
	} else {
		PDEBUG(D_PROBE, "Writing 6x30 registers");
		if (write_i2c_regvals(sd, norm_6x30,
				sizeof norm_6x30 / sizeof norm_6x30[0]))
			return -1;
	}
	return 0;
}

/* Turns on or off the LED. Only has an effect with OV511+/OV518(+)/OV519 */
static void ov51x_led_control(struct sd *sd, int on)
{
	PDEBUG(D_STREAM, "LED (%s)", on ? "on" : "off");

/*	if (sd->bridge == BRG_OV511PLUS) */
/*		reg_w(sd, R511_SYS_LED_CTL, on ? 1 : 0); */
/*	else if (sd->bridge == BRG_OV519) */
		reg_w_mask(sd, OV519_GPIO_DATA_OUT0, !on, 1);	/* 0 / 1 */
/*	else if (sd->bclass == BCL_OV518) */
/*		reg_w_mask(sd, R518_GPIO_OUT, on ? 0x02 : 0x00, 0x02); */
}

/* 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;

/* (from ov519_configure) */
	static const struct ov_regvals init_519[] = {
		{ 0x5a,  0x6d }, /* EnableSystem */
/* jfm trace usbsnoop3-1.txt */
/* jfm 53 = fb */
		{ 0x53,  0x9b },
		{ 0x54,  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 },
		{ 0x51,  0x0f }, /* SetUsbInit */
		{ 0x51,  0x00 },
		{ 0x22,  0x00 },
		/* windows reads 0x55 at this point*/
	};

	if (write_regvals(sd, init_519, ARRAY_SIZE(init_519)))
		goto error;
/* jfm: not seen in windows trace */
	if (ov519_init_compression(sd))
		goto error;
	ov51x_led_control(sd, 0);	/* turn LED off */

	/* Test for 76xx */
	sd->primary_i2c_slave = OV7xx0_SID;
	if (ov51x_set_slave_ids(sd, OV7xx0_SID) < 0)
		goto error;

	/* 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 */
	if (init_ov_sensor(sd) < 0) {
		/* Test for 6xx0 */
		sd->primary_i2c_slave = OV6xx0_SID;
		if (ov51x_set_slave_ids(sd, OV6xx0_SID) < 0)
			goto error;

		if (init_ov_sensor(sd) < 0) {
			/* Test for 8xx0 */
			sd->primary_i2c_slave = OV8xx0_SID;
			if (ov51x_set_slave_ids(sd, OV8xx0_SID) < 0)
				goto error;

			if (init_ov_sensor(sd) < 0) {
				PDEBUG(D_ERR,
					"Can't determine sensor slave IDs");
				goto error;
			} else {
				if (ov8xx0_configure(sd) < 0) {
					PDEBUG(D_ERR,
					   "Failed to configure OV8xx0 sensor");
					goto error;
				}
			}
		} else {
			if (ov6xx0_configure(sd) < 0) {
				PDEBUG(D_ERR, "Failed to configure OV6xx0");
				goto error;
			}
		}
	} else {
		if (ov7xx0_configure(sd) < 0) {
			PDEBUG(D_ERR, "Failed to configure OV7xx0");
			goto error;
		}
	}

	cam = &gspca_dev->cam;
	cam->epaddr = OV511_ENDPOINT_ADDRESS;
	if (sd->maxwidth == 640) {
		cam->cam_mode = vga_mode;
		cam->nmodes = sizeof vga_mode / sizeof vga_mode[0];
	} else {
		cam->cam_mode = sif_mode;
		cam->nmodes = sizeof sif_mode / sizeof sif_mode[0];
	}
	sd->brightness = sd_ctrls[SD_BRIGHTNESS].qctrl.default_value;
	sd->contrast = sd_ctrls[SD_CONTRAST].qctrl.default_value;
	sd->colors = sd_ctrls[SD_COLOR].qctrl.default_value;
	return 0;
error:
	PDEBUG(D_ERR, "OV519 Config failed");
	return -EBUSY;
}

/* this function is called at open time */
static int sd_open(struct gspca_dev *gspca_dev)
{
	return 0;
}

/* 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 int ov519_mode_init_regs(struct sd *sd,
				int width, int height)
{
	static const struct ov_regvals mode_init_519_ov7670[] = {
		{ 0x5d,	0x03 }, /* Turn off suspend mode */
		{ 0x53,	0x9f }, /* was 9b in 1.65-1.08 */
		{ 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 },
		{ 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 */
		{ 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?) */
		{ 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? */
	};

/* int hi_res; */

	PDEBUG(D_CONF, "mode init %dx%d", width, height);

/*	if (width >= 800 && height >= 600)
		hi_res = 1;
	else
		hi_res = 0; */

/*	if (ov51x_stop(sd) < 0)
		return -EIO; */

	/******** Set the mode ********/
	if (sd->sensor != SEN_OV7670) {
		if (write_regvals(sd, mode_init_519,
				  ARRAY_SIZE(mode_init_519)))
			return -EIO;
	} else {
		if (write_regvals(sd, mode_init_519_ov7670,
				  ARRAY_SIZE(mode_init_519_ov7670)))
			return -EIO;
	}

	if (sd->sensor == SEN_OV7640) {
		/* Select 8-bit input mode */
		reg_w_mask(sd, OV519_CAM_DFR, 0x10, 0x10);
	}

	reg_w(sd, OV519_CAM_H_SIZE,	width >> 4);
	reg_w(sd, OV519_CAM_V_SIZE,	height >> 3);
	reg_w(sd, OV519_CAM_X_OFFSETL,	0x00);
	reg_w(sd, OV519_CAM_X_OFFSETH,	0x00);
	reg_w(sd, OV519_CAM_Y_OFFSETL,	0x00);
	reg_w(sd, OV519_CAM_Y_OFFSETH,	0x00);
	reg_w(sd, OV519_CAM_DIVIDER,	0x00);
	reg_w(sd, OV519_CAM_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;
	if (sd->sensor == SEN_OV7640) {
		switch (sd->frame_rate) {
/*jfm: default was 30 fps */
		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;
		default:
/*		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;
		}
	} else if (sd->sensor == 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;
		}
		sd->clockdiv = 0;
	} else if (sd->sensor == SEN_OV7670) { /* guesses, based on 7640 */
		PDEBUG(D_STREAM, "Setting framerate to %d fps",
				 (sd->frame_rate == 0) ? 15 : sd->frame_rate);
		switch (sd->frame_rate) {
		case 30:
			reg_w(sd, 0xa4, 0x10);
			reg_w(sd, 0x23, 0xff);
			break;
		case 20:
			reg_w(sd, 0xa4, 0x10);
			reg_w(sd, 0x23, 0x1b);
			break;
		default: /* 15 fps */
/*			case 15: */
			reg_w(sd, 0xa4, 0x10);
			reg_w(sd, 0x23, 0xff);
			sd->clockdiv = 1;
			break;
		}
	}

/*	if (ov51x_restart(sd) < 0)
		return -EIO; */

	/* Reset it just for good measure */
/*	if (ov51x_reset(sd, OV511_RESET_NOREGS) < 0)
		return -EIO; */
	return 0;
}

static int mode_init_ov_sensor_regs(struct sd *sd,
				struct ovsensor_window *win)
{
	int qvga = win->quarter;

	/******** Mode (VGA/QVGA) and sensor specific regs ********/
	switch (sd->sensor) {
	case SEN_OV8610:
		/* For OV8610 qvga means qsvga */
		i2c_w_mask(sd, OV7610_REG_COM_C, qvga ? (1 << 5) : 0, 1 << 5);
		break;
	case SEN_OV7610:
		i2c_w_mask(sd, 0x14, qvga ? 0x20 : 0x00, 0x20);
		break;
	case SEN_OV7620:
/*		i2c_w(sd, 0x2b, 0x00); */
		i2c_w_mask(sd, 0x14, qvga ? 0x20 : 0x00, 0x20);
		i2c_w_mask(sd, 0x28, qvga ? 0x00 : 0x20, 0x20);
		i2c_w(sd, 0x24, qvga ? 0x20 : 0x3a);
		i2c_w(sd, 0x25, qvga ? 0x30 : 0x60);
		i2c_w_mask(sd, 0x2d, qvga ? 0x40 : 0x00, 0x40);
		i2c_w_mask(sd, 0x67, qvga ? 0xf0 : 0x90, 0xf0);
		i2c_w_mask(sd, 0x74, qvga ? 0x20 : 0x00, 0x20);
		break;
	case SEN_OV76BE:
/*		i2c_w(sd, 0x2b, 0x00); */
		i2c_w_mask(sd, 0x14, qvga ? 0x20 : 0x00, 0x20);
		break;
	case SEN_OV7640:
/*		i2c_w(sd, 0x2b, 0x00); */
		i2c_w_mask(sd, 0x14, qvga ? 0x20 : 0x00, 0x20);
		i2c_w_mask(sd, 0x28, qvga ? 0x00 : 0x20, 0x20);
/*		i2c_w(sd, 0x24, qvga ? 0x20 : 0x3a); */
/*		i2c_w(sd, 0x25, qvga ? 0x30 : 0x60); */
/*		i2c_w_mask(sd, 0x2d, qvga ? 0x40 : 0x00, 0x40); */
/*		i2c_w_mask(sd, 0x67, qvga ? 0xf0 : 0x90, 0xf0); */
/*		i2c_w_mask(sd, 0x74, qvga ? 0x20 : 0x00, 0x20); */
		break;
	case SEN_OV7670:
		/* set COM7_FMT_VGA or COM7_FMT_QVGA
		 * do we need to set anything else?
		 *	HSTART etc are set in set_ov_sensor_window itself */
		i2c_w_mask(sd, OV7670_REG_COM7,
			 qvga ? OV7670_COM7_FMT_QVGA : OV7670_COM7_FMT_VGA,
			 OV7670_COM7_FMT_MASK);
		break;
	case SEN_OV6620:
		i2c_w_mask(sd, 0x14, qvga ? 0x20 : 0x00, 0x20);
		break;
	case SEN_OV6630:
		i2c_w_mask(sd, 0x14, qvga ? 0x20 : 0x00, 0x20);
		break;
	default:
		return -EINVAL;
	}

	/******** Palette-specific regs ********/
/* Need to do work here for the OV7670 */

		if (sd->sensor == SEN_OV7610 || sd->sensor == SEN_OV76BE) {
			/* not valid on the OV6620/OV7620/6630? */
			i2c_w_mask(sd, 0x0e, 0x00, 0x40);
		}

		/* The OV518 needs special treatment. Although both the OV518
		 * and the OV6630 support a 16-bit video bus, only the 8 bit Y
		 * bus is actually used. The UV bus is tied to ground.
		 * Therefore, the OV6630 needs to be in 8-bit multiplexed
		 * output mode */

		/* OV7640 is 8-bit only */

		if (sd->sensor != SEN_OV6630 && sd->sensor != SEN_OV7640)
			i2c_w_mask(sd, 0x13, 0x00, 0x20);
/*	} */

	/******** Clock programming ********/
	/* The OV6620 needs special handling. This prevents the
	 * severe banding that normally occurs */
	if (sd->sensor == SEN_OV6620) {

		/* Clock down */
		i2c_w(sd, 0x2a, 0x04);
		i2c_w(sd, 0x11, win->clockdiv);
		i2c_w(sd, 0x2a, 0x84);
		/* This next setting is critical. It seems to improve
		 * the gain or the contrast. The "reserved" bits seem
		 * to have some effect in this case. */
		i2c_w(sd, 0x2d, 0x85);
	} else if (win->clockdiv >= 0) {
		i2c_w(sd, 0x11, win->clockdiv);
	}

	/******** Special Features ********/
/* no evidence this is possible with OV7670, either */
	/* Test Pattern */
	if (sd->sensor != SEN_OV7640 && sd->sensor != SEN_OV7670)
		i2c_w_mask(sd, 0x12, 0x00, 0x02);

	/* Enable auto white balance */
	if (sd->sensor == SEN_OV7670)
		i2c_w_mask(sd, OV7670_REG_COM8, OV7670_COM8_AWB,
				OV7670_COM8_AWB);
	else
		i2c_w_mask(sd, 0x12, 0x04, 0x04);

	/* This will go away as soon as ov51x_mode_init_sensor_regs() */
	/* is fully tested. */
	/* 7620/6620/6630? don't have register 0x35, so play it safe */
	if (sd->sensor == SEN_OV7610 || sd->sensor == SEN_OV76BE) {
		if (win->width == 640 /*&& win->height == 480*/)
			i2c_w(sd, 0x35, 0x9e);
		else
			i2c_w(sd, 0x35, 0x1e);
	}
	return 0;
}

static int set_ov_sensor_window(struct sd *sd,
				struct ovsensor_window *win)
{
	int hwsbase, hwebase, vwsbase, vwebase, hwscale, vwscale;
	int ret, hstart, hstop, vstop, vstart;
	__u8 v;

	/* The different sensor ICs handle setting up of window differently.
	 * IF YOU SET IT WRONG, YOU WILL GET ALL ZERO ISOC DATA FROM OV51x!! */
	switch (sd->sensor) {
	case SEN_OV8610:
		hwsbase = 0x1e;
		hwebase = 0x1e;
		vwsbase = 0x02;
		vwebase = 0x02;
		break;
	case SEN_OV7610:
	case SEN_OV76BE:
		hwsbase = 0x38;
		hwebase = 0x3a;
		vwsbase = vwebase = 0x05;
		break;
	case SEN_OV6620:
	case SEN_OV6630:
		hwsbase = 0x38;
		hwebase = 0x3a;
		vwsbase = 0x05;
		vwebase = 0x06;
		break;
	case SEN_OV7620:
		hwsbase = 0x2f;		/* From 7620.SET (spec is wrong) */
		hwebase = 0x2f;
		vwsbase = vwebase = 0x05;
		break;
	case SEN_OV7640:
		hwsbase = 0x1a;
		hwebase = 0x1a;
		vwsbase = vwebase = 0x03;
		break;
	case SEN_OV7670:
		/*handling of OV7670 hardware sensor start and stop values
		 * is very odd, compared to the other OV sensors */
		vwsbase = vwebase = hwebase = hwsbase = 0x00;
		break;
	default:
		return -EINVAL;
	}

	switch (sd->sensor) {
	case SEN_OV6620:
	case SEN_OV6630:
		if (win->quarter) {	/* QCIF */
			hwscale = 0;
			vwscale = 0;
		} else {		/* CIF */
			hwscale = 1;
			vwscale = 1;	/* The datasheet says 0;
					 * it's wrong */
		}
		break;
	case SEN_OV8610:
		if (win->quarter) {	/* QSVGA */
			hwscale = 1;
			vwscale = 1;
		} else {		/* SVGA */
			hwscale = 2;
			vwscale = 2;
		}
		break;
	default:			/* SEN_OV7xx0 */
		if (win->quarter) {	/* QVGA */
			hwscale = 1;
			vwscale = 0;
		} else {		/* VGA */
			hwscale = 2;
			vwscale = 1;
		}
	}

	ret = mode_init_ov_sensor_regs(sd, win);
	if (ret < 0)
		return ret;

	if (sd->sensor == SEN_OV8610) {
		i2c_w_mask(sd, 0x2d, 0x05, 0x40);
				/* old 0x95, new 0x05 from windrv 090403 */
						/* bits 5-7: reserved */
		i2c_w_mask(sd, 0x28, 0x20, 0x20);
					/* bit 5: progressive mode on */
	}

	/* The below is wrong for OV7670s because their window registers
	 * only store the high bits in 0x17 to 0x1a */

	/* SRH Use sd->max values instead of requested win values */
	/* SCS Since we're sticking with only the max hardware widths
	 * for a given mode */
	/* I can hard code this for OV7670s */
	/* Yes, these numbers do look odd, but they're tested and work! */
	if (sd->sensor == SEN_OV7670) {
		if (win->quarter) {	/* QVGA from ov7670.c by
					 * Jonathan Corbet */
			hstart = 164;
			hstop = 20;
			vstart = 14;
			vstop = 494;
		} else {		/* VGA */
			hstart = 158;
			hstop = 14;
			vstart = 10;
			vstop = 490;
		}
		/* OV7670 hardware window registers are split across
		 * multiple locations */
		i2c_w(sd, OV7670_REG_HSTART, (hstart >> 3) & 0xff);
		i2c_w(sd, OV7670_REG_HSTOP, (hstop >> 3) & 0xff);
		v = i2c_r(sd, OV7670_REG_HREF);
		v = (v & 0xc0) | ((hstop & 0x7) << 3) | (hstart & 0x07);
		msleep(10);	/* need to sleep between read and write to
				 * same reg! */
		i2c_w(sd, OV7670_REG_HREF, v);

		i2c_w(sd, OV7670_REG_VSTART, (vstart >> 2) & 0xff);
		i2c_w(sd, OV7670_REG_VSTOP, (vstop >> 2) & 0xff);
		v = i2c_r(sd, OV7670_REG_VREF);
		v = (v & 0xc0) | ((vstop & 0x3) << 2) | (vstart & 0x03);
		msleep(10);	/* need to sleep between read and write to
				 * same reg! */
		i2c_w(sd, OV7670_REG_VREF, v);

	} else {
		i2c_w(sd, 0x17, hwsbase + (win->x >> hwscale));
		i2c_w(sd, 0x18, hwebase + ((win->x + win->width) >> hwscale));
		i2c_w(sd, 0x19, vwsbase + (win->y >> vwscale));
		i2c_w(sd, 0x1a, vwebase + ((win->y + win->height) >> vwscale));
	}
	return 0;
}

static int ov_sensor_mode_setup(struct sd *sd,
				int width, int height)
{
	struct ovsensor_window win;

/*	win.format = mode; */

	/* Unless subcapture is enabled,
	 * center the image window and downsample
	 * if possible to increase the field of view */
	/* NOTE: OV518(+) and OV519 does downsampling on its own */
	win.width = width;
	win.height = height;
	if (width == sd->maxwidth)
		win.quarter = 0;
	else
		win.quarter = 1;

	/* Center it */
	win.x = (win.width - width) / 2;
	win.y = (win.height - height) / 2;

	/* Clock is determined by OV519 frame rate code */
	win.clockdiv = sd->clockdiv;

	PDEBUG(D_CONF, "Setting clock divider to %d", win.clockdiv);
	return set_ov_sensor_window(sd, &win);
}

/* -- start the camera -- */
static void sd_start(struct gspca_dev *gspca_dev)
{
	struct sd *sd = (struct sd *) gspca_dev;
	int ret;


	ret = ov519_mode_init_regs(sd, gspca_dev->width, gspca_dev->height);
	if (ret < 0)
		goto out;
	ret = ov_sensor_mode_setup(sd, gspca_dev->width, gspca_dev->height);
	if (ret < 0)
		goto out;

	ret = ov51x_restart((struct sd *) gspca_dev);
	if (ret < 0)
		goto out;
	PDEBUG(D_STREAM, "camera started alt: 0x%02x", gspca_dev->alt);
	ov51x_led_control(sd, 1);
	return;
out:
	PDEBUG(D_ERR, "camera start error:%d", ret);
}

static void sd_stopN(struct gspca_dev *gspca_dev)
{
	ov51x_stop((struct sd *) gspca_dev);
	ov51x_led_control((struct sd *) gspca_dev, 0);
}

static void sd_stop0(struct gspca_dev *gspca_dev)
{
}

static void sd_close(struct gspca_dev *gspca_dev)
{
}

static void sd_pkt_scan(struct gspca_dev *gspca_dev,
			struct gspca_frame *frame,	/* target */
			__u8 *data,			/* isoc packet */
			int len)			/* iso packet length */
{
	/* Header of ov519 is 16 bytes:
	 *     Byte     Value      Description
	 *	0	0xff	magic
	 *	1	0xff	magic
	 *	2	0xff	magic
	 *	3	0xXX	0x50 = SOF, 0x51 = EOF
	 *	9	0xXX	0x01 initial frame without data,
	 *			0x00 standard frame with image
	 *	14	Lo	in EOF: length of image data / 8
	 *	15	Hi
	 */

	if (data[0] == 0xff && data[1] == 0xff && data[2] == 0xff) {
		switch (data[3]) {
		case 0x50:		/* start of frame */
#define HDRSZ 16
			data += HDRSZ;
			len -= HDRSZ;
#undef HDRSZ
			if (data[0] == 0xff || data[1] == 0xd8)
				gspca_frame_add(gspca_dev, FIRST_PACKET, frame,
						data, len);
			else
				gspca_dev->last_packet_type = DISCARD_PACKET;
			return;
		case 0x51:		/* end of frame */
			if (data[9] != 0)
				gspca_dev->last_packet_type = DISCARD_PACKET;
			gspca_frame_add(gspca_dev, LAST_PACKET, frame,
					data, 0);
			return;
		}
	}

	/* intermediate packet */
	gspca_frame_add(gspca_dev, INTER_PACKET, frame,
			data, len);
}

/* -- management routines -- */

static void setbrightness(struct gspca_dev *gspca_dev)
{
	struct sd *sd = (struct sd *) gspca_dev;
	int val;
/*	int was_streaming; */

	val = sd->brightness;
	PDEBUG(D_CONF, "brightness:%d", val);
/*	was_streaming = gspca_dev->streaming;
 *	if (was_streaming)
 *		ov51x_stop(sd); */
	switch (sd->sensor) {
	case SEN_OV8610:
	case SEN_OV7610:
	case SEN_OV76BE:
	case SEN_OV6620:
	case SEN_OV6630:
	case SEN_OV7640:
		i2c_w(sd, OV7610_REG_BRT, val);
		break;
	case SEN_OV7620:
		/* 7620 doesn't like manual changes when in auto mode */
/*fixme
 *		if (!sd->auto_brt) */
			i2c_w(sd, OV7610_REG_BRT, val);
		break;
	case SEN_OV7670:
/*jfm - from windblows
 *		i2c_w_mask(sd, OV7670_REG_COM8, 0, OV7670_COM8_AEC); */
		i2c_w(sd, OV7670_REG_BRIGHT, ov7670_abs_to_sm(val));
		break;
	}
/*	if (was_streaming)
 *		ov51x_restart(sd); */
}

static void setcontrast(struct gspca_dev *gspca_dev)
{
	struct sd *sd = (struct sd *) gspca_dev;
	int val;
/*	int was_streaming; */

	val = sd->contrast;
	PDEBUG(D_CONF, "contrast:%d", val);
/*	was_streaming = gspca_dev->streaming;
	if (was_streaming)
		ov51x_stop(sd); */
	switch (sd->sensor) {
	case SEN_OV7610:
	case SEN_OV6620:
		i2c_w(sd, OV7610_REG_CNT, val);
		break;
	case SEN_OV6630:
		i2c_w_mask(sd, OV7610_REG_CNT, val >> 4, 0x0f);
	case SEN_OV8610: {
		static const __u8 ctab[] = {
			0x03, 0x09, 0x0b, 0x0f, 0x53, 0x6f, 0x35, 0x7f
		};

		/* Use Y gamma control instead. Bit 0 enables it. */
		i2c_w(sd, 0x64, ctab[val >> 5]);
		break;
	    }
	case SEN_OV7620: {
		static const __u8 ctab[] = {
			0x01, 0x05, 0x09, 0x11, 0x15, 0x35, 0x37, 0x57,
			0x5b, 0xa5, 0xa7, 0xc7, 0xc9, 0xcf, 0xef, 0xff
		};

		/* Use Y gamma control instead. Bit 0 enables it. */
		i2c_w(sd, 0x64, ctab[val >> 4]);
		break;
	    }
	case SEN_OV7640:
		/* Use gain control instead. */
		i2c_w(sd, OV7610_REG_GAIN, val >> 2);
		break;
	case SEN_OV7670:
		/* check that this isn't just the same as ov7610 */
		i2c_w(sd, OV7670_REG_CONTRAS, val >> 1);
		break;
	}
/*	if (was_streaming)
		ov51x_restart(sd); */
}

static void setcolors(struct gspca_dev *gspca_dev)
{
	struct sd *sd = (struct sd *) gspca_dev;
	int val;
/*	int was_streaming; */

	val = sd->colors;
	PDEBUG(D_CONF, "saturation:%d", val);
/*	was_streaming = gspca_dev->streaming;
	if (was_streaming)
		ov51x_stop(sd); */
	switch (sd->sensor) {
	case SEN_OV8610:
	case SEN_OV7610:
	case SEN_OV76BE:
	case SEN_OV6620:
	case SEN_OV6630:
		i2c_w(sd, OV7610_REG_SAT, val);
		break;
	case SEN_OV7620:
		/* Use UV gamma control instead. Bits 0 & 7 are reserved. */
/*		rc = ov_i2c_write(sd->dev, 0x62, (val >> 9) & 0x7e);
		if (rc < 0)
			goto out; */
		i2c_w(sd, OV7610_REG_SAT, val);
		break;
	case SEN_OV7640:
		i2c_w(sd, OV7610_REG_SAT, val & 0xf0);
		break;
	case SEN_OV7670:
		/* supported later once I work out how to do it
		 * transparently fail now! */
		/* set REG_COM13 values for UV sat auto mode */
		break;
	}
/*	if (was_streaming)
		ov51x_restart(sd); */
}

static int sd_setbrightness(struct gspca_dev *gspca_dev, __s32 val)
{
	struct sd *sd = (struct sd *) gspca_dev;

	sd->brightness = val;
	setbrightness(gspca_dev);
	return 0;
}

static int sd_getbrightness(struct gspca_dev *gspca_dev, __s32 *val)
{
	struct sd *sd = (struct sd *) gspca_dev;

	*val = sd->brightness;
	return 0;
}

static int sd_setcontrast(struct gspca_dev *gspca_dev, __s32 val)
{
	struct sd *sd = (struct sd *) gspca_dev;

	sd->contrast = val;
	setcontrast(gspca_dev);
	return 0;
}

static int sd_getcontrast(struct gspca_dev *gspca_dev, __s32 *val)
{
	struct sd *sd = (struct sd *) gspca_dev;

	*val = sd->contrast;
	return 0;
}

static int sd_setcolors(struct gspca_dev *gspca_dev, __s32 val)
{
	struct sd *sd = (struct sd *) gspca_dev;

	sd->colors = val;
	setcolors(gspca_dev);
	return 0;
}

static int sd_getcolors(struct gspca_dev *gspca_dev, __s32 *val)
{
	struct sd *sd = (struct sd *) gspca_dev;

	*val = sd->colors;
	return 0;
}

/* sub-driver description */
static const struct sd_desc sd_desc = {
	.name = MODULE_NAME,
	.ctrls = sd_ctrls,
	.nctrls = ARRAY_SIZE(sd_ctrls),
	.config = sd_config,
	.open = sd_open,
	.start = sd_start,
	.stopN = sd_stopN,
	.stop0 = sd_stop0,
	.close = sd_close,
	.pkt_scan = sd_pkt_scan,
};

/* -- module initialisation -- */
static const __devinitdata struct usb_device_id device_table[] = {
	{USB_DEVICE(0x041e, 0x4052)},
	{USB_DEVICE(0x041e, 0x405f)},
	{USB_DEVICE(0x041e, 0x4060)},
	{USB_DEVICE(0x041e, 0x4061)},
	{USB_DEVICE(0x041e, 0x4064)},
	{USB_DEVICE(0x041e, 0x4068)},
	{USB_DEVICE(0x045e, 0x028c)},
	{USB_DEVICE(0x054c, 0x0154)},
	{USB_DEVICE(0x054c, 0x0155)},
	{USB_DEVICE(0x05a9, 0x0519)},
	{USB_DEVICE(0x05a9, 0x0530)},
	{USB_DEVICE(0x05a9, 0x4519)},
	{USB_DEVICE(0x05a9, 0x8519)},
	{}
};
#undef DVNAME
MODULE_DEVICE_TABLE(usb, device_table);

/* -- device connect -- */
static int sd_probe(struct usb_interface *intf,
			const struct usb_device_id *id)
{
	return gspca_dev_probe(intf, id, &sd_desc, sizeof(struct sd),
				THIS_MODULE);
}

static struct usb_driver sd_driver = {
	.name = MODULE_NAME,
	.id_table = device_table,
	.probe = sd_probe,
	.disconnect = gspca_disconnect,
};

/* -- module insert / remove -- */
static int __init sd_mod_init(void)
{
	if (usb_register(&sd_driver) < 0)
		return -1;
	PDEBUG(D_PROBE, "registered");
	return 0;
}
static void __exit sd_mod_exit(void)
{
	usb_deregister(&sd_driver);
	PDEBUG(D_PROBE, "deregistered");
}

module_init(sd_mod_init);
module_exit(sd_mod_exit);

module_param(frame_rate, int, 0644);
MODULE_PARM_DESC(frame_rate, "Frame rate (5, 10, 15, 20 or 30 fps)");