From: Andrzej Hajda <a.hajda@samsung.com>
Date: Wed, 5 Apr 2017 09:28:33 +0200
Subject: drm/exynos/decon5433: kill BIT_IRQS_ENABLED flag
Git-commit: 358eccc0eec8d76db7221f6d51d7d528c656ef46
Patch-mainline: v4.13-rc1
References: FATE#326289 FATE#326079 FATE#326049 FATE#322398 FATE#326166
Since DECON uses enable_irq/disable_irq to full control IRQs,
there is no point in having flags to trace it separately.
As a bonus condition for software trigger becomes always true,
so it can be removed.
Signed-off-by: Andrzej Hajda <a.hajda@samsung.com>
Signed-off-by: Inki Dae <inki.dae@samsung.com>
Acked-by: Petr Tesarik <ptesarik@suse.com>
---
drivers/gpu/drm/exynos/exynos5433_drm_decon.c | 8 +-------
1 file changed, 1 insertion(+), 7 deletions(-)
--- a/drivers/gpu/drm/exynos/exynos5433_drm_decon.c
+++ b/drivers/gpu/drm/exynos/exynos5433_drm_decon.c
@@ -49,7 +49,6 @@ static const char * const decon_clks_nam
enum decon_flag_bits {
BIT_CLKS_ENABLED,
- BIT_IRQS_ENABLED,
BIT_WIN_UPDATED,
BIT_SUSPENDED
};
@@ -112,8 +111,6 @@ static int decon_enable_vblank(struct ex
if (!(ctx->out_type & I80_HW_TRG))
enable_irq(ctx->te_irq);
- set_bit(BIT_IRQS_ENABLED, &ctx->flags);
-
return 0;
}
@@ -121,7 +118,6 @@ static void decon_disable_vblank(struct
{
struct decon_context *ctx = crtc->ctx;
- clear_bit(BIT_IRQS_ENABLED, &ctx->flags);
if (test_bit(BIT_SUSPENDED, &ctx->flags))
return;
@@ -536,9 +532,7 @@ static irqreturn_t decon_te_irq_handler(
(ctx->out_type & I80_HW_TRG))
return IRQ_HANDLED;
- if (test_and_clear_bit(BIT_WIN_UPDATED, &ctx->flags) ||
- test_bit(BIT_IRQS_ENABLED, &ctx->flags))
- decon_set_bits(ctx, DECON_TRIGCON, TRIGCON_SWTRIGCMD, ~0);
+ decon_set_bits(ctx, DECON_TRIGCON, TRIGCON_SWTRIGCMD, ~0);
return IRQ_HANDLED;
}