Re: [PATCH v2 0/2] media: i2c: Add support for GC08A3 sensor

Krzysztof Kozlowski
2023-12-07T08:19:46Z

On 07/12/2023 06:20, 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:
> v1: https://lore.kernel.org/linux-media/20231123115104.32094-1-zhi.mao@mediatek.com/
>
> Changes of v2 mainly address comments from Krzysztof/Rob Herring&Conor Dooley.
> Compared to v1:
> - Fix some review comments

What exactly fixed? This cannot be that vague!

Best regards,
Krzysztof