linux-kernel - Re: [PATCH v4 2/2] media: i2c: ov02e10: add OV02E10 image sensor driver

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] [day] [month] [year] [list]

Message-ID: <1d88dd72-8c5d-459a-b681-6a44d32e93df@ixit.cz>
Date: Fri, 18 Apr 2025 18:24:03 +0200
From: David Heidelberg <david@...t.cz>
To: Bryan O'Donoghue <bryan.odonoghue@...aro.org>,
 Mauro Carvalho Chehab <mchehab@...nel.org>,
 Sakari Ailus <sakari.ailus@...ux.intel.com>,
 Liam Girdwood <lgirdwood@...il.com>, Mark Brown <broonie@...nel.org>,
 Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
 Conor Dooley <conor+dt@...nel.org>, Bryan O'Donoghue <bod@...nel.org>,
 Hans de Goede <hansg@...nel.org>
Cc: Jingjing Xiong <jingjing.xiong@...el.com>, Hao Yao <hao.yao@...el.com>,
 Jim Lai <jim.lai@...el.com>, You-Sheng Yang <vicamo.yang@...onical.com>,
 Alan Stern <stern@...land.harvard.edu>, Hans de Goede <hdegoede@...hat.com>,
 linux-kernel@...r.kernel.org, linux-media@...r.kernel.org,
 devicetree@...r.kernel.org
Subject: Re: [PATCH v4 2/2] media: i2c: ov02e10: add OV02E10 image sensor
 driver

Tested-by: David Heidelberg <david@...t.cz>  # Dell Latitude 9440 2024

Thank you!
David

On 07/04/2025 11:13, Bryan O'Donoghue wrote:
> From: Jingjing Xiong <jingjing.xiong@...el.com>
> 
> Add in the ov02e10 driver from the Intel IPU6 repository.
> 
> Changelog to IPU6 incoming driver:
> 
> - handhake GPIO
>    The IPU6 handshake GPIO signals to the FPGA on IPU6/IPU7 laptops and as
>    such is not a sensor signal.
> 
>    Hans has a proposal for handling those GPIOs below. Dropf from the
>    incoming sensor driver.
> 
>    Link: https://lore.kernel.org/linux-media/4b87a956-a767-48dc-b98b-f80d9a44adc8@redhat.com
> 
> - Regulators
>    Change to regulator_bulk API to capture the full range of rails to the
>    o02e10 sensor.
> 
>    This will work for ACPI systems which only have the avdd rail available
>    as missing rails won't fail the regulator_bulk_* function calls.
> 
>    Convert the existing avdd single rail to regulator_bulk API.
> 
> - Clocks
> 
>    Move the MCLK validation logic to check_hwcfg() and move check_hwcfg()
>    to after devm_kzalloc() so that we can pass a struct ov02e pointer to
>    check_hwcfg() to begin to enumerate that structure.
> 
>    Reworks MCLK clock get logic to facilitate OF
> 
> - Power on/off
> 
>    Implement specification on and off delays on power-up
> 
>    The ov02e10 specification says for power-on:
> 
>    t3 = the time between dvdd stable and XSHUTDOWN deassert
>    t5 = the time between XSHUTDOWN deassert and SCCB ready
> 
>    The power-off path in the spec shows no required delays between XSHUTDOWN
>    and power-rail shut off but, if the reset GPIO is not defined no action is
>    taken to change its logical state.
> 
> - Remove dev_dbg() and extra dev_err()
> 
>    Several instances of dev_dbg() and dev_err() were removed.
>    dev_xxx() calls common in development can be stripped away for upstream.
> 
>    We have multiple on-list tests for this code on X86 and Arm so most of
>    the verbose dev_xxx() calls are redundant now.
> 
> - Add OF probe support
> 
>    Simple addition of ovti,0v02e10 compat string and associated resource
>    structure entry.
> 
> - Extend MODULE_AUTHOR list
>    Adds Hans, Alan and myself
> 
> - Conversion to v4l2 subdevice-state
> 
> - Added HFLIP and VFLIP support
> 
> Signed-off-by: Jingjing Xiong <jingjing.xiong@...el.com>
> Co-developed-by: Hao Yao <hao.yao@...el.com>
> Signed-off-by: Hao Yao <hao.yao@...el.com>
> Co-developed-by: Jim Lai <jim.lai@...el.com>
> Signed-off-by: Jim Lai <jim.lai@...el.com>
> Co-developed-by: You-Sheng Yang <vicamo.yang@...onical.com>
> Signed-off-by: You-Sheng Yang <vicamo.yang@...onical.com>
> Co-developed-by: Alan Stern <stern@...land.harvard.edu>
> Signed-off-by: Alan Stern <stern@...land.harvard.edu>
> Co-developed-by: Hans de Goede <hdegoede@...hat.com>
> Signed-off-by: Hans de Goede <hdegoede@...hat.com>
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@...aro.org>
> ---
>   MAINTAINERS                 |   9 +
>   drivers/media/i2c/Kconfig   |  10 +
>   drivers/media/i2c/Makefile  |   1 +
>   drivers/media/i2c/ov02e10.c | 969 ++++++++++++++++++++++++++++++++++++++++++++
>   4 files changed, 989 insertions(+)
>