Re: [Intel-gfx] [PATCH] drm/i915/dp: Don't invalidate intel_dp_detect result

2017-08-23 Thread Manasi Navare
On Wed, Aug 23, 2017 at 03:40:28PM -0700, Puthikorn Voravootivat wrote: > If the full detect is already done, we shouldn't need to do it > again. > > This fixes the screen blinking issue that happen when calling > DRM_IOCTL_MODE_GETCONNECTOR while PSR is active. The blinking > is caused by full

Re: [Intel-gfx] [PATCH] drm/i915/dp: Don't invalidate intel_dp_detect result

2017-08-23 Thread Rodrigo Vivi
On Wed, Aug 23, 2017 at 4:32 PM, Puthikorn Voravootivat wrote: >> what about instead of removing this "detect_done=false" like we move >> to another place like whenever conector is disconected? > > Look like that is already done in >

Re: [Intel-gfx] [PATCH] drm/i915/dp: Don't invalidate intel_dp_detect result

2017-08-23 Thread Puthikorn Voravootivat
> what about instead of removing this "detect_done=false" like we move > to another place like whenever conector is disconected? Look like that is already done in https://patchwork.freedesktop.org/patch/113363/ On Wed, Aug 23, 2017 at 4:00 PM, Rodrigo Vivi wrote: > On

Re: [Intel-gfx] [PATCH] drm/i915/dp: Don't invalidate intel_dp_detect result

2017-08-23 Thread Rodrigo Vivi
On Wed, Aug 23, 2017 at 3:40 PM, Puthikorn Voravootivat wrote: > If the full detect is already done, we shouldn't need to do it > again. > > This fixes the screen blinking issue that happen when calling > DRM_IOCTL_MODE_GETCONNECTOR while PSR is active. The blinking > is

[Intel-gfx] [PATCH] drm/i915/dp: Don't invalidate intel_dp_detect result

2017-08-23 Thread Puthikorn Voravootivat
If the full detect is already done, we shouldn't need to do it again. This fixes the screen blinking issue that happen when calling DRM_IOCTL_MODE_GETCONNECTOR while PSR is active. The blinking is caused by full dp detect in intel_dp_long_pulse(). Signed-off-by: Puthikorn Voravootivat