From: Sylvain Petinot <sylvain.petinot@foss.st.com>
To: <benjamin.mugnier@foss.st.com>, <sylvain.petinot@foss.st.com>,
<mchehab@kernel.org>, <robh@kernel.org>,
<krzysztof.kozlowski+dt@linaro.org>, <conor+dt@kernel.org>
Cc: <linux-media@vger.kernel.org>, <devicetree@vger.kernel.org>,
<linux-kernel@vger.kernel.org>
Subject: [PATCH v2 0/2] media: Add driver for ST VD56G3 camera sensor
Date: Tue, 21 May 2024 18:29:48 +0200 [thread overview]
Message-ID: <20240521162950.6987-1-sylvain.petinot@foss.st.com> (raw)
Hello,
This serie adds support for STMicroelectronics VD56G3 camera sensor.
This is a 1.5M pixel global shutter camera available in both Mono (VD56G3) and
colour (VD66GY) variants.
The following features are supported:
- Auto exposure with expo bias or
- Manual exposure with analog / digital gain
- H/V flip
- vblank/hblank/link freq
- Test pattern
- Supported resolutions in both raw8/raw10 :
- 1124x1364
- 1120x1360
- 1024x1280
- 1024x768
- 768x1024
- 720x1280
- 640x480
- 480x640
- 320x240
This driver supports coldstart parameters for internal AE feature.
To make it work, the driver save gain/expo values in ctrl's cur.val during
poweroff phase. This implementation transgress V4L2 rules... Any advice to make
it proper would be greatly appreciated.
Driver tested on RB5 and RPI (with and without libcamera) for V1. V2 mainly
tested on RPI.
---
v1 -> v2:
- driver: Drop VD56G3_NUM_SUPPLIES
- driver: Rename 'ext_clock' to 'xclk_freq'
- driver: Make use of 'container_of_const' instead of 'container_of'
- driver: Drop usage of WARN()
- driver: Move a few variables to unsigned int
- driver: Add defines for the different Cut revisions
- driver: Replace dev_warn() by dev_err() in situation we're returning errors
- driver: Ensure sensor has dedicated 3.5ms to boot after reset
- driver: Take into account return value of __v4l2_ctrl_modify_range() and
__v4l2_ctrl_s_ctrl() functions
- driver: Merge vd56g3_power_on() and vd56g3_boot()
- dt-bindings: Lowercase power supply names
- dt-bindings: Drop clock-lanes property
- dt-bindings: Drop unecessary 'items' contraint for lane-polarities
- dt-bindings: Drop unused labels
Sylvain Petinot (2):
media: dt-bindings: Add ST VD56G3 camera sensor binding
media: i2c: Add driver for ST VD56G3 camera sensor
.../bindings/media/i2c/st,st-vd56g3.yaml | 132 ++
MAINTAINERS | 9 +
drivers/media/i2c/Kconfig | 11 +
drivers/media/i2c/Makefile | 1 +
drivers/media/i2c/st-vd56g3.c | 1608 +++++++++++++++++
5 files changed, 1761 insertions(+)
create mode 100644 Documentation/devicetree/bindings/media/i2c/st,st-vd56g3.yaml
create mode 100644 drivers/media/i2c/st-vd56g3.c
--
2.17.1
next reply other threads:[~2024-05-21 16:31 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-05-21 16:29 Sylvain Petinot [this message]
2024-05-21 16:29 ` [PATCH v2 1/2] media: dt-bindings: Add ST VD56G3 camera sensor binding Sylvain Petinot
2024-05-21 17:37 ` Krzysztof Kozlowski
2024-05-27 13:14 ` Sylvain Petinot
2024-05-27 19:00 ` Krzysztof Kozlowski
2024-06-03 8:46 ` Sylvain Petinot
2024-05-27 20:08 ` Sakari Ailus
2024-05-27 19:04 ` Krzysztof Kozlowski
2024-05-28 9:22 ` Sakari Ailus
2024-05-28 9:46 ` Krzysztof Kozlowski
2024-05-28 10:01 ` Sakari Ailus
2024-06-03 8:47 ` Sylvain Petinot
2024-05-21 16:29 ` [PATCH v2 2/2] media: i2c: Add driver for ST VD56G3 camera sensor Sylvain Petinot
2024-05-28 11:49 ` Sakari Ailus
2024-06-03 9:59 ` Sylvain Petinot
2024-09-09 7:06 ` Sakari Ailus
2024-09-09 8:42 ` Sylvain Petinot
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240521162950.6987-1-sylvain.petinot@foss.st.com \
--to=sylvain.petinot@foss.st.com \
--cc=benjamin.mugnier@foss.st.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=robh@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).