From: Steve Longerbeam <slongerb...@gmail.com>

The need_abort flag is not really needed anymore in
__ipu_image_convert_abort(), remove it.
No functional changes.

Signed-off-by: Steve Longerbeam <slongerb...@gmail.com>
---
New since v3.
---
 drivers/gpu/ipu-v3/ipu-image-convert.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/drivers/gpu/ipu-v3/ipu-image-convert.c 
b/drivers/gpu/ipu-v3/ipu-image-convert.c
index abd8afb22b48..b8a400182a00 100644
--- a/drivers/gpu/ipu-v3/ipu-image-convert.c
+++ b/drivers/gpu/ipu-v3/ipu-image-convert.c
@@ -1531,7 +1531,6 @@ static void __ipu_image_convert_abort(struct 
ipu_image_convert_ctx *ctx)
        struct ipu_image_convert_run *run, *active_run, *tmp;
        unsigned long flags;
        int run_count, ret;
-       bool need_abort;
 
        spin_lock_irqsave(&chan->irqlock, flags);
 
@@ -1550,13 +1549,11 @@ static void __ipu_image_convert_abort(struct 
ipu_image_convert_ctx *ctx)
        if (active_run)
                reinit_completion(&ctx->aborted);
 
-       need_abort = (run_count || active_run);
-
        ctx->aborting = true;
 
        spin_unlock_irqrestore(&chan->irqlock, flags);
 
-       if (!need_abort) {
+       if (!run_count && !active_run) {
                dev_dbg(priv->ipu->dev,
                        "%s: task %u: no abort needed for ctx %p\n",
                        __func__, chan->ic_task, ctx);
-- 
2.19.0

Reply via email to