Re: [PATCH v2 0/7] [PATCH v2 0/7] Add support of OV9655 camera
From: Hugues FRUCHET
Date: Thu Jul 06 2017 - 04:00:51 EST
Do you have the possibility to check for non-regression of this patchset
on 9650/52 camera ?
On 07/03/2017 11:16 AM, Hugues Fruchet wrote:
> This patchset enables OV9655 camera support.
> OV9655 support has been tested using STM32F4DIS-CAM extension board
> plugged on connector P1 of STM32F746G-DISCO board.
> Due to lack of OV9650/52 hardware support, the modified related code
> could not have been checked for non-regression.
> First patches upgrade current support of OV9650/52 to prepare then
> introduction of OV9655 variant patch.
> Because of OV9655 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.
> OV9655 initial support is based on a driver written by H. Nikolaus Schaller .
> OV9655 registers sequences come from STM32CubeF7 embedded software .
>  http://git.goldelico.com/?p=gta04-kernel.git;a=shortlog;h=refs/heads/work/hns/video/ov9655
>  https://developer.mbed.org/teams/ST/code/BSP_DISCO_F746NG/file/e1d9da7fe856/Drivers/BSP/Components/ov9655/ov9655.c
> = history =
> version 2:
> - Remove some unneeded semicolons (kbuild test robot):
> - Remove patch [media] ov9650: select the nearest higher resolution:
> it is up to the application to find the best matching resolution
> using ENUM_FRAMESIZES/S_FMT/S_SELECTION (S_CROP), see
> - dt-bindings: Fix remarks from Rob Herring about polarity:
> - dt-bindings: Add optional regulators avdd, dvdd, dovdd:
> - fix missing semicolons in if condition:
> - move ov965x_pixfmt relocation in right patch:
> - revisit OV965x renaming to ov965x for device id names and DT compatible strings,
> drop of_device_id .data device identification
> - Add analog power supply and clock gating, needed for GTA04 platform:
> version 1:
> - Initial submission.
> H. Nikolaus Schaller (1):
> DT bindings: add bindings for ov965x camera module
> Hugues Fruchet (6):
> [media] ov9650: switch i2c device id to lower case
> [media] ov9650: add device tree support
> [media] ov9650: use write_array() for resolution sequences
> [media] ov9650: add multiple variant support
> [media] ov9650: add support of OV9655 variant
> [media] ov9650: add analog power supply and clock gating
> .../devicetree/bindings/media/i2c/ov965x.txt | 45 ++
> drivers/media/i2c/Kconfig | 6 +-
> drivers/media/i2c/ov9650.c | 816 +++++++++++++++++----
> 3 files changed, 736 insertions(+), 131 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/media/i2c/ov965x.txt