diff options
author | Stanimir Varbanov <stanimir.varbanov@linaro.org> | 2020-09-28 18:44:31 +0200 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab+huawei@kernel.org> | 2021-01-12 18:32:36 +0100 |
commit | 21560ddf782688ca7f6acbdbf4efc68fa55f353b (patch) | |
tree | 0cb7dcd906db3f8a5f6657007e9c9855e1bab4df /drivers/media/platform/qcom/venus/vdec.c | |
parent | a4ca67af8b831a781ac53060c5d5c3dccaf7676e (diff) | |
download | linux-21560ddf782688ca7f6acbdbf4efc68fa55f353b.tar.gz linux-21560ddf782688ca7f6acbdbf4efc68fa55f353b.tar.bz2 linux-21560ddf782688ca7f6acbdbf4efc68fa55f353b.zip |
media: venus: helpers: Lock outside of buffer queue helper
After adding more logic in vdec buf_queue vb2 op it is not
practical to have two lock/unlock for one decoder buf_queue.
So move the instance lock in encoder and decoder vb2 buf_queue
operations.
Signed-off-by: Stanimir Varbanov <stanimir.varbanov@linaro.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
Diffstat (limited to 'drivers/media/platform/qcom/venus/vdec.c')
-rw-r--r-- | drivers/media/platform/qcom/venus/vdec.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/media/platform/qcom/venus/vdec.c b/drivers/media/platform/qcom/venus/vdec.c index a3e64d1af913..708af6adafc1 100644 --- a/drivers/media/platform/qcom/venus/vdec.c +++ b/drivers/media/platform/qcom/venus/vdec.c @@ -1242,9 +1242,8 @@ static void vdec_vb2_buf_queue(struct vb2_buffer *vb) return; } - mutex_unlock(&inst->lock); - venus_helper_vb2_buf_queue(vb); + mutex_unlock(&inst->lock); } static const struct vb2_ops vdec_vb2_ops = { |