This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: media/test_drivers: Drop unnecessary NULL check after 
container_of
Author:  Guenter Roeck <[email protected]>
Date:    Fri May 14 17:27:34 2021 +0200

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 <[email protected]>
Signed-off-by: Hans Verkuil <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>

 drivers/media/test-drivers/vim2m.c | 5 -----
 1 file changed, 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);

_______________________________________________
linuxtv-commits mailing list
[email protected]
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to