[Intel-gfx] [PATCH 24/33] drm/i915/gt: Pull intel_gt_init_hw() into intel_gt_resume()

[Chris Wilson]

Since intel_gt_resume() is always immediately proceeded by init_hw, pull
the call into intel_gt_resume, where we have the rpm and fw already
held.

Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
---
 drivers/gpu/drm/i915/gem/i915_gem_pm.c | 20 +-------------------
 drivers/gpu/drm/i915/gt/intel_gt.c     |  5 -----
 drivers/gpu/drm/i915/gt/intel_gt_pm.c  | 12 +++++++++++-
 3 files changed, 12 insertions(+), 25 deletions(-)

diff --git a/drivers/gpu/drm/i915/gem/i915_gem_pm.c 
b/drivers/gpu/drm/i915/gem/i915_gem_pm.c
index f88ee1317bb4..544b99c4f70f 100644
--- a/drivers/gpu/drm/i915/gem/i915_gem_pm.c
+++ b/drivers/gpu/drm/i915/gem/i915_gem_pm.c
@@ -101,28 +101,10 @@ void i915_gem_resume(struct drm_i915_private *i915)
 {
        GEM_TRACE("\n");
 
-       intel_uncore_forcewake_get(&i915->uncore, FORCEWAKE_ALL);
-
-       if (intel_gt_init_hw(&i915->gt))
-               goto err_wedged;
-
        /*
         * As we didn't flush the kernel context before suspend, we cannot
         * guarantee that the context image is complete. So let's just reset
         * it and start again.
         */
-       if (intel_gt_resume(&i915->gt))
-               goto err_wedged;
-
-out_unlock:
-       intel_uncore_forcewake_put(&i915->uncore, FORCEWAKE_ALL);
-       return;
-
-err_wedged:
-       if (!intel_gt_is_wedged(&i915->gt)) {
-               dev_err(i915->drm.dev,
-                       "Failed to re-initialize GPU, declaring it wedged!\n");
-               intel_gt_set_wedged(&i915->gt);
-       }
-       goto out_unlock;
+       intel_gt_resume(&i915->gt);
 }
diff --git a/drivers/gpu/drm/i915/gt/intel_gt.c 
b/drivers/gpu/drm/i915/gt/intel_gt.c
index e7f4c1beb3b6..fc15acee3984 100644
--- a/drivers/gpu/drm/i915/gt/intel_gt.c
+++ b/drivers/gpu/drm/i915/gt/intel_gt.c
@@ -593,11 +593,6 @@ int intel_gt_init(struct intel_gt *gt)
 
        intel_uc_init(&gt->uc);
 
-       err = intel_gt_init_hw(gt);
-       if (err)
-               goto err_uc_init;
-
-       /* Only when the HW is re-initialised, can we replay the requests */
        err = intel_gt_resume(gt);
        if (err)
                goto err_uc_init;
diff --git a/drivers/gpu/drm/i915/gt/intel_gt_pm.c 
b/drivers/gpu/drm/i915/gt/intel_gt_pm.c
index ecde67a75e32..147b86538317 100644
--- a/drivers/gpu/drm/i915/gt/intel_gt_pm.c
+++ b/drivers/gpu/drm/i915/gt/intel_gt_pm.c
@@ -186,7 +186,7 @@ int intel_gt_resume(struct intel_gt *gt)
 {
        struct intel_engine_cs *engine;
        enum intel_engine_id id;
-       int err = 0;
+       int err;
 
        GEM_TRACE("\n");
 
@@ -201,6 +201,15 @@ int intel_gt_resume(struct intel_gt *gt)
        intel_uncore_forcewake_get(gt->uncore, FORCEWAKE_ALL);
        intel_rc6_sanitize(&gt->rc6);
 
+       /* Only when the HW is re-initialised, can we replay the requests */
+       err = intel_gt_init_hw(gt);
+       if (err) {
+               dev_err(gt->i915->drm.dev,
+                       "Failed to initialize GPU, declaring it wedged!\n");
+               intel_gt_set_wedged(gt);
+               goto err_fw;
+       }
+
        intel_rps_enable(&gt->rps);
        intel_llc_enable(&gt->llc);
 
@@ -233,6 +242,7 @@ int intel_gt_resume(struct intel_gt *gt)
 
        user_forcewake(gt, false);
 
+err_fw:
        intel_uncore_forcewake_put(gt->uncore, FORCEWAKE_ALL);
        intel_gt_pm_put(gt);
 
-- 
2.24.0

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