Commit 7edd18b6 authored by Philipp Zabel's avatar Philipp Zabel Committed by Mauro Carvalho Chehab

media: coda: add lockdep asserts

coda_command_sync, coda_hw_reset, and __coda_start_decoding
all expect to be called under the coda_mutex device lock.
Signed-off-by: default avatarPhilipp Zabel <p.zabel@pengutronix.de>
Signed-off-by: default avatarHans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+samsung@kernel.org>
parent 8a8621ba
...@@ -102,6 +102,8 @@ static int coda_command_sync(struct coda_ctx *ctx, int cmd) ...@@ -102,6 +102,8 @@ static int coda_command_sync(struct coda_ctx *ctx, int cmd)
struct coda_dev *dev = ctx->dev; struct coda_dev *dev = ctx->dev;
int ret; int ret;
lockdep_assert_held(&dev->coda_mutex);
coda_command_async(ctx, cmd); coda_command_async(ctx, cmd);
ret = coda_wait_timeout(dev); ret = coda_wait_timeout(dev);
trace_coda_bit_done(ctx); trace_coda_bit_done(ctx);
...@@ -116,6 +118,8 @@ int coda_hw_reset(struct coda_ctx *ctx) ...@@ -116,6 +118,8 @@ int coda_hw_reset(struct coda_ctx *ctx)
unsigned int idx; unsigned int idx;
int ret; int ret;
lockdep_assert_held(&dev->coda_mutex);
if (!dev->rstc) if (!dev->rstc)
return -ENOENT; return -ENOENT;
...@@ -1676,6 +1680,8 @@ static int __coda_start_decoding(struct coda_ctx *ctx) ...@@ -1676,6 +1680,8 @@ static int __coda_start_decoding(struct coda_ctx *ctx)
u32 val; u32 val;
int ret; int ret;
lockdep_assert_held(&dev->coda_mutex);
coda_dbg(1, ctx, "Video Data Order Adapter: %s\n", coda_dbg(1, ctx, "Video Data Order Adapter: %s\n",
ctx->use_vdoa ? "Enabled" : "Disabled"); ctx->use_vdoa ? "Enabled" : "Disabled");
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment