Oh great then :) Thanks, Sonika -----Original Message----- From: R, Durgadoss Sent: Thursday, May 7, 2015 11:19 AM To: Jindal, Sonika; intel-gfx@xxxxxxxxxxxxxxxxxxxxx Subject: RE: [PATCH] Removing redundant is_edp_psr >-----Original Message----- >From: Jindal, Sonika >Sent: Thursday, May 7, 2015 9:58 AM >To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx >Cc: Jindal, Sonika; R, Durgadoss >Subject: [PATCH] Removing redundant is_edp_psr > >Since we already store the sink's psr status in dev_priv, use it. >Without this we were ignoring the case where sink supports psr2. Looks like it is already updated in this patch from Rodrigo: drm/i915: Add psr_ready on pipe_config [https://tango.freedesktop.org/patch/45695/] Thanks, Durga > >Cc: Durgadoss R <durgadoss.r@xxxxxxxxx> >Signed-off-by: Sonika Jindal <sonika.jindal@xxxxxxxxx> >--- > drivers/gpu/drm/i915/intel_psr.c | 7 +------ > 1 file changed, 1 insertion(+), 6 deletions(-) > >diff --git a/drivers/gpu/drm/i915/intel_psr.c >b/drivers/gpu/drm/i915/intel_psr.c >index 27608ce..406d3ac 100644 >--- a/drivers/gpu/drm/i915/intel_psr.c >+++ b/drivers/gpu/drm/i915/intel_psr.c >@@ -56,11 +56,6 @@ > #include "intel_drv.h" > #include "i915_drv.h" > >-static bool is_edp_psr(struct intel_dp *intel_dp) -{ >- return intel_dp->psr_dpcd[0] & DP_PSR_IS_SUPPORTED; >-} >- > static bool vlv_is_psr_active_on_pipe(struct drm_device *dev, int >pipe) { > struct drm_i915_private *dev_priv = dev->dev_private; @@ -365,7 >+360,7 @@ void intel_psr_enable(struct intel_dp *intel_dp) > return; > } > >- if (!is_edp_psr(intel_dp)) { >+ if (!dev_priv->psr.sink_support) { > DRM_DEBUG_KMS("PSR not supported by this panel\n"); > return; > } >-- >1.7.10.4 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx