From: Dan Carpenter <dan.carpenter@oracle.com>
To: amasule@codeaurora.org
Cc: linux-media@vger.kernel.org
Subject: [bug report] media: venus: introduce core selection
Date: Wed, 11 Mar 2020 11:15:11 +0300 [thread overview]
Message-ID: <20200311081511.GA7593@mwanda> (raw)
Hello Aniket Masule,
The patch 4ebf969375bc: "media: venus: introduce core selection" from
Dec 2, 2019, leads to the following static checker warning:
drivers/media/platform/qcom/venus/vdec.c:968 vdec_start_streaming()
warn: inconsistent returns 'inst->lock'.
drivers/media/platform/qcom/venus/vdec.c
943 static int vdec_start_streaming(struct vb2_queue *q, unsigned int count)
944 {
945 struct venus_inst *inst = vb2_get_drv_priv(q);
946 int ret;
947
948 mutex_lock(&inst->lock);
949
950 ret = venus_pm_acquire_core(inst);
951 if (ret)
952 return ret;
goto error or just goto unlock?
953
954 if (q->type == V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE)
955 ret = vdec_start_capture(inst);
956 else
957 ret = vdec_start_output(inst);
958
959 if (ret)
960 goto error;
961
962 mutex_unlock(&inst->lock);
963 return 0;
964
965 error:
966 venus_helper_buffers_done(inst, VB2_BUF_STATE_QUEUED);
967 mutex_unlock(&inst->lock);
968 return ret;
969 }
regards,
dan carpenter
next reply other threads:[~2020-03-11 8:15 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-11 8:15 Dan Carpenter [this message]
2020-03-11 8:57 ` [bug report] media: venus: introduce core selection Stanimir Varbanov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200311081511.GA7593@mwanda \
--to=dan.carpenter@oracle.com \
--cc=amasule@codeaurora.org \
--cc=linux-media@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.