[Intel-gfx] [PATCH] Removing redundant is_edp_psr

[Jindal, Sonika]

Oh great then :)

Thanks,
Sonika

-----Original Message-----
From: R, Durgadoss 
Sent: Thursday, May 7, 2015 11:19 AM
To: Jindal, Sonika; intel-gfx at lists.freedesktop.org
Subject: RE: [PATCH] Removing redundant is_edp_psr

>-----Original Message-----
>From: Jindal, Sonika
>Sent: Thursday, May 7, 2015 9:58 AM
>To: intel-gfx at lists.freedesktop.org
>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 at intel.com>
>Signed-off-by: Sonika Jindal <sonika.jindal at intel.com>
>---
> 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