On Wed, Apr 03, 2024 at 02:26:32PM +0200, Benjamin Mugnier wrote: > > @@ -1188,20 +1188,35 @@ static int vgxy61_stream_disable(struct vgxy61_dev *sensor) > > return ret; > > } > > > > -static int vgxy61_s_stream(struct v4l2_subdev *sd, int enable) > > +static int vgxy61_enable_streams(struct v4l2_subdev *sd, > > + struct v4l2_subdev_state *state, u32 pad, > > + u64 streams_mask) > > { > > Should we also check that 'pad == 0' ? Or is it always so ? If the number of pads in the sub-device is 1, then the pad is always 0 here: this is checked in the framework. -- Sakari Ailus