Why not move it inside the mutex_lock/unlock on psr.lock ? Regards, Sonika -----Original Message----- From: Vivi, Rodrigo Sent: Thursday, November 19, 2015 6:10 AM To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx Cc: Vivi, Rodrigo; Jindal, Sonika Subject: [PATCH 3/3] 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. Cc: Sonika Jindal <sonika.jindal@xxxxxxxxx> Suggested-by: Sonika Jindal <sonika.jindal@xxxxxxxxx> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> --- drivers/gpu/drm/i915/intel_psr.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/i915/intel_psr.c b/drivers/gpu/drm/i915/intel_psr.c index 38ea4d0..de31162 100644 --- a/drivers/gpu/drm/i915/intel_psr.c +++ b/drivers/gpu/drm/i915/intel_psr.c @@ -527,6 +527,9 @@ void intel_psr_disable(struct intel_dp *intel_dp) mutex_unlock(&dev_priv->psr.lock); cancel_delayed_work_sync(&dev_priv->psr.work); + + /* Disable PSR on Sink */ + drm_dp_dpcd_writeb(&intel_dp->aux, DP_PSR_EN_CFG, 0); } static void intel_psr_work(struct work_struct *work) -- 2.4.3 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx