Re: [Intel-gfx] [PATCH v2 3/9] drm/i915: Remove intel_crtc->atomic.disable_ips.

2016-01-18 Thread Daniel Stone
Hi, On 18 January 2016 at 12:10, Maarten Lankhorst wrote: > Op 13-01-16 om 14:02 schreef Ville Syrjälä: >> Also I'm not sure it isn't a step backwards. Based on the spec we should >> be able to keep IPS enabled as long as one plane (possibly referring to >>

Re: [Intel-gfx] [PATCH v2 3/9] drm/i915: Remove intel_crtc->atomic.disable_ips.

2016-01-18 Thread Maarten Lankhorst
Op 13-01-16 om 14:02 schreef Ville Syrjälä: > On Mon, Jan 11, 2016 at 01:27:43PM +0100, Maarten Lankhorst wrote: >> This is a revert of commit 066cf55b9ce3 "drm/i915: Fix IPS related flicker". >> intel_pre_disable_primary already handles this, and now everything >> goes through the atomic path

Re: [Intel-gfx] [PATCH v2 3/9] drm/i915: Remove intel_crtc->atomic.disable_ips.

2016-01-13 Thread Ville Syrjälä
On Mon, Jan 11, 2016 at 01:27:43PM +0100, Maarten Lankhorst wrote: > This is a revert of commit 066cf55b9ce3 "drm/i915: Fix IPS related flicker". > intel_pre_disable_primary already handles this, and now everything > goes through the atomic path there's no need to try to disable ips twice. If

[Intel-gfx] [PATCH v2 3/9] drm/i915: Remove intel_crtc->atomic.disable_ips.

2016-01-11 Thread Maarten Lankhorst
This is a revert of commit 066cf55b9ce3 "drm/i915: Fix IPS related flicker". intel_pre_disable_primary already handles this, and now everything goes through the atomic path there's no need to try to disable ips twice. Signed-off-by: Maarten Lankhorst