Re: [PATCH v2] media: v4l2-subdev: Refactor events

From: Sakari Ailus
Date: Tue Oct 29 2024 - 02:42:59 EST


On Mon, Oct 28, 2024 at 08:36:57PM +0200, Laurent Pinchart wrote:
> > > What I meant is looking at the I2C sensor drivers that currently
> > >
> > > - call v4l2_subdev_init_finalize()
> > > - set V4L2_SUBDEV_FL_HAS_EVENTS
> > > - set the .subscribe_event() and .unsubscribe_event() handlers
> > >
> > > and dropping the flag and handlers from them. Is that what you plan to
> > > work on ?
> >
> > It's ok for you per/driver patch or you prefer a big single patch?
>
> I'm fine either way. Maybe one large patch to address all the drivers
> where the flag and handlers are simply dropped, and then one patch per
> driver where changes are larger (such as adding calls to
> v4l2_subdev_init_finalize()) ?

Sounds good to me.

--
Sakari Ailus