diff options
author | Hans Verkuil <hverkuil@xs4all.nl> | 2025-02-24 17:40:58 +0300 |
---|---|---|
committer | Hans Verkuil <hverkuil@xs4all.nl> | 2025-03-12 18:43:51 +0300 |
commit | d9e7c172a7f247f7ef0b151fa8c8f044b6a2a070 (patch) | |
tree | 4d28d2839e79d0c7f1180f6c4a04a5477c0dcda6 /drivers/phy/qualcomm/phy-qcom-usb-ss.c | |
parent | 2014c95afecee3e76ca4a56956a936e23283f05b (diff) | |
download | linux-d9e7c172a7f247f7ef0b151fa8c8f044b6a2a070.tar.xz |
media: rtl2832_sdr: assign vb2 lock before vb2_queue_init
Commit c780d01cf1a6 ("media: vb2: vb2_core_queue_init(): sanity check lock
and wait_prepare/finish") added a sanity check to ensure that if there are
no wait_prepare/finish callbacks set by the driver, then the vb2_queue lock
must be set, since otherwise the vb2 core cannot do correct locking.
The rtl2832_sdr.c triggered this warning: it turns out that while the
driver does set this lock, it sets it too late. So move it up to before
the vb2_queue_init() call.
Reported-by: Arthur Marsh <arthur.marsh@internode.on.net>
Closes: https://lore.kernel.org/linux-media/20241211042355.8479-1-user@am64/
Fixes: 8fcd2795d22a ("media: rtl2832_sdr: drop vb2_ops_wait_prepare/finish")
Cc: stable@vger.kernel.org
Reviewed-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
Diffstat (limited to 'drivers/phy/qualcomm/phy-qcom-usb-ss.c')
0 files changed, 0 insertions, 0 deletions