From: Sebastian Reichel <sebastian.reichel@xxxxxxxxxxxxx>
Hi,
I did the following tests on an i.MX6ULL based system [0]:
* v4l2-compliance -u /dev/v4l-subdev1
- v4l2-compliance 1.26.0, 32 bits, 32-bit time_t
(from Debian testing)
- Total for device /dev/v4l-subdev1: 45, Succeeded: 45
* Using gstreamer + v4l2-ctl
- Tried 640x480, 320x240, 160x120 YUYV8_2X8 formats
- Tested effect of all exposed user controls
* checkpatch does not report any driver issues
* dt_binding_check does not report anything
* rebind driver works
[0] https://embedded-recipes.org/2023/schedule/running-foss-thermal-camera/
Changes since PATCHv4:
* https://lore.kernel.org/all/20231113230421.226808-1-sre@xxxxxxxxxx/
* Rebase to latest master branch from Sakari
- v4l2_subdev_get_pad_format -> v4l2_subdev_state_get_format
- v4l2_subdev_get_try_format -> v4l2_subdev_state_get_format
- v4l2_subdev_pad_ops.init_cfg -> v4l2_subdev_internal_ops.init_state
* Fix feedback from Tommaso Merciai
- uint8_t -> u8
- drop extra newline
- introduce gc0308_init_controls
* Drop galaxycore vendor binding (merged)
* Drop async fixes (merged to different tree)
Changes since PATCHv3:
* https://lore.kernel.org/all/20231027011417.2174658-1-sre@xxxxxxxxxx/
* Rebased to v6.7-rc1
* Replace gc0308_is_valid_format() with gc0308_get_format_idx(),
which can be reused in gc0308_set_format().
* Remove the runtime PM get call in remove function
* Move the runtime PM get call before the subdev register call in probe
* Drop message about unsupported format code
* Add Reviewed-by: Jacopo Mondi <jacopo.mondi@xxxxxxxxxxxxxxxx>
* Merged binding patch with the one from Alain Volmat and add
Acked-by: Rob Herring <robh@xxxxxxxxxx> and assign authorship
to him, since his patch arrived first on the ML
* Added two patches to fix an issue with v4l2_async_unregister_subdev()
list handling
Changes since PATCHv2:
* https://lore.kernel.org/all/20231024010355.1877523-1-sre@xxxxxxxxxx/
* Simplify Kconfig dependencies
* Do not store code/resolution; which is available from subdev state
* Store register values for mode settings to avoid second lookup in
s_stream
* Reduce power_on sleep times
* remove debug dev_err() print, that I accidently added in v2
* add missing format check in gc0308_enum_frame_size()
* do not PM resume in gc0308_s_ctrl()
* enable and use runtime PM autosuspend
* add .init_cfg() PAD op
* use CCI helper instead of raw regmap
* cluster both flip controls
(that's not just a performance optimization, but fixes an issue,
that register update is slow, so fast sequential setting of VFLIP/HFLIP
override each other without this)
* simplify gc0308_set_power_line_freq
* free control handler on probe error
* use first format by default
* expose V4L2_CID_HBLANK, V4L2_CID_VBLANK, V4L2_CID_PIXEL_RATE
* remove incorrect support for framerate and instead add a comment
Changes since PATCHv1:
* https://lore.kernel.org/all/20231023002547.1754190-1-sre@xxxxxxxxxx/
* Update binding: i2c0 -> i2c
* Update binding: make GC0309 use GC0308 as fallback compatible
* Add regulator handling in power_on/power_off
* Fix alignment of regmap_multi_reg_write arguments
* Fix useless extra return
* Do not check for pad in gc0308_enum_frame_size()
* Drop get_mbus_config implementation
* Use V4L2_CID_AUTO_EXPOSURE_BIAS instead of V4L2_CID_EXPOSURE
* Drop gc0308_get_format in favour of v4l2_subdev_get_fmt
* Replace open-coded v4l2_find_nearest_size() logic
* check clock rate instead of setting it
* use fwnode_graph_get_endpoint_by_id()
* power off device when probe errors out after power on
* replace mutex with sub-device state
* add Galaxycore to generic camera sensors section in MAINTAINERS
* add GC0308 entry in MAINTAINERS
Greetings,
-- Sebastian
Sebastian Reichel (3):
media: dt-bindings: gc0308: add binding
media: MAINTAINERS: Add GalaxyCore in camera sensor section
media: i2c: gc0308: new driver
.../bindings/media/i2c/galaxycore,gc0308.yaml | 108 ++
MAINTAINERS | 8 +
drivers/media/i2c/Kconfig | 10 +
drivers/media/i2c/Makefile | 1 +
drivers/media/i2c/gc0308.c | 1451 +++++++++++++++++
5 files changed, 1578 insertions(+)
create mode 100644 Documentation/devicetree/bindings/media/i2c/galaxycore,gc0308.yaml
create mode 100644 drivers/media/i2c/gc0308.c
--
2.42.0
