From: Ezequiel Garcia Date: Mon, 10 Sep 2018 15:21:54 +0000 (-0400) Subject: media: vicodec: Drop unused job_abort() X-Git-Tag: v4.20-rc1~69^2~124 X-Git-Url: https://git.kernel.dk/?a=commitdiff_plain;h=e5c413ed9fef975bd90a9dd2dffddbfcc82167af;p=linux-2.6-block.git media: vicodec: Drop unused job_abort() 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 Signed-off-by: Hans Verkuil Signed-off-by: Mauro Carvalho Chehab --- diff --git a/drivers/media/platform/vicodec/vicodec-core.c b/drivers/media/platform/vicodec/vicodec-core.c index 152ac44b0277..1eb9132bfc85 100644 --- a/drivers/media/platform/vicodec/vicodec-core.c +++ b/drivers/media/platform/vicodec/vicodec-core.c @@ -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, };