Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751450AbdFZGDv (ORCPT ); Mon, 26 Jun 2017 02:03:51 -0400 Received: from mo4-p00-ob.smtp.rzone.de ([81.169.146.220]:25519 "EHLO mo4-p00-ob.smtp.rzone.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751190AbdFZGDn (ORCPT ); Mon, 26 Jun 2017 02:03:43 -0400 X-RZG-AUTH: :JGIXVUS7cutRB/49FwqZ7WcecEarQROEYabkiUo6mSAGQ+qKID8yPJWK6w== X-RZG-CLASS-ID: mo00 Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (Mac OS X Mail 9.3 \(3124\)) Subject: Re: [PATCH v1 6/6] [media] ov9650: add support of OV9655 variant From: "H. Nikolaus Schaller" In-Reply-To: <1498143942-12682-7-git-send-email-hugues.fruchet@st.com> Date: Mon, 26 Jun 2017 08:03:07 +0200 Cc: Sylwester Nawrocki , Guennadi Liakhovetski , Rob Herring , Mark Rutland , Maxime Coquelin , Alexandre Torgue , Mauro Carvalho Chehab , Hans Verkuil , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, Benjamin Gaignard , Yannick Fertre Message-Id: References: <1498143942-12682-1-git-send-email-hugues.fruchet@st.com> <1498143942-12682-7-git-send-email-hugues.fruchet@st.com> To: Hugues Fruchet X-Mailer: Apple Mail (2.3124) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id v5Q642Cv015312 Content-Length: 27416 Lines: 720 > Am 22.06.2017 um 17:05 schrieb Hugues Fruchet : > > Add a first support of OV9655 variant. > Because of register set slightly different from OV9650/9652, > not all of the driver features are supported (controls). > Supported resolutions are limited to VGA, QVGA, QQVGA. > Supported format is limited to RGB565. > Controls are limited to color bar test pattern for test purpose. > > Signed-off-by: H. Nikolaus Schaller > Signed-off-by: Hugues Fruchet > --- > drivers/media/i2c/Kconfig | 4 +- > drivers/media/i2c/ov9650.c | 486 ++++++++++++++++++++++++++++++++++++++++++--- > 2 files changed, 457 insertions(+), 33 deletions(-) > > diff --git a/drivers/media/i2c/Kconfig b/drivers/media/i2c/Kconfig > index efea14d..a8f638c 100644 > --- a/drivers/media/i2c/Kconfig > +++ b/drivers/media/i2c/Kconfig > @@ -594,11 +594,11 @@ config VIDEO_OV7670 > controller. > > config VIDEO_OV9650 > - tristate "OmniVision OV9650/OV9652 sensor support" > + tristate "OmniVision OV9650/OV9652/OV9655 sensor support" > depends on GPIOLIB && I2C && VIDEO_V4L2 && VIDEO_V4L2_SUBDEV_API > ---help--- > This is a V4L2 sensor-level driver for the Omnivision > - OV9650 and OV9652 camera sensors. > + OV9650 and OV9652 and OV9655 camera sensors. > > config VIDEO_VS6624 > tristate "ST VS6624 sensor support" > diff --git a/drivers/media/i2c/ov9650.c b/drivers/media/i2c/ov9650.c > index a9d268d..c0819af 100644 > --- a/drivers/media/i2c/ov9650.c > +++ b/drivers/media/i2c/ov9650.c > @@ -1,5 +1,5 @@ > /* > - * Omnivision OV9650/OV9652 CMOS Image Sensor driver > + * Omnivision OV9650/OV9652/OV9655 CMOS Image Sensor driver > * > * Copyright (C) 2013, Sylwester Nawrocki > * > @@ -7,6 +7,15 @@ > * by Vladimir Fonov. > * Copyright (c) 2010, Vladimir Fonov > * > + * > + * Copyright (C) STMicroelectronics SA 2017 > + * Author: Hugues Fruchet for STMicroelectronics. > + * > + * OV9655 initial support based on a driver written by H. Nikolaus Schaller: > + * http://git.goldelico.com/?p=gta04-kernel.git;a=shortlog;h=refs/heads/work/hns/video/ov9655 > + * OV9655 registers sequence from STM32CubeF7 embedded software, see: > + * https://developer.mbed.org/teams/ST/code/BSP_DISCO_F746NG/file/e1d9da7fe856/Drivers/BSP/Components/ov9655/ov9655.c > + * > * This program is free software; you can redistribute it and/or modify > * it under the terms of the GNU General Public License version 2 as > * published by the Free Software Foundation. > @@ -58,14 +67,21 @@ > #define REG_PID 0x0a /* Product ID MSB */ > #define REG_VER 0x0b /* Product ID LSB */ > #define REG_COM3 0x0c > -#define COM3_SWAP 0x40 > +#define COM3_COLORBAR 0x80 > +#define COM3_RGB565 0x00 > +#define COM3_SWAP 0x40 /* Doesn't work in RGB */ > +#define COM3_RESETB 0x08 > #define COM3_VARIOPIXEL1 0x04 > +#define OV9655_SINGLEFRAME 0x01 > #define REG_COM4 0x0d /* Vario Pixels */ > #define COM4_VARIOPIXEL2 0x80 > +#define OV9655_TRISTATE /* seems to have a different function */ > #define REG_COM5 0x0e /* System clock options */ > #define COM5_SLAVE_MODE 0x10 > -#define COM5_SYSTEMCLOCK48MHZ 0x80 > +#define COM5_SYSTEMCLOCK48MHZ 0x80 /* not on OV9655 */ > +#define OV9655_EXPOSURESTEP 0x01 > #define REG_COM6 0x0f /* HREF & ADBLC options */ > +#define COM6_BLC_OPTICAL 0x40 /* Optical black */ > #define REG_AECH 0x10 /* Exposure value, AEC[9:2] */ > #define REG_CLKRC 0x11 /* Clock control */ > #define CLK_EXT 0x40 /* Use external clock directly */ > @@ -74,13 +90,18 @@ > #define COM7_RESET 0x80 > #define COM7_FMT_MASK 0x38 > #define COM7_FMT_VGA 0x40 > -#define COM7_FMT_CIF 0x20 > +#define COM7_FMT_CIF 0x20 > #define COM7_FMT_QVGA 0x10 > #define COM7_FMT_QCIF 0x08 > -#define COM7_RGB 0x04 > -#define COM7_YUV 0x00 > -#define COM7_BAYER 0x01 > -#define COM7_PBAYER 0x05 > +#define COM7_RGB 0x04 > +#define COM7_YUV 0x00 > +#define COM7_BAYER 0x01 > +#define COM7_PBAYER 0x05 > +#define OV9655_COM7_VGA 0x60 > +#define OV9655_COM7_RAWRGB 0x00 /* different format encoding */ > +#define OV9655_COM7_RAWRGBINT 0x01 > +#define OV9655_COM7_YUV 0x02 > +#define OV9655_COM7_RGB 0x03 > #define REG_COM8 0x13 /* AGC/AEC options */ > #define COM8_FASTAEC 0x80 /* Enable fast AGC/AEC */ > #define COM8_AECSTEP 0x40 /* Unlimited AEC step size */ > @@ -89,14 +110,23 @@ > #define COM8_AWB 0x02 /* White balance enable */ > #define COM8_AEC 0x01 /* Auto exposure enable */ > #define REG_COM9 0x14 /* Gain ceiling */ > -#define COM9_GAIN_CEIL_MASK 0x70 /* */ > +#define COM9_GAIN_CEIL_MASK 0x70 > +#define COM9_GAIN_CEIL_16X 0x30 > +#define OV9655_COM9_EXPTIMING 0x08 > +#define OV9655_COM9_VSYNCDROP 0x04 > +#define OV9655_COM9_AECDROP 0x02 > #define REG_COM10 0x15 /* PCLK, HREF, HSYNC signals polarity */ > +#define OV9655_SLAVE_PIN 0x80 /* SLHS/SLVS instead of RESETB/PWDN */ > #define COM10_HSYNC 0x40 /* HSYNC instead of HREF */ > #define COM10_PCLK_HB 0x20 /* Suppress PCLK on horiz blank */ > -#define COM10_HREF_REV 0x08 /* Reverse HREF */ > +#define OV9655_COM10_PCLK_REV 0x10 /* PCLK reverse */ > +#define COM10_HREF_REV 0x08 /* Reverse HREF */ > #define COM10_VS_LEAD 0x04 /* VSYNC on clock leading edge */ > +#define OV9655_COM10_RESET_OPTION 0x04 /* Reset signal end point */ > #define COM10_VS_NEG 0x02 /* VSYNC negative */ > #define COM10_HS_NEG 0x01 /* HSYNC negative */ > +#define OV9655_REG16 0x16 /* dummy frame and blanking */ > +#define OV9655_REG16_DUMMY_8 0x20 /* dummy frame when gain > 8 */ > #define REG_HSTART 0x17 /* Horiz start high bits */ > #define REG_HSTOP 0x18 /* Horiz stop high bits */ > #define REG_VSTART 0x19 /* Vert start high bits */ > @@ -117,6 +147,7 @@ > #define REG_BBIAS 0x27 /* B channel output bias */ > #define REG_GBBIAS 0x28 /* Gb channel output bias */ > #define REG_GRCOM 0x29 /* Analog BLC & regulator */ > +#define OV9655_PREGAIN 0x29 > #define REG_EXHCH 0x2a /* Dummy pixel insert MSB */ > #define REG_EXHCL 0x2b /* Dummy pixel insert LSB */ > #define REG_RBIAS 0x2c /* R channel output bias */ > @@ -127,12 +158,30 @@ > #define REG_HSYEN 0x31 /* HSYNC falling edge delay LSB*/ > #define REG_HREF 0x32 /* HREF pieces */ > #define REG_CHLF 0x33 /* reserved */ > +#define OV9655_CLKF 0x33 /* Array current control */ > +#define OV9655_AREF1 0x34 /* Array reference control */ > +#define OV9655_AREF2 0x35 /* Array reference control */ > +#define OV9655_AREF3 0x36 /* Array reference control */ > #define REG_ADC 0x37 /* reserved */ > +#define OV9655_ADC 0x37 /* ADC Control 1 (Range adjustment) */ > #define REG_ACOM 0x38 /* reserved */ > -#define REG_OFON 0x39 /* Power down register */ > +#define OV9655_ADC2 0x38 /* ADC Control 2 (Range adjustment) */ > +#define REG_OFON 0x39 /* Power down register (ov9650 only) */ > #define OFON_PWRDN 0x08 /* Power down bit */ > +#define OV9655_AREF4 0x39 /* Array reference control */ > #define REG_TSLB 0x3a /* YUVU format */ > +#define OV9655_PCLKDELAY2NS 0x40 > +#define OV9655_PCLKDELAY4NS 0x80 > +#define OV9655_PCLKDELAY6NS 0xc0 > +#define OV9655_OUTREVERSE 0x20 > +#define OV9655_FIXEDUV 0x10 > #define TSLB_YUYV_MASK 0x0c /* UYVY or VYUY - see com13 */ > +#define TSLB_YUYV 0x00 > +#define TSLB_YVYU 0x04 > +#define TSLB_VYUY 0x08 > +#define TSLB_UYVY 0x0c > +#define OV9655_BANDINGAUTO 0x02 > + > #define REG_COM11 0x3b /* Night mode, banding filter enable */ > #define COM11_NIGHT 0x80 /* Night mode enable */ > #define COM11_NMFR 0x60 /* Two bit NM frame rate */ > @@ -142,25 +191,38 @@ > #define COM12_HREF 0x80 /* HREF always */ > #define REG_COM13 0x3d /* Gamma selection, Color matrix en. */ > #define COM13_GAMMA 0x80 /* Gamma enable */ > -#define COM13_UVSAT 0x40 /* UV saturation auto adjustment */ > +#define COM13_UVSAT 0x40 /* UV saturation auto adjustment */ > +#define COM13_Y_DELAY 0x08 /* Delay Y channel */ > #define COM13_UVSWAP 0x01 /* V before U - w/TSLB */ > #define REG_COM14 0x3e /* Edge enhancement options */ > #define COM14_EDGE_EN 0x02 > #define COM14_EEF_X2 0x01 > +#define OV9655_REG_COM14 0x3e /* pixel correction/zoom ON/OFF sel. */ > +#define OV9655_COM14_BLACK_PIX 0x08 /* Black pixel correction */ > +#define OV9655_COM14_WHITE_PIX 0x04 /* White pixel correction */ > +#define OV9655_COM14_ZOOM 0x02 /* Zoom function ON */ > #define REG_EDGE 0x3f /* Edge enhancement factor */ > #define EDGE_FACTOR_MASK 0x0f > #define REG_COM15 0x40 /* Output range, RGB 555/565 */ > #define COM15_R10F0 0x00 /* Data range 10 to F0 */ > -#define COM15_R01FE 0x80 /* 01 to FE */ > +#define COM15_R01FE 0x80 /* 01 to FE */ > #define COM15_R00FF 0xc0 /* 00 to FF */ > #define COM15_RGB565 0x10 /* RGB565 output */ > #define COM15_RGB555 0x30 /* RGB555 output */ > #define COM15_SWAPRB 0x04 /* Swap R&B */ > #define REG_COM16 0x41 /* Color matrix coeff options */ > #define REG_COM17 0x42 /* Single frame out, banding filter */ > +#define OV9655_REG_COM17 0x42 /* Denoise, edge, auto gain, ... */ > +#define OV9655_COM17_EDGE_AUTO 0x40 /* Edge auto */ > +#define OV9655_COM17_DENOISE_AUTO 0x80 /* Denoise auto */ > +#define OV9655_REG_RSVD(__n) (0x43 + (__n) - 1) /* reserved but used... */ > /* n = 1...9, 0x4f..0x57 */ > -#define REG_MTX(__n) (0x4f + (__n) - 1) > +#define REG_MTX(__n) (0x4f + (__n) - 1) > #define REG_MTXS 0x58 > +#define REG_AWBOP(__n) (0x59 + (__n) - 1) /* AWB control options */ > +#define REG_BLMT 0x5F /* AWB Blue Component Gain Limit */ > +#define REG_RLMT 0x60 /* AWB Red Component Gain Limit */ > +#define REG_GLMT 0x61 /* AWB Green Component Gain Limit */ > /* Lens Correction Option 1...5, __n = 0...5 */ > #define REG_LCC(__n) (0x62 + (__n) - 1) > #define LCC5_LCC_ENABLE 0x01 /* LCC5, enable lens correction */ > @@ -170,10 +232,26 @@ > #define REG_HV 0x69 /* Manual banding filter MSB */ > #define REG_MBD 0x6a /* Manual banding filter value */ > #define REG_DBLV 0x6b /* reserved */ > +#define OV9655_REG_DBLV 0x6b /* PLL, DVDD regu bypass, bandgap */ > +#define OV9655_DBLV_BANDGAP 0x0a /* default value */ > +#define OV9655_DBLV_LDO_BYPASS 0x10 > +#define OV9655_DBLV_PLL_BYPASS 0x00 > +#define OV9655_DBLV_PLL_4X 0x40 > +#define OV9655_DBLV_PLL_6X 0x80 > +#define OV9655_DBLV_PLL_8X 0xc0 > #define REG_GSP 0x6c /* Gamma curve */ > #define GSP_LEN 15 > +#define OV9655_REG_DNSTH 0x70 /* De-noise Function Threshold Adj. */ > +#define OV9655_REG_POIDX 0x72 /* Pixel output index */ > +#define OV9655_REG_PCKDV 0x73 /* Pixel Clock Output Selection */ > +#define OV9655_REG_XINDX 0x74 /* Horizontal Scaling Down Coeff. */ > +#define OV9655_REG_YINDX 0x75 /* Vertical Scaling Down Coeff. */ > +#define OV9655_REG_SLOP 0x7A /* Gamma Curve Highest Segment Slope */ > +#define OV9655_REG_GAM(__n) (0x7B + (__n) - 1) /* Gamma curve */ > #define REG_GST 0x7c /* Gamma curve */ > #define GST_LEN 15 > +#define OV9655_REG_COM18 0x8b /* Zoom mode in VGA */ > +#define OV9655_REG_COM19 0x8c /* UV adjustment */ > #define REG_COM21 0x8b > #define REG_COM22 0x8c /* Edge enhancement, denoising */ > #define COM22_WHTPCOR 0x02 /* White pixel correction enable */ > @@ -181,6 +259,8 @@ > #define COM22_DENOISE 0x10 /* White pixel correction option */ > #define REG_COM23 0x8d /* Color bar test, color gain */ > #define COM23_TEST_MODE 0x10 > +#define OV9655_REG_COM20 0x8d > +#define OV9655_COM20_TEST_MODE 0x10 > #define REG_DBLC1 0x8f /* Digital BLC */ > #define REG_DBLC_B 0x90 /* Digital BLC B channel offset */ > #define REG_DBLC_R 0x91 /* Digital BLC R channel offset */ > @@ -193,6 +273,17 @@ > #define REG_AECHM 0xa1 /* Exposure value - bits AEC[15:10] */ > #define REG_BD50ST 0xa2 /* Banding filter value for 50Hz */ > #define REG_BD60ST 0xa3 /* Banding filter value for 60Hz */ > +#define OV9655_REG_COM21 0xa4 /* Digital gain */ > +#define OV9655_REG_AWB_GREEN 0xa6 /* AWB green */ > +#define OV9655_REG_REF_A8 0xa8 /* Analog Reference Control */ > +#define OV9655_REG_REF_A9 0xa9 /* Analog Reference Control */ > +#define OV9655_REG_BLC(__n) (0xac + (__n) - 1) /* Black Level Control */ > +#define OV9655_REG_CTRLB4 0xb4 /* UV adjustment */ > +#define OV9655_REG_ADBOFF 0xbc /* ADC B channel offset setting */ > +#define OV9655_REG_ADROFF 0xbd /* ADC R channel offset setting */ > +#define OV9655_REG_ADGBOFF 0xbe /* ADC Gb channel offset setting */ > +#define OV9655_REG_ADGEOFF 0xbf /* ADC Gr channel offset setting */ > +#define OV9655_REG_COM24 0xc7 /* Pixel clock frequency selection */ > #define REG_NULL 0xff /* Array end token */ > > #define DEF_CLKRC 0x80 > @@ -200,6 +291,7 @@ > #define OV965X_ID(_msb, _lsb) ((_msb) << 8 | (_lsb)) > #define OV9650_ID 0x9650 > #define OV9652_ID 0x9652 > +#define OV9655V5_ID 0x9657 > > struct ov965x_ctrls { > struct v4l2_ctrl_handler handler; > @@ -458,6 +550,292 @@ struct ov965x_pixfmt { > {{ 1, 25 }, { QVGA_WIDTH, QVGA_HEIGHT }, 1 }, /* 25 fps */ > }; > > +/* OV9655 */ > +static const struct i2c_rv ov9655_init_regs[] = { > + { REG_GAIN, 0x00 }, > + { REG_BLUE, 0x80 }, > + { REG_RED, 0x80 }, > + { REG_VREF, 0x02 }, > + { REG_COM1, 0x03 }, > + { REG_COM2, 0x01 },/* Output drive x2 */ > + { REG_COM3, COM3_RGB565 },/* Output drive x2, RGB565 */ > + { REG_COM5, 0x60 | OV9655_EXPOSURESTEP },/* 0x60 ? */ > + { REG_COM6, COM6_BLC_OPTICAL }, > + { REG_CLKRC, 0x01 },/* F(internal clk) = F(input clk) / 2 */ > + { REG_COM7, OV9655_COM7_VGA | OV9655_COM7_YUV }, > + { REG_COM8, COM8_FASTAEC | COM8_AECSTEP | > + COM8_AGC | COM8_AWB | COM8_AEC }, > + { REG_COM9, COM9_GAIN_CEIL_16X | OV9655_COM9_EXPTIMING | > + OV9655_COM9_AECDROP }, > + { OV9655_REG16, OV9655_REG16_DUMMY_8 | 0x4 }, > + { REG_HSTART, 0x18 }, > + { REG_HSTOP, 0x04 }, > + { REG_VSTART, 0x01 }, > + { REG_VSTOP, 0x81 }, > + { REG_MVFP, 0x00 },/* No mirror/flip */ > + { REG_AEW, 0x3c }, > + { REG_AEB, 0x36 }, > + { REG_VPT, 0x72 }, > + { REG_BBIAS, 0x08 }, > + { REG_GBBIAS, 0x08 }, > + { OV9655_PREGAIN, 0x15 }, > + { REG_EXHCH, 0x00 }, > + { REG_EXHCL, 0x00 }, > + { REG_RBIAS, 0x08 }, > + { REG_HREF, 0x12 },/* QVGA */ > + { REG_CHLF, 0x00 }, > + { OV9655_AREF1, 0x3f }, > + { OV9655_AREF2, 0x00 }, > + { OV9655_AREF3, 0x3a }, > + { OV9655_ADC2, 0x72 }, > + { OV9655_AREF4, 0x57 }, > + { REG_TSLB, OV9655_PCLKDELAY6NS | TSLB_UYVY }, > + { REG_COM11, 0x04 },/* 0x04 ? */ > + { REG_COM13, COM13_GAMMA | 0x10 | > + COM13_Y_DELAY | COM13_UVSWAP },/* 0x10 ? */ > + {OV9655_REG_COM14, OV9655_COM14_ZOOM }, /* QVGA */ > + { REG_EDGE, 0xc1 }, > + { REG_COM15, COM15_R00FF },/* Full range output */ > + { REG_COM16, 0x41 },/* 0x41 ? */ > + { OV9655_REG_COM17, OV9655_COM17_EDGE_AUTO | > + OV9655_COM17_DENOISE_AUTO }, > + { OV9655_REG_RSVD(1), 0x0a }, > + { OV9655_REG_RSVD(2), 0xf0 }, > + { OV9655_REG_RSVD(3), 0x46 }, > + { OV9655_REG_RSVD(4), 0x62 }, > + { OV9655_REG_RSVD(5), 0x2a }, > + { OV9655_REG_RSVD(6), 0x3c }, > + { OV9655_REG_RSVD(7), 0xfc }, > + { OV9655_REG_RSVD(8), 0xfc }, > + { OV9655_REG_RSVD(9), 0x7f }, > + { OV9655_REG_RSVD(10), 0x7f }, > + { OV9655_REG_RSVD(11), 0x7f }, > + { REG_MTX(1), 0x98 }, > + { REG_MTX(2), 0x98 }, > + { REG_MTX(3), 0x00 }, > + { REG_MTX(4), 0x28 }, > + { REG_MTX(5), 0x70 }, > + { REG_MTX(6), 0x98 }, > + { REG_MTXS, 0x1a }, > + { REG_AWBOP(1), 0x85 }, > + { REG_AWBOP(2), 0xa9 }, > + { REG_AWBOP(3), 0x64 }, > + { REG_AWBOP(4), 0x84 }, > + { REG_AWBOP(5), 0x53 }, > + { REG_AWBOP(6), 0x0e }, > + { REG_BLMT, 0xf0 }, > + { REG_RLMT, 0xf0 }, > + { REG_GLMT, 0xf0 }, > + { REG_LCC(1), 0x00 }, > + { REG_LCC(2), 0x00 }, > + { REG_LCC(3), 0x02 }, > + { REG_LCC(4), 0x20 }, > + { REG_LCC(5), 0x00 }, > + { 0x69, 0x0a },/* Reserved... */ > + { OV9655_REG_DBLV, OV9655_DBLV_PLL_4X | OV9655_DBLV_LDO_BYPASS | > + OV9655_DBLV_BANDGAP }, > + { 0x6c, 0x04 },/* Reserved... */ > + { 0x6d, 0x55 },/* Reserved... */ > + { 0x6e, 0x00 },/* Reserved... */ > + { 0x6f, 0x9d },/* Reserved... */ > + { OV9655_REG_DNSTH, 0x21 }, > + { 0x71, 0x78 },/* Reserved... */ > + { OV9655_REG_POIDX, 0x11 },/* QVGA */ > + { OV9655_REG_PCKDV, 0x01 },/* QVGA */ > + { OV9655_REG_XINDX, 0x10 }, > + { OV9655_REG_YINDX, 0x10 }, > + { 0x76, 0x01 },/* Reserved... */ > + { 0x77, 0x02 },/* Reserved... */ > + { 0x7A, 0x12 },/* Reserved... */ > + { OV9655_REG_GAM(1), 0x08 }, > + { OV9655_REG_GAM(2), 0x16 }, > + { OV9655_REG_GAM(3), 0x30 }, > + { OV9655_REG_GAM(4), 0x5e }, > + { OV9655_REG_GAM(5), 0x72 }, > + { OV9655_REG_GAM(6), 0x82 }, > + { OV9655_REG_GAM(7), 0x8e }, > + { OV9655_REG_GAM(8), 0x9a }, > + { OV9655_REG_GAM(9), 0xa4 }, > + { OV9655_REG_GAM(10), 0xac }, > + { OV9655_REG_GAM(11), 0xb8 }, > + { OV9655_REG_GAM(12), 0xc3 }, > + { OV9655_REG_GAM(13), 0xd6 }, > + { OV9655_REG_GAM(14), 0xe6 }, > + { OV9655_REG_GAM(15), 0xf2 }, > + { 0x8a, 0x24 },/* Reserved... */ > + { OV9655_REG_COM19, 0x80 }, > + { 0x90, 0x7d },/* Reserved... */ > + { 0x91, 0x7b },/* Reserved... */ > + { REG_LCCFB, 0x02 }, > + { REG_LCCFR, 0x02 }, > + { REG_DBLC_GB, 0x7a }, > + { REG_DBLC_GR, 0x79 }, > + { REG_AECHM, 0x40 }, > + { OV9655_REG_COM21, 0x50 }, > + { 0xa5, 0x68 },/* Reserved... */ > + { OV9655_REG_AWB_GREEN, 0x4a }, > + { OV9655_REG_REF_A8, 0xc1 }, > + { OV9655_REG_REF_A9, 0xef }, > + { 0xaa, 0x92 },/* Reserved... */ > + { 0xab, 0x04 },/* Reserved... */ > + { OV9655_REG_BLC(1), 0x80 }, > + { OV9655_REG_BLC(2), 0x80 }, > + { OV9655_REG_BLC(3), 0x80 }, > + { OV9655_REG_BLC(4), 0x80 }, > + { OV9655_REG_BLC(7), 0xf2 }, > + { OV9655_REG_BLC(8), 0x20 }, > + { OV9655_REG_CTRLB4, 0x20 }, > + { 0xb5, 0x00 },/* Reserved... */ > + { 0xb6, 0xaf },/* Reserved... */ > + { 0xb6, 0xaf },/* Reserved... */ > + { 0xbb, 0xae },/* Reserved... */ > + { OV9655_REG_ADBOFF, 0x7f }, > + { OV9655_REG_ADROFF, 0x7f }, > + { OV9655_REG_ADGBOFF, 0x7f }, > + { OV9655_REG_ADGEOFF, 0x7f }, > + { OV9655_REG_ADGEOFF, 0x7f }, > + { 0xc0, 0xaa },/* Reserved... */ > + { 0xc1, 0xc0 },/* Reserved... */ > + { 0xc2, 0x01 },/* Reserved... */ > + { 0xc3, 0x4e },/* Reserved... */ > + { 0xc6, 0x05 },/* Reserved... */ > + { OV9655_REG_COM24, 0x81 },/* QVGA */ > + { 0xc9, 0xe0 },/* Reserved... */ > + { 0xca, 0xe8 },/* Reserved... */ > + { 0xcb, 0xf0 },/* Reserved... */ > + { 0xcc, 0xd8 },/* Reserved... */ > + { 0xcd, 0x93 },/* Reserved... */ > + { REG_COM7, OV9655_COM7_VGA | OV9655_COM7_RGB }, > + { REG_COM15, COM15_RGB565 }, > + { REG_NULL, 0} > +}; > + > +static const struct i2c_rv ov9655_qvga_regs[] = { > + { REG_HREF, 0x12 }, > + { OV9655_REG_COM14, OV9655_COM14_ZOOM }, > + { OV9655_REG_POIDX, 0x11 }, > + { OV9655_REG_PCKDV, 0x01 }, > + { OV9655_REG_COM24, 0x81 }, > + { REG_NULL, 0} > +}; > + > +static const struct i2c_rv ov9655_qqvga_regs[] = { > + { REG_HREF, 0xa4 }, > + { REG_COM14, OV9655_COM14_BLACK_PIX | OV9655_COM14_WHITE_PIX | > + OV9655_COM14_ZOOM }, > + { OV9655_REG_POIDX, 0x22 }, > + { OV9655_REG_PCKDV, 0x02 }, > + { OV9655_REG_COM24, 0x82 }, > + { REG_NULL, 0} > +}; > + > +static const struct i2c_rv ov9655_vga_regs[] = { > + { REG_GAIN, 0x11 }, > + { REG_VREF, 0x12 }, > + { REG_B_AVE, 0x2e }, > + { REG_GB_AVE, 0x2e }, > + { REG_GR_AVE, 0x2e }, > + { REG_R_AVE, 0x2e }, > + { REG_COM6, 0x48 }, > + { REG_AECH, 0x7b }, > + { REG_CLKRC, 0x03 }, > + { REG_COM8, COM8_FASTAEC | COM8_AECSTEP | COM8_BFILT | > + COM8_AGC | COM8_AWB | COM8_AEC }, > + { REG_HSTART, 0x16 }, > + { REG_HSTOP, 0x02 }, > + { REG_VSTART, 0x01 }, > + { REG_VSTOP, 0x3d }, > + { REG_MVFP, 0x04 }, > + { REG_YAVE, 0x2e }, > + { REG_HREF, 0xff }, > + { OV9655_AREF1, 0x3d }, > + { OV9655_AREF3, 0xfa }, > + { REG_TSLB, 0xcc }, > + { REG_COM11, 0xcc }, > + { REG_COM14, 0x0c }, > + { REG_EDGE, 0x82 }, > + { REG_COM15, COM15_R00FF | COM15_RGB565 },/* full range */ > + { REG_COM16, 0x40 }, > + { OV9655_REG_RSVD(1), 0x14 }, > + { OV9655_REG_RSVD(2), 0xf0 }, > + { OV9655_REG_RSVD(3), 0x46 }, > + { OV9655_REG_RSVD(4), 0x62 }, > + { OV9655_REG_RSVD(5), 0x2a }, > + { OV9655_REG_RSVD(6), 0x3c }, > + { OV9655_REG_RSVD(8), 0xe9 }, > + { OV9655_REG_RSVD(9), 0xdd }, > + { OV9655_REG_RSVD(10), 0xdd }, > + { OV9655_REG_RSVD(11), 0xdd }, > + { OV9655_REG_RSVD(12), 0xdd }, > + { REG_LCC(1), 0x00 }, > + { REG_LCC(2), 0x00 }, > + { REG_LCC(3), 0x02 }, > + { REG_LCC(4), 0x20 }, > + { REG_LCC(5), 0x01 }, > + { REG_GSP, 0x0c }, > + { 0x6f, 0x9e },/* Reserved... */ > + { OV9655_REG_DNSTH, 0x06 }, > + { OV9655_REG_POIDX, 0x00 }, > + { OV9655_REG_PCKDV, 0x00 }, > + { OV9655_REG_XINDX, 0x3a }, > + { OV9655_REG_YINDX, 0x35 }, > + { OV9655_REG_SLOP, 0x20 }, > + { OV9655_REG_GAM(1), 0x1c }, > + { OV9655_REG_GAM(2), 0x28 }, > + { OV9655_REG_GAM(3), 0x3c }, > + { OV9655_REG_GAM(4), 0x5a }, > + { OV9655_REG_GAM(5), 0x68 }, > + { OV9655_REG_GAM(6), 0x76 }, > + { OV9655_REG_GAM(7), 0x80 }, > + { OV9655_REG_GAM(8), 0x88 }, > + { OV9655_REG_GAM(9), 0x8f }, > + { OV9655_REG_GAM(10), 0x96 }, > + { OV9655_REG_GAM(11), 0xa3 }, > + { OV9655_REG_GAM(12), 0xaf }, > + { OV9655_REG_GAM(13), 0xc4 }, > + { OV9655_REG_GAM(14), 0xd7 }, > + { OV9655_REG_GAM(15), 0xe8 }, > + { 0x8a, 0x23 },/* Reserved... */ > + { OV9655_REG_COM19, 0x8d }, > + { 0x90, 0x92 },/* Reserved... */ > + { 0x91, 0x92 },/* Reserved... */ > + { REG_DBLC_GB, 0x90 }, > + { REG_DBLC_GR, 0x90 }, > + { OV9655_REG_AWB_GREEN, 0x40 }, > + { OV9655_REG_ADBOFF, 0x02 }, > + { OV9655_REG_ADROFF, 0x01 }, > + { OV9655_REG_ADGBOFF, 0x02 }, > + { OV9655_REG_ADGEOFF, 0x01 }, > + { 0xc1, 0xc8 },/* Reserved... */ > + { 0xc6, 0x85 },/* Reserved... */ > + { OV9655_REG_COM24, 0x80 }, > + { REG_NULL, 0} > +}; > + > +static const struct ov965x_framesize ov9655_framesizes[] = { > + { > + .width = VGA_WIDTH, > + .height = VGA_HEIGHT, > + .regs = ov9655_vga_regs, > + .max_exp_lines = 498, > + }, { > + .width = QVGA_WIDTH, > + .height = QVGA_HEIGHT, > + .regs = ov9655_qvga_regs, > + .max_exp_lines = 248, > + }, > + { > + .width = QQVGA_WIDTH, > + .height = QQVGA_HEIGHT, > + .regs = ov9655_qqvga_regs, > + .max_exp_lines = 124, > + }, > +}; > + > +static const struct ov965x_pixfmt ov9655_formats[] = { > + { MEDIA_BUS_FMT_RGB565_2X8_LE, V4L2_COLORSPACE_SRGB, 0x08}, > +}; > + > static inline struct v4l2_subdev *ctrl_to_sd(struct v4l2_ctrl *ctrl) > { > return &container_of(ctrl->handler, struct ov965x, ctrls.handler)->sd; > @@ -894,12 +1272,16 @@ static int ov965x_set_test_pattern(struct ov965x *ov965x, int value) > { > int ret; > u8 reg; > + u8 addr = (ov965x->id == OV9655V5_ID) ? > + REG_COM3 : REG_COM23; > + u8 mask = (ov965x->id == OV9655V5_ID) ? > + COM3_COLORBAR : COM23_TEST_MODE; > > - ret = ov965x_read(ov965x->client, REG_COM23, ®); > + ret = ov965x_read(ov965x->client, addr, ®); > if (ret < 0) > return ret; > - reg = value ? reg | COM23_TEST_MODE : reg & ~COM23_TEST_MODE; > - return ov965x_write(ov965x->client, REG_COM23, reg); > + reg = value ? reg | mask : reg & ~mask; > + return ov965x_write(ov965x->client, addr, reg); > } > > static int __g_volatile_ctrl(struct ov965x *ov965x, struct v4l2_ctrl *ctrl) > @@ -1102,6 +1484,30 @@ static int ov965x_initialize_controls(struct ov965x *ov965x) > return 0; > } > > +static int ov9655_initialize_controls(struct ov965x *ov965x) > +{ > + const struct v4l2_ctrl_ops *ops = &ov965x_ctrl_ops; > + struct ov965x_ctrls *ctrls = &ov965x->ctrls; > + struct v4l2_ctrl_handler *hdl = &ctrls->handler; > + int ret; > + > + ret = v4l2_ctrl_handler_init(hdl, 16); > + if (ret < 0) > + return ret; > + > + v4l2_ctrl_new_std_menu_items(hdl, ops, V4L2_CID_TEST_PATTERN, > + ARRAY_SIZE(test_pattern_menu) - 1, 0, 0, > + test_pattern_menu); > + if (hdl->error) { > + ret = hdl->error; > + v4l2_ctrl_handler_free(hdl); > + return ret; > + } > + > + ov965x->sd.ctrl_handler = hdl; > + return 0; > +} > + > /* > * V4L2 subdev video and pad level operations > */ > @@ -1518,9 +1924,15 @@ static int ov965x_detect_sensor(struct v4l2_subdev *sd) > > if (!ret) { > ov965x->id = OV965X_ID(pid, ver); > - if (ov965x->id == OV9650_ID || ov965x->id == OV9652_ID) { > + switch (ov965x->id) { > + case OV9650_ID: > + case OV9652_ID: > v4l2_info(sd, "Found OV%04X sensor\n", ov965x->id); > - } else { > + break; > + case OV9655V5_ID: > + v4l2_info(sd, "Found OV%04X sensor\n", ov965x->id - 2); > + break; > + default: > v4l2_err(sd, "Sensor detection failed (%04X, %d)\n", > ov965x->id, ret); > ret = -ENODEV; > @@ -1598,18 +2010,28 @@ static int ov965x_probe(struct i2c_client *client, > if (ret < 0) > goto err_me; > > - ov965x->init_regs = ov965x_init_regs; > - ov965x->initialize_controls = ov965x_initialize_controls; > - ov965x->framesizes = ov965x_framesizes; > - ov965x->nb_of_framesizes = ARRAY_SIZE(ov965x_framesizes); > - ov965x->formats = ov965x_formats; > - ov965x->nb_of_formats = ARRAY_SIZE(ov965x_formats); > - ov965x->intervals = ov965x_intervals; > - ov965x->nb_of_intervals = ARRAY_SIZE(ov965x_intervals); > - ov965x->fiv = &ov965x_intervals[0]; > - ov965x->set_frame_interval = __ov965x_set_frame_interval; > - ov965x->update_exposure_ctrl = ov965x_update_exposure_ctrl; > - ov965x->set_params = __ov965x_set_params; > + if (ov965x->id != OV9655V5_ID) { > + ov965x->init_regs = ov965x_init_regs; > + ov965x->initialize_controls = ov965x_initialize_controls; > + ov965x->framesizes = ov965x_framesizes; > + ov965x->nb_of_framesizes = ARRAY_SIZE(ov965x_framesizes); > + ov965x->formats = ov965x_formats; > + ov965x->nb_of_formats = ARRAY_SIZE(ov965x_formats); > + ov965x->intervals = ov965x_intervals; > + ov965x->nb_of_intervals = ARRAY_SIZE(ov965x_intervals); > + ov965x->fiv = &ov965x_intervals[0]; > + ov965x->set_frame_interval = __ov965x_set_frame_interval; > + ov965x->update_exposure_ctrl = ov965x_update_exposure_ctrl; > + ov965x->set_params = __ov965x_set_params; > + } else { > + ov965x->init_regs = ov9655_init_regs; > + ov965x->initialize_controls = ov9655_initialize_controls; > + ov965x->framesizes = ov9655_framesizes; > + ov965x->nb_of_framesizes = ARRAY_SIZE(ov9655_framesizes); > + ov965x->formats = ov9655_formats; > + ov965x->nb_of_formats = ARRAY_SIZE(ov9655_formats); > + ov965x->set_params = ov965x_set_frame_size; > + }; > > ov965x->frame_size = &ov965x->framesizes[0]; > ov965x_get_default_format(ov965x, &ov965x->format); > @@ -1652,6 +2074,7 @@ static int ov965x_remove(struct i2c_client *client) > static const struct i2c_device_id ov965x_id[] = { > { "OV9650", 0x9650 }, > { "OV9652", 0x9652 }, > + { "OV9655", 0x9655 }, i2c device ids should be lower case to match compatible-strings in DT for automatic modprobing. Please pick/merge/copy&paste With this change I get: root@letux:~# dmesg|fgrep ov96 [ 12.727600] ov965x: Found OV9655 sensor [ 12.747711] ov965x 1-0030: ov965x driver probed root@letux:~# during probe. > { /* sentinel */ } > }; > MODULE_DEVICE_TABLE(i2c, ov965x_id); > @@ -1659,6 +2082,7 @@ static int ov965x_remove(struct i2c_client *client) > static const struct of_device_id ov965x_of_match[] = { > { .compatible = "ovti,ov9650", .data = (void *)0x9650 }, > { .compatible = "ovti,ov9652", .data = (void *)0x9652 }, > + { .compatible = "ovti,ov9655", .data = (void *)0x9655 }, > { /* sentinel */ } > }; > MODULE_DEVICE_TABLE(of, ov965x_of_match); > -- > 1.9.1 >