mbox series

[v7,0/2] media: i2c: Add support for GC08A3 sensor

Message ID 20240303022609.26263-1-zhi.mao@mediatek.com (mailing list archive)
Headers show
Series media: i2c: Add support for GC08A3 sensor | expand

Message

Zhi Mao (毛智) March 3, 2024, 2:26 a.m. UTC
This series adds YAML DT binding and V4L2 sub-device driver for Galaxycore's
GC08A3 8-megapixel 10-bit RAW CMOS 1/4" sensor, with an MIPI CSI-2 image data
interface and the I2C control bus.

The driver is implemented with V4L2 framework.
 - Async registered as a V4L2 sub-device.
 - As the first component of camera system including Seninf, ISP pipeline.
 - A media entity that provides one source pad in common.
 - Used in camera features on ChromeOS application.

Also this driver supports following features:
 - manual exposure and analog gain control support
 - vertical blanking control support
 - test pattern support
 - media controller support
 - runtime PM support
 - support resolution: 3264x2448@30fps, 1920x1080@60fps

Previous versions of this patch-set can be found here:
v6: https://lore.kernel.org/linux-media/20240227013221.21512-1-zhi.mao@mediatek.com/
v5: https://lore.kernel.org/linux-media/20240220012540.10607-1-zhi.mao@mediatek.com/
v4: https://lore.kernel.org/linux-media/20240204061538.2105-1-zhi.mao@mediatek.com/
v3: https://lore.kernel.org/linux-media/20240109022715.30278-1-zhi.mao@mediatek.com/
v2: https://lore.kernel.org/linux-media/20231207052016.25954-1-zhi.mao@mediatek.com/
v1: https://lore.kernel.org/linux-media/20231123115104.32094-1-zhi.mao@mediatek.com/

This series is based on linux-next, tag: next-20240229
Changes in v7:
- gc08a3 sensor driver:
-- refine header files in driver code follow iwyu rules
-- modify some commets to match code style and english grammar and punctuation
-- use the default case for assign variable in function: gc08a3_test_pattern
-- use DEFINE_* PM macro for runtime pm ops

Thanks

Zhi Mao (2):
  media: dt-bindings: i2c: add GalaxyCore GC08A3 image sensor
  media: i2c: Add GC08A3 image sensor driver

 .../bindings/media/i2c/galaxycore,gc08a3.yaml |  112 ++
 drivers/media/i2c/Kconfig                     |   10 +
 drivers/media/i2c/Makefile                    |    1 +
 drivers/media/i2c/gc08a3.c                    | 1339 +++++++++++++++++
 4 files changed, 1462 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/media/i2c/galaxycore,gc08a3.yaml
 create mode 100644 drivers/media/i2c/gc08a3.c

Comments

Zhi Mao (毛智) March 12, 2024, 1:31 a.m. UTC | #1
ping...

On Sun, 2024-03-03 at 10:26 +0800, Zhi Mao wrote:
> This series adds YAML DT binding and V4L2 sub-device driver for
> Galaxycore's
> GC08A3 8-megapixel 10-bit RAW CMOS 1/4" sensor, with an MIPI CSI-2
> image data
> interface and the I2C control bus.
> 
> The driver is implemented with V4L2 framework.
>  - Async registered as a V4L2 sub-device.
>  - As the first component of camera system including Seninf, ISP
> pipeline.
>  - A media entity that provides one source pad in common.
>  - Used in camera features on ChromeOS application.
> 
> Also this driver supports following features:
>  - manual exposure and analog gain control support
>  - vertical blanking control support
>  - test pattern support
>  - media controller support
>  - runtime PM support
>  - support resolution: 3264x2448@30fps, 1920x1080@60fps
> 
> Previous versions of this patch-set can be found here:
> v6: 
> https://lore.kernel.org/linux-media/20240227013221.21512-1-zhi.mao@mediatek.com/
> v5: 
> https://lore.kernel.org/linux-media/20240220012540.10607-1-zhi.mao@mediatek.com/
> v4: 
> https://lore.kernel.org/linux-media/20240204061538.2105-1-zhi.mao@mediatek.com/
> v3: 
> https://lore.kernel.org/linux-media/20240109022715.30278-1-zhi.mao@mediatek.com/
> v2: 
> https://lore.kernel.org/linux-media/20231207052016.25954-1-zhi.mao@mediatek.com/
> v1: 
> https://lore.kernel.org/linux-media/20231123115104.32094-1-zhi.mao@mediatek.com/
> 
> This series is based on linux-next, tag: next-20240229
> Changes in v7:
> - gc08a3 sensor driver:
> -- refine header files in driver code follow iwyu rules
> -- modify some commets to match code style and english grammar and
> punctuation
> -- use the default case for assign variable in function:
> gc08a3_test_pattern
> -- use DEFINE_* PM macro for runtime pm ops
> 
> Thanks
> 
> Zhi Mao (2):
>   media: dt-bindings: i2c: add GalaxyCore GC08A3 image sensor
>   media: i2c: Add GC08A3 image sensor driver
> 
>  .../bindings/media/i2c/galaxycore,gc08a3.yaml |  112 ++
>  drivers/media/i2c/Kconfig                     |   10 +
>  drivers/media/i2c/Makefile                    |    1 +
>  drivers/media/i2c/gc08a3.c                    | 1339
> +++++++++++++++++
>  4 files changed, 1462 insertions(+)
>  create mode 100644
> Documentation/devicetree/bindings/media/i2c/galaxycore,gc08a3.yaml
>  create mode 100644 drivers/media/i2c/gc08a3.c
>
Sakari Ailus March 12, 2024, 10:44 a.m. UTC | #2
Hi Zhi,

On Tue, Mar 12, 2024 at 01:31:20AM +0000, Zhi Mao (毛智) wrote:
> ping...

The set (v6) has been in my tree for some time now:
<URL:https://git.linuxtv.org/sailus/media_tree.git/log/>.

If you have further changes, please post them on top. Thanks.