media: vicodec: Drop unused job_abort()
authorEzequiel Garcia <ezequiel@collabora.com>
Mon, 10 Sep 2018 15:21:54 +0000 (11:21 -0400)
committerMauro Carvalho Chehab <mchehab+samsung@kernel.org>
Mon, 24 Sep 2018 13:13:48 +0000 (09:13 -0400)
The vicodec does not use the aborting field. In fact, this driver
can't really cancel any work, since it performs all the work
in device_run().

Signed-off-by: Ezequiel Garcia <ezequiel@collabora.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
drivers/media/platform/vicodec/vicodec-core.c

index 152ac44..1eb9132 100644 (file)
@@ -112,8 +112,6 @@ struct vicodec_ctx {
 
        struct v4l2_ctrl_handler hdl;
 
-       /* Abort requested by m2m */
-       int                     aborting;
        struct vb2_v4l2_buffer *last_src_buf;
        struct vb2_v4l2_buffer *last_dst_buf;
 
@@ -378,14 +376,6 @@ restart:
        return 1;
 }
 
-static void job_abort(void *priv)
-{
-       struct vicodec_ctx *ctx = priv;
-
-       /* Will cancel the transaction in the next interrupt handler */
-       ctx->aborting = 1;
-}
-
 /*
  * video ioctls
  */
@@ -1270,7 +1260,6 @@ static const struct video_device vicodec_videodev = {
 
 static const struct v4l2_m2m_ops m2m_ops = {
        .device_run     = device_run,
-       .job_abort      = job_abort,
        .job_ready      = job_ready,
 };