drm/i915/display: rename intel_display_driver_suspend/resume functions
authorJani Nikula <jani.nikula@intel.com>
Fri, 14 Apr 2023 09:42:01 +0000 (12:42 +0300)
committerJani Nikula <jani.nikula@intel.com>
Mon, 17 Apr 2023 08:18:00 +0000 (11:18 +0300)
Follow the usual naming conventions. Switch to i915 arguments and naming
while at it.

Reviewed-by: Gustavo Sousa <gustavo.sousa@intel.com>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/cdd4228337678609967ed176dcfc9690de5d490b.1681465222.git.jani.nikula@intel.com
drivers/gpu/drm/i915/display/intel_display_driver.c
drivers/gpu/drm/i915/display/intel_display_driver.h
drivers/gpu/drm/i915/display/intel_display_reset.c
drivers/gpu/drm/i915/i915_driver.c

index eb03b0a..d20a279 100644 (file)
@@ -471,29 +471,28 @@ void intel_display_driver_unregister(struct drm_i915_private *i915)
  * turn all crtc's off, but do not adjust state
  * This has to be paired with a call to intel_modeset_setup_hw_state.
  */
-int intel_display_suspend(struct drm_device *dev)
+int intel_display_driver_suspend(struct drm_i915_private *i915)
 {
-       struct drm_i915_private *dev_priv = to_i915(dev);
        struct drm_atomic_state *state;
        int ret;
 
-       if (!HAS_DISPLAY(dev_priv))
+       if (!HAS_DISPLAY(i915))
                return 0;
 
-       state = drm_atomic_helper_suspend(dev);
+       state = drm_atomic_helper_suspend(&i915->drm);
        ret = PTR_ERR_OR_ZERO(state);
        if (ret)
-               drm_err(&dev_priv->drm, "Suspending crtc's failed with %i\n",
+               drm_err(&i915->drm, "Suspending crtc's failed with %i\n",
                        ret);
        else
-               dev_priv->display.restore.modeset_state = state;
+               i915->display.restore.modeset_state = state;
        return ret;
 }
 
 int
-__intel_display_resume(struct drm_i915_private *i915,
-                      struct drm_atomic_state *state,
-                      struct drm_modeset_acquire_ctx *ctx)
+__intel_display_driver_resume(struct drm_i915_private *i915,
+                             struct drm_atomic_state *state,
+                             struct drm_modeset_acquire_ctx *ctx)
 {
        struct drm_crtc_state *crtc_state;
        struct drm_crtc *crtc;
@@ -530,9 +529,8 @@ __intel_display_resume(struct drm_i915_private *i915,
        return ret;
 }
 
-void intel_display_resume(struct drm_device *dev)
+void intel_display_driver_resume(struct drm_i915_private *i915)
 {
-       struct drm_i915_private *i915 = to_i915(dev);
        struct drm_atomic_state *state = i915->display.restore.modeset_state;
        struct drm_modeset_acquire_ctx ctx;
        int ret;
@@ -547,7 +545,7 @@ void intel_display_resume(struct drm_device *dev)
        drm_modeset_acquire_init(&ctx, 0);
 
        while (1) {
-               ret = drm_modeset_lock_all_ctx(dev, &ctx);
+               ret = drm_modeset_lock_all_ctx(&i915->drm, &ctx);
                if (ret != -EDEADLK)
                        break;
 
@@ -555,7 +553,7 @@ void intel_display_resume(struct drm_device *dev)
        }
 
        if (!ret)
-               ret = __intel_display_resume(i915, state, &ctx);
+               ret = __intel_display_driver_resume(i915, state, &ctx);
 
        skl_watermark_ipc_update(i915);
        drm_modeset_drop_locks(&ctx);
index 7b5ff43..84e7977 100644 (file)
@@ -9,7 +9,6 @@
 #include <linux/types.h>
 
 struct drm_atomic_state;
-struct drm_device;
 struct drm_i915_private;
 struct drm_modeset_acquire_ctx;
 struct pci_dev;
@@ -24,13 +23,13 @@ void intel_display_driver_remove(struct drm_i915_private *i915);
 void intel_display_driver_remove_noirq(struct drm_i915_private *i915);
 void intel_display_driver_remove_nogem(struct drm_i915_private *i915);
 void intel_display_driver_unregister(struct drm_i915_private *i915);
-int intel_display_suspend(struct drm_device *dev);
-void intel_display_resume(struct drm_device *dev);
+int intel_display_driver_suspend(struct drm_i915_private *i915);
+void intel_display_driver_resume(struct drm_i915_private *i915);
 
 /* interface for intel_display_reset.c */
-int __intel_display_resume(struct drm_i915_private *i915,
-                          struct drm_atomic_state *state,
-                          struct drm_modeset_acquire_ctx *ctx);
+int __intel_display_driver_resume(struct drm_i915_private *i915,
+                                 struct drm_atomic_state *state,
+                                 struct drm_modeset_acquire_ctx *ctx);
 
 #endif /* __INTEL_DISPLAY_DRIVER_H__ */
 
index 166aa0c..17178d5 100644 (file)
@@ -117,7 +117,7 @@ void intel_display_reset_finish(struct drm_i915_private *i915)
                intel_clock_gating_init(i915);
                intel_hpd_init(i915);
 
-               ret = __intel_display_resume(i915, state, ctx);
+               ret = __intel_display_driver_resume(i915, state, ctx);
                if (ret)
                        drm_err(&i915->drm,
                                "Restoring old state failed with %i\n", ret);
index d38ce38..d36e4b0 100644 (file)
@@ -1043,7 +1043,7 @@ static int i915_drm_prepare(struct drm_device *dev)
        intel_pxp_suspend_prepare(i915->pxp);
 
        /*
-        * NB intel_display_suspend() may issue new requests after we've
+        * NB intel_display_driver_suspend() may issue new requests after we've
         * ostensibly marked the GPU as ready-to-sleep here. We need to
         * split out that work and pull it forward so that after point,
         * the GPU is not woken again.
@@ -1067,7 +1067,7 @@ static int i915_drm_suspend(struct drm_device *dev)
 
        pci_save_state(pdev);
 
-       intel_display_suspend(dev);
+       intel_display_driver_suspend(dev_priv);
 
        intel_dp_mst_suspend(dev_priv);
 
@@ -1241,7 +1241,7 @@ static int i915_drm_resume(struct drm_device *dev)
 
        /* MST sideband requires HPD interrupts enabled */
        intel_dp_mst_resume(dev_priv);
-       intel_display_resume(dev);
+       intel_display_driver_resume(dev_priv);
 
        intel_hpd_poll_disable(dev_priv);
        if (HAS_DISPLAY(dev_priv))