Re: [PATCH v3 3/3] media: i2c: os02g10: implement crop handling with set_selection

[Elgin Perumbilly]

Hi Sakari, Laurent
 
> On Thu, Jun 18, 2026 at 04:02:53PM +0300, Laurent Pinchart wrote:
> > On Thu, Jun 18, 2026 at 02:47:23PM +0300, Sakari Ailus wrote:
> > > Hi Elgin,
> > >
> > > On Fri, Apr 24, 2026 at 02:55:47PM +0530, Elgin Perumbilly wrote:
> > > > From: Tarang Raval <tarang.raval@xxxxxxxxxxxxxxxxx>
> > > >
> > > > Add crop support to os02g10 by implementing .set_selection() and
> > > > storing the crop rectangle in subdev state.
> > > >
> > > > Initialize the default crop to the active area, make set_fmt() use the
> > > > current crop, and update the output format when the crop size changes.
> > > > Also program the sensor window from the active crop/format state instead
> > > > of using the fixed supported_modes entry.
> > > >
> > > > This allows userspace to configure the sensor crop window explicitly.
> > >
> > > Please wait for the Common Raw Sensor Model patches to be merged before
> > > adding this -- we don't have an established way to configure cropping
> > > before that. Some drivers might do something but it's all a bit haphazard.
> >
> > Does it mean we should wait for the raw sensor model to merge this
> > driver ?
>
> I only suggested this for the 3rd patch.
 
What is the conclusion?
 
Should I drop the 3rd patch for the time being, or include it in
the next revision?
 
Best Regards,
Elgin