2022-05-16 21:40:32 +00:00
|
|
|
From 5fa4e656501eed725b9151a38e5b55a45571bf31 Mon Sep 17 00:00:00 2001
|
|
|
|
|
From: Maxime Ripard <maxime@cerno.tech>
|
|
|
|
|
Date: Mon, 6 Dec 2021 16:17:56 +0100
|
|
|
|
|
Subject: [PATCH] drm/vc4: hvs: Remove dlist setup duplication
|
|
|
|
|
|
|
|
|
|
Setting the DISPLISTx register needs to occur in every case, and we
|
|
|
|
|
don't need to protect the register using the event_lock, so we can just
|
|
|
|
|
move it after the if branches and simplify a bit the function.
|
|
|
|
|
|
|
|
|
|
Signed-off-by: Maxime Ripard <maxime@cerno.tech>
|
|
|
|
|
---
|
|
|
|
|
drivers/gpu/drm/vc4/vc4_hvs.c | 9 +++------
|
|
|
|
|
1 file changed, 3 insertions(+), 6 deletions(-)
|
|
|
|
|
|
|
|
|
|
--- a/drivers/gpu/drm/vc4/vc4_hvs.c
|
|
|
|
|
+++ b/drivers/gpu/drm/vc4/vc4_hvs.c
|
2022-06-09 13:19:03 +00:00
|
|
|
@@ -639,15 +639,12 @@ static void vc4_hvs_update_dlist(struct
|
2022-05-16 21:40:32 +00:00
|
|
|
crtc->state->event = NULL;
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
- HVS_WRITE(SCALER_DISPLISTX(vc4_state->assigned_channel),
|
|
|
|
|
- vc4_state->mm.start);
|
|
|
|
|
-
|
|
|
|
|
spin_unlock_irqrestore(&dev->event_lock, flags);
|
|
|
|
|
- } else {
|
|
|
|
|
- HVS_WRITE(SCALER_DISPLISTX(vc4_state->assigned_channel),
|
|
|
|
|
- vc4_state->mm.start);
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
+ HVS_WRITE(SCALER_DISPLISTX(vc4_state->assigned_channel),
|
|
|
|
|
+ vc4_state->mm.start);
|
|
|
|
|
+
|
|
|
|
|
spin_lock_irqsave(&vc4_crtc->irq_lock, flags);
|
|
|
|
|
vc4_crtc->current_dlist = vc4_state->mm.start;
|
|
|
|
|
spin_unlock_irqrestore(&vc4_crtc->irq_lock, flags);
|
