drm/i915/guc: Rename the function that resets the GuC
authorMichel Thierry <michel.thierry@intel.com>
Mon, 30 Oct 2017 18:56:14 +0000 (11:56 -0700)
committerChris Wilson <chris@chris-wilson.co.uk>
Thu, 2 Nov 2017 08:08:05 +0000 (08:08 +0000)
intel_guc_reset sounds more like the microcontroller is the one performing
a reset, while in this case is the opposite. intel_reset_guc not only
makes it clearer, it follows the other intel_reset functions available.

v2: Print error message in English (Tvrtko).

Cc: Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>
Signed-off-by: Michel Thierry <michel.thierry@intel.com>
Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20171030185616.32836-2-michel.thierry@intel.com
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
drivers/gpu/drm/i915/i915_drv.h
drivers/gpu/drm/i915/intel_uc.c
drivers/gpu/drm/i915/intel_uncore.c

index c10fece58e867951581e13906d386a1ced1ff5d4..05425bb7565b6e581ad7156f31b83fd38f7a2ad6 100644 (file)
@@ -3324,7 +3324,7 @@ extern int i915_reset_engine(struct intel_engine_cs *engine,
                             unsigned int flags);
 
 extern bool intel_has_reset_engine(struct drm_i915_private *dev_priv);
-extern int intel_guc_reset(struct drm_i915_private *dev_priv);
+extern int intel_reset_guc(struct drm_i915_private *dev_priv);
 extern void intel_engine_init_hangcheck(struct intel_engine_cs *engine);
 extern void intel_hangcheck_init(struct drm_i915_private *dev_priv);
 extern unsigned long i915_chipset_val(struct drm_i915_private *dev_priv);
index 25bd162f38d25afd6c5b0f0bc9518a5041389069..aec295470e0d9f7e8e2b22351e9a98bc8d6ea06c 100644 (file)
@@ -33,9 +33,9 @@ static int __intel_uc_reset_hw(struct drm_i915_private *dev_priv)
        int ret;
        u32 guc_status;
 
-       ret = intel_guc_reset(dev_priv);
+       ret = intel_reset_guc(dev_priv);
        if (ret) {
-               DRM_ERROR("GuC reset failed, ret = %d\n", ret);
+               DRM_ERROR("Failed to reset GuC, ret = %d\n", ret);
                return ret;
        }
 
index 96ee6b2754bed12500d2747772d5e24dab7cbac0..d629b2e40013d4bc20c7d47894ed8c7ad3ad6df4 100644 (file)
@@ -1803,7 +1803,7 @@ bool intel_has_reset_engine(struct drm_i915_private *dev_priv)
                i915_modparams.reset >= 2);
 }
 
-int intel_guc_reset(struct drm_i915_private *dev_priv)
+int intel_reset_guc(struct drm_i915_private *dev_priv)
 {
        int ret;