Re: [PATCH v2 2/2] media: i2c: add os02g10 image sensor driver

[Elgin Perumbilly]

Hi Laurent,
 
> I sent a review comment on v1.
>
> On Tue, Apr 14, 2026 at 02:19:45PM +0530, Elgin Perumbilly wrote:
> > Add a v4l2 subdevice driver for the Omnivision os02g10 sensor.
> >
> > The Omnivision os02g10 is a CMOS image sensor with an active array size of
> > 1920 x 1080.
> >
> > The following features are supported:
> > - Manual exposure an gain control support
> > - vblank/hblank control support
> > - vflip/hflip control support
> > - Test pattern control support
> > - Supported resolution: 1920 x 1080 @ 30fps (SBGGR10)
> >
> > Signed-off-by: Elgin Perumbilly <elgin.perumbilly@xxxxxxxxxxxxxxxxx>
> > Reviewed-by: Tarang Raval <tarang.raval@xxxxxxxxxxxxxxxxx>
> > ---
> >  MAINTAINERS                 |    1 +
> >  drivers/media/i2c/Kconfig   |   10 +
> >  drivers/media/i2c/Makefile  |    1 +
> >  drivers/media/i2c/os02g10.c | 1039 +++++++++++++++++++++++++++++++++++
> >  4 files changed, 1051 insertions(+)
> >  create mode 100644 drivers/media/i2c/os02g10.c
 
I have added a new function, os02g10_set_framefmt, which dynamically sets
the mode register.
 
Please let me know if I have missed anything or if further changes are
needed.
 
Best regards,
Elgin