summaryrefslogtreecommitdiffstats
path: root/drivers/media/test-drivers/vim2m.c
diff options
context:
space:
mode:
authorGuenter Roeck <linux@roeck-us.net>2021-05-14 17:27:34 +0200
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>2021-06-02 13:31:13 +0200
commitd67fa04ce41f7b5d92563734d76c55a676846cc4 (patch)
tree6b100717799a71364588ab83a066d78b660a83db /drivers/media/test-drivers/vim2m.c
parent8f2e452730d2bcd59fe05246f0e19a4c52e0012d (diff)
downloadlinux-stable-d67fa04ce41f7b5d92563734d76c55a676846cc4.tar.gz
linux-stable-d67fa04ce41f7b5d92563734d76c55a676846cc4.tar.bz2
linux-stable-d67fa04ce41f7b5d92563734d76c55a676846cc4.zip
media: media/test_drivers: Drop unnecessary NULL check after container_of
The result of container_of() operations is never NULL unless the embedded element is the first element of the structure. This is not the case here. The NULL check is therefore unnecessary and misleading. Remove it. This change was made automatically with the following Coccinelle script. @@ type t; identifier v; statement s; @@ <+... ( t v = container_of(...); | v = container_of(...); ) ... when != v - if (\( !v \| v == NULL \) ) s ...+> Signed-off-by: Guenter Roeck <linux@roeck-us.net> Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl> Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
Diffstat (limited to 'drivers/media/test-drivers/vim2m.c')
-rw-r--r--drivers/media/test-drivers/vim2m.c5
1 files changed, 0 insertions, 5 deletions
diff --git a/drivers/media/test-drivers/vim2m.c b/drivers/media/test-drivers/vim2m.c
index a24624353f9e..d714fe50afe5 100644
--- a/drivers/media/test-drivers/vim2m.c
+++ b/drivers/media/test-drivers/vim2m.c
@@ -624,11 +624,6 @@ static void device_work(struct work_struct *w)
curr_ctx = container_of(w, struct vim2m_ctx, work_run.work);
- if (!curr_ctx) {
- pr_err("Instance released before the end of transaction\n");
- return;
- }
-
vim2m_dev = curr_ctx->dev;
src_vb = v4l2_m2m_src_buf_remove(curr_ctx->fh.m2m_ctx);