drm/i915/psr: Add activate vfunc.
authorRodrigo Vivi <rodrigo.vivi@intel.com>
Thu, 7 Sep 2017 23:00:34 +0000 (16:00 -0700)
committerRodrigo Vivi <rodrigo.vivi@intel.com>
Mon, 11 Sep 2017 22:25:18 +0000 (15:25 -0700)
Continue on VLV PSR split with vfunc, let's move activate
function there.

Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
Cc: Vathsala Nagaraju <vathsala.nagaraju@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Reviewed-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20170907230041.22978-5-rodrigo.vivi@intel.com
drivers/gpu/drm/i915/i915_drv.h
drivers/gpu/drm/i915/intel_psr.c

index d24793c..fa279f3 100644 (file)
@@ -1183,6 +1183,7 @@ struct i915_psr {
 
        void (*disable_source)(struct intel_dp *,
                               const struct intel_crtc_state *);
+       void (*activate)(struct intel_dp *);
 };
 
 enum intel_pch {
index 92b3db3..b3c1e60 100644 (file)
@@ -472,12 +472,7 @@ static void intel_psr_activate(struct intel_dp *intel_dp)
        WARN_ON(dev_priv->psr.active);
        lockdep_assert_held(&dev_priv->psr.lock);
 
-       /* Enable/Re-enable PSR on the host */
-       if (HAS_DDI(dev_priv))
-               hsw_psr_activate(intel_dp);
-       else
-               vlv_psr_activate(intel_dp);
-
+       dev_priv->psr.activate(intel_dp);
        dev_priv->psr.active = true;
 }
 
@@ -987,7 +982,9 @@ void intel_psr_init(struct drm_i915_private *dev_priv)
 
        if (IS_VALLEYVIEW(dev_priv) || IS_CHERRYVIEW(dev_priv)) {
                dev_priv->psr.disable_source = vlv_psr_disable;
+               dev_priv->psr.activate = vlv_psr_activate;
        } else {
                dev_priv->psr.disable_source = hsw_psr_disable;
+               dev_priv->psr.activate = hsw_psr_activate;
        }
 }