[Intel-gfx] [PATCH 18/19] drm/i915/execlists: Tidy enabling execlists

[Chris Wilson]

Move the register settings for enabling execlists into its own function
for clarity.

Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
---
 drivers/gpu/drm/i915/intel_lrc.c | 22 ++++++++++++++--------
 1 file changed, 14 insertions(+), 8 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_lrc.c b/drivers/gpu/drm/i915/intel_lrc.c
index 2e1adbaaec2f..2b2c61ed81db 100644
--- a/drivers/gpu/drm/i915/intel_lrc.c
+++ b/drivers/gpu/drm/i915/intel_lrc.c
@@ -1467,9 +1467,20 @@ static u8 gtiir[] = {
        [VECS] = 3,
 };
 
-static int gen8_init_common_ring(struct intel_engine_cs *engine)
+static void enable_execlists(struct intel_engine_cs *engine)
 {
        struct drm_i915_private *dev_priv = engine->i915;
+
+       I915_WRITE(RING_HWSTAM(engine->mmio_base), 0xffffffff);
+       I915_WRITE(RING_MODE_GEN7(engine),
+                  _MASKED_BIT_ENABLE(GFX_RUN_LIST_ENABLE));
+       I915_WRITE(RING_HWS_PGA(engine->mmio_base),
+                  engine->status_page.ggtt_offset);
+       POSTING_READ(RING_HWS_PGA(engine->mmio_base));
+}
+
+static int gen8_init_common_ring(struct intel_engine_cs *engine)
+{
        struct intel_engine_execlists * const execlists = &engine->execlists;
        int ret;
 
@@ -1480,13 +1491,7 @@ static int gen8_init_common_ring(struct intel_engine_cs 
*engine)
        intel_engine_reset_breadcrumbs(engine);
        intel_engine_init_hangcheck(engine);
 
-       I915_WRITE(RING_HWSTAM(engine->mmio_base), 0xffffffff);
-       I915_WRITE(RING_MODE_GEN7(engine),
-                  _MASKED_BIT_ENABLE(GFX_RUN_LIST_ENABLE));
-       I915_WRITE(RING_HWS_PGA(engine->mmio_base),
-                  engine->status_page.ggtt_offset);
-       POSTING_READ(RING_HWS_PGA(engine->mmio_base));
-
+       enable_execlists(engine);
        DRM_DEBUG_DRIVER("Execlists enabled for %s\n", engine->name);
 
        GEM_BUG_ON(engine->id >= ARRAY_SIZE(gtiir));
@@ -1913,6 +1918,7 @@ void intel_logical_ring_cleanup(struct intel_engine_cs 
*engine)
        intel_engine_cleanup_common(engine);
 
        lrc_destroy_wa_ctx(engine);
+
        engine->i915 = NULL;
        dev_priv->engine[engine->id] = NULL;
        kfree(engine);
-- 
2.15.1

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx