[Intel-gfx] [PATCH] Removing redundant is_edp_psr
R, Durgadoss
durgadoss.r at intel.com
Wed May 6 22:48:59 PDT 2015
>-----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
More information about the Intel-gfx
mailing list