Sent with Proton Mail secure email. On Monday, September 2nd, 2024 at 22:04, Hans Verkuil <hverkuil-cisco@xxxxxxxxx> wrote: > The vb2_queue_init() will expect the vb2_queue lock pointer to be set in > the future. So for those drivers that set the lock later, move it up to > before the vb2_queue_init() call. > > Signed-off-by: Hans Verkuil hverkuil-cisco@xxxxxxxxx > LGTM Acked-by: Matt Ranostay <matt@xxxxxxxxxxx> > Cc: Matt Ranostay matt@xxxxxxxxxxx > > --- > drivers/media/i2c/video-i2c.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/media/i2c/video-i2c.c b/drivers/media/i2c/video-i2c.c > index 56dbe07a1c99..ffb3d9d61a74 100644 > --- a/drivers/media/i2c/video-i2c.c > +++ b/drivers/media/i2c/video-i2c.c > @@ -798,13 +798,13 @@ static int video_i2c_probe(struct i2c_client *client) > queue->min_queued_buffers = 1; > > queue->ops = &video_i2c_video_qops; > > queue->mem_ops = &vb2_vmalloc_memops; > > + queue->lock = &data->queue_lock; > > > ret = vb2_queue_init(queue); > if (ret < 0) > goto error_unregister_device; > > data->vdev.queue = queue; > > - data->vdev.queue->lock = &data->queue_lock; > > > snprintf(data->vdev.name, sizeof(data->vdev.name), > > "I2C %d-%d Transport Video", > -- > 2.34.1
