drm/i915/guc: Update uncore access path in flush_ggtt_writes
diff mbox series

Message ID 20191207010033.24667-1-John.C.Harrison@Intel.com
State New
Headers show
Series
  • drm/i915/guc: Update uncore access path in flush_ggtt_writes
Related show

Commit Message

John Harrison Dec. 7, 2019, 1 a.m. UTC
From: Matthew Brost <matthew.brost@intel.com>

The preferred way to access the uncore is through the GT structure.
Update the GuC function, flush_ggtt_writes, to use this path.

Signed-off-by: Matthew Brost <matthew.brost@intel.com>
Cc: John Harrison <john.c.harrison@intel.com>
---
 drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

Comments

Chris Wilson Dec. 7, 2019, 4:55 p.m. UTC | #1
Quoting John.C.Harrison@Intel.com (2019-12-07 01:00:33)
> From: Matthew Brost <matthew.brost@intel.com>
> 
> The preferred way to access the uncore is through the GT structure.
> Update the GuC function, flush_ggtt_writes, to use this path.
> 
> Signed-off-by: Matthew Brost <matthew.brost@intel.com>
> Cc: John Harrison <john.c.harrison@intel.com>

John, you should provide an s-o-b here, being a conduit.

Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
-Chris

Patch
diff mbox series

diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
index 6f94af7ad1de..4df296a5e7c7 100644
--- a/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
+++ b/drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c
@@ -488,10 +488,9 @@  static void guc_add_request(struct intel_guc *guc, struct i915_request *rq)
  */
 static void flush_ggtt_writes(struct i915_vma *vma)
 {
-	struct drm_i915_private *i915 = vma->vm->i915;
-
 	if (i915_vma_is_map_and_fenceable(vma))
-		intel_uncore_posting_read_fw(&i915->uncore, GUC_STATUS);
+		intel_uncore_posting_read_fw(vma->vm->gt->uncore,
+					     GUC_STATUS);
 }
 
 static void guc_submit(struct intel_engine_cs *engine,