linux-kernel - Re: [PATCH v2 0/7] [PATCH v2 0/7] Add support of OV9655 camera

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <26a55285-509c-b7f4-7806-db537a582631@st.com>
Date:   Thu, 6 Jul 2017 07:51:28 +0000
From:   Hugues FRUCHET <hugues.fruchet@...com>
To:     Sylwester Nawrocki <sylvester.nawrocki@...il.com>
CC:     "H. Nikolaus Schaller" <hns@...delico.com>,
        Guennadi Liakhovetski <g.liakhovetski@....de>,
        Rob Herring <robh+dt@...nel.org>,
        Mark Rutland <mark.rutland@....com>,
        Maxime Coquelin <mcoquelin.stm32@...il.com>,
        Alexandre TORGUE <alexandre.torgue@...com>,
        Mauro Carvalho Chehab <mchehab@...nel.org>,
        Hans Verkuil <hverkuil@...all.nl>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "linux-media@...r.kernel.org" <linux-media@...r.kernel.org>,
        Benjamin Gaignard <benjamin.gaignard@...aro.org>,
        Yannick FERTRE <yannick.fertre@...com>
Subject: Re: [PATCH v2 0/7] [PATCH v2 0/7] Add support of OV9655 camera

Hi Sylwester,

Do you have the possibility to check for non-regression of this patchset 
on 9650/52 camera ?

Best regards,
Hugues.

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 [1].
> OV9655 registers sequences come from STM32CubeF7 embedded software [2].
> 
> [1] http://git.goldelico.com/?p=gta04-kernel.git;a=shortlog;h=refs/heads/work/hns/video/ov9655
> [2] 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):
>        http://www.mail-archive.com/linux-media@vger.kernel.org/msg114616.html
>    - 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
>        http://www.mail-archive.com/linux-media@vger.kernel.org/msg114667.html
>    - dt-bindings: Fix remarks from Rob Herring about polarity:
>        http://www.mail-archive.com/linux-media@vger.kernel.org/msg114705.html
>    - dt-bindings: Add optional regulators avdd, dvdd, dovdd:
>        http://www.mail-archive.com/linux-media@vger.kernel.org/msg114785.html
>    - fix missing semicolons in if condition:
>        http://www.mail-archive.com/linux-media@vger.kernel.org/msg114611.html
>    - move ov965x_pixfmt relocation in right patch:
>        http://www.mail-archive.com/linux-media@vger.kernel.org/msg114849.html
>    - revisit OV965x renaming to ov965x for device id names and DT compatible strings,
>      drop of_device_id .data device identification
>        http://www.mail-archive.com/linux-media@vger.kernel.org/msg114635.html
>        http://www.mail-archive.com/linux-media@vger.kernel.org/msg114738.html
>    - Add analog power supply and clock gating, needed for GTA04 platform:
>        http://www.mail-archive.com/linux-media@vger.kernel.org/msg114519.html
> 
> 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
>