diff options
| author | Hans Verkuil <hverkuil@xs4all.nl> | 2024-10-14 17:06:29 +0200 |
|---|---|---|
| committer | Hans Verkuil <hverkuil@xs4all.nl> | 2024-10-28 09:19:49 +0100 |
| commit | d01e5a4d866d70de11e957c11a4f3b54b996137c (patch) | |
| tree | 27f890584d47c1f2c3e11449c7c35ecce12ceaaa /drivers/media/test-drivers/vimc/vimc-capture.c | |
| parent | 1d8aaa86a32a76021f6c37fff9f7c3a26f48dae7 (diff) | |
media: test-drivers: drop vb2_ops_wait_prepare/finish
Since commit 88785982a19d ("media: vb2: use lock if wait_prepare/finish
are NULL") it is no longer needed to set the wait_prepare/finish
vb2_ops callbacks as long as the lock field in vb2_queue is set.
Since the vb2_ops_wait_prepare/finish callbacks already rely on that field,
we can safely drop these callbacks.
This simplifies the code and this is a step towards the goal of deleting
these callbacks.
Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
Reviewed-by: Shuah Khan <skhan@linuxfoundation.org>
Diffstat (limited to 'drivers/media/test-drivers/vimc/vimc-capture.c')
| -rw-r--r-- | drivers/media/test-drivers/vimc/vimc-capture.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/drivers/media/test-drivers/vimc/vimc-capture.c b/drivers/media/test-drivers/vimc/vimc-capture.c index 89506ae00901..10df039278e7 100644 --- a/drivers/media/test-drivers/vimc/vimc-capture.c +++ b/drivers/media/test-drivers/vimc/vimc-capture.c @@ -326,12 +326,6 @@ static const struct vb2_ops vimc_capture_qops = { .buf_queue = vimc_capture_buf_queue, .queue_setup = vimc_capture_queue_setup, .buf_prepare = vimc_capture_buffer_prepare, - /* - * Since q->lock is set we can use the standard - * vb2_ops_wait_prepare/finish helper functions. - */ - .wait_prepare = vb2_ops_wait_prepare, - .wait_finish = vb2_ops_wait_finish, }; static const struct media_entity_operations vimc_capture_mops = { |