Hi Hans, On 12/8/22 09:12, Hans Verkuil wrote: > Don't touch q->start_streaming_called directly, use the > vb2_start_streaming_called() function instead. > > Signed-off-by: Hans Verkuil <hverkuil-cisco@xxxxxxxxx> Thanks, patch looks good to me: Reviewed-by: Hans de Goede <hdegoede@xxxxxxxxxx> Note I have been doing a lot of work on atomisp cleanups / improvements recently. So I'm sending pull-reqs from: https://git.kernel.org/pub/scm/linux/kernel/git/hansg/linux.git/log/?h=media-atomisp to Mauro for atomisp changes. And I hope to eventually cleanup this function a bit, so I plan to take this upstream through my media-atomisp branch to avoid conflicts. Regards, Hans > --- > drivers/staging/media/atomisp/pci/atomisp_ioctl.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/staging/media/atomisp/pci/atomisp_ioctl.c b/drivers/staging/media/atomisp/pci/atomisp_ioctl.c > index cb01ba65c88f..4f35e8f8250a 100644 > --- a/drivers/staging/media/atomisp/pci/atomisp_ioctl.c > +++ b/drivers/staging/media/atomisp/pci/atomisp_ioctl.c > @@ -636,10 +636,10 @@ static int atomisp_enum_input(struct file *file, void *fh, > static unsigned int > atomisp_subdev_streaming_count(struct atomisp_sub_device *asd) > { > - return asd->video_out_preview.vb_queue.start_streaming_called > - + asd->video_out_capture.vb_queue.start_streaming_called > - + asd->video_out_video_capture.vb_queue.start_streaming_called > - + asd->video_out_vf.vb_queue.start_streaming_called; > + return vb2_start_streaming_called(&asd->video_out_preview.vb_queue) + > + vb2_start_streaming_called(&asd->video_out_capture.vb_queue) + > + vb2_start_streaming_called(&asd->video_out_video_capture.vb_queue) + > + vb2_start_streaming_called(&asd->video_out_vf.vb_queue); > } > > unsigned int atomisp_streaming_count(struct atomisp_device *isp)
