Re: [Intel-gfx] [PATCH v2 07/17] drm/i915: Use crtc_state->active instead of crtc_state->enable

2015-05-19 Thread Daniel Vetter
On Mon, May 18, 2015 at 06:35:59PM +0200, Maarten Lankhorst wrote: > Op 18-05-15 om 17:30 schreef Daniel Vetter: > > On Wed, May 13, 2015 at 10:23:37PM +0200, Maarten Lankhorst wrote: > >> crtc_state->enable means a crtc is configured, but it may be turned > >> off for dpms. Until the previous comm

Re: [Intel-gfx] [PATCH v2 07/17] drm/i915: Use crtc_state->active instead of crtc_state->enable

2015-05-18 Thread Maarten Lankhorst
Op 18-05-15 om 17:30 schreef Daniel Vetter: > On Wed, May 13, 2015 at 10:23:37PM +0200, Maarten Lankhorst wrote: >> crtc_state->enable means a crtc is configured, but it may be turned >> off for dpms. Until the previous commit crtc_state->active was not >> updated on crtc off, but now that we do we

Re: [Intel-gfx] [PATCH v2 07/17] drm/i915: Use crtc_state->active instead of crtc_state->enable

2015-05-18 Thread Daniel Vetter
On Wed, May 13, 2015 at 10:23:37PM +0200, Maarten Lankhorst wrote: > crtc_state->enable means a crtc is configured, but it may be turned > off for dpms. Until the previous commit crtc_state->active was not > updated on crtc off, but now that we do we should use that for tracking > whether a crtc is

[Intel-gfx] [PATCH v2 07/17] drm/i915: Use crtc_state->active instead of crtc_state->enable

2015-05-13 Thread Maarten Lankhorst
crtc_state->enable means a crtc is configured, but it may be turned off for dpms. Until the previous commit crtc_state->active was not updated on crtc off, but now that we do we should use that for tracking whether a crtc is scanning out or not. At this point crtc->active should mirror crtc_state-