Re: [PATCH] drm/i915: Also disable PSR on Sink when disabling it on Source.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Nov 24, 2015 at 04:40:08AM +0000, Jindal, Sonika wrote:
> Reviewed-by: Sonika Jindal <sonika.jindal@xxxxxxxxx>
> 
> -----Original Message-----
> From: Vivi, Rodrigo 
> Sent: Tuesday, November 24, 2015 3:50 AM
> To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> Cc: Vivi, Rodrigo; Jindal, Sonika
> Subject: [PATCH] drm/i915: Also disable PSR on Sink when disabling it on Source.
> 
> It is not a bad idea to disable the PSR feature on Sink when we are disabling on the Source.
> 
> v2: Move dpcd write inside mutex protected area as suggested by Sonika.
> 
> Cc: Sonika Jindal <sonika.jindal@xxxxxxxxx>
> Suggested-by: Sonika Jindal <sonika.jindal@xxxxxxxxx>
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>

Queued for -next, thanks for the patch.
-Daniel

> ---
>  drivers/gpu/drm/i915/intel_psr.c | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/drivers/gpu/drm/i915/intel_psr.c b/drivers/gpu/drm/i915/intel_psr.c
> index 2b2f84d..3bbb270 100644
> --- a/drivers/gpu/drm/i915/intel_psr.c
> +++ b/drivers/gpu/drm/i915/intel_psr.c
> @@ -518,11 +518,15 @@ void intel_psr_disable(struct intel_dp *intel_dp)
>  		return;
>  	}
>  
> +	/* Disable PSR on Source */
>  	if (HAS_DDI(dev))
>  		hsw_psr_disable(intel_dp);
>  	else
>  		vlv_psr_disable(intel_dp);
>  
> +	/* Disable PSR on Sink */
> +	drm_dp_dpcd_writeb(&intel_dp->aux, DP_PSR_EN_CFG, 0);
> +
>  	dev_priv->psr.enabled = NULL;
>  	mutex_unlock(&dev_priv->psr.lock);
>  
> --
> 2.4.3
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux