Re: [Intel-gfx] [PATCH v2 2/9] drm/i915: Kill off intel_crtc->atomic.wait_vblank, v3.

2016-02-09 Thread Maarten Lankhorst
Hey, Op 13-01-16 om 13:58 schreef Ville Syrjälä: > On Mon, Jan 11, 2016 at 01:27:42PM +0100, Maarten Lankhorst wrote: >> Currently we perform our own wait in post_plane_update, >> but the atomic core performs another one in wait_for_vblanks. >> This means that 2 vblanks are done when a fb is

Re: [Intel-gfx] [PATCH v2 2/9] drm/i915: Kill off intel_crtc->atomic.wait_vblank, v3.

2016-02-09 Thread Ville Syrjälä
On Tue, Feb 09, 2016 at 11:27:44AM +0100, Maarten Lankhorst wrote: > Hey, > > Op 13-01-16 om 13:58 schreef Ville Syrjälä: > > On Mon, Jan 11, 2016 at 01:27:42PM +0100, Maarten Lankhorst wrote: > >> Currently we perform our own wait in post_plane_update, > >> but the atomic core performs another

Re: [Intel-gfx] [PATCH v2 2/9] drm/i915: Kill off intel_crtc->atomic.wait_vblank, v3.

2016-02-09 Thread Maarten Lankhorst
Op 09-02-16 om 11:42 schreef Ville Syrjälä: > On Tue, Feb 09, 2016 at 11:27:44AM +0100, Maarten Lankhorst wrote: >> Hey, >> >> Op 13-01-16 om 13:58 schreef Ville Syrjälä: >>> On Mon, Jan 11, 2016 at 01:27:42PM +0100, Maarten Lankhorst wrote: Currently we perform our own wait in

Re: [Intel-gfx] [PATCH v2 2/9] drm/i915: Kill off intel_crtc->atomic.wait_vblank, v3.

2016-02-09 Thread Ville Syrjälä
On Tue, Feb 09, 2016 at 12:07:01PM +0100, Maarten Lankhorst wrote: > Op 09-02-16 om 11:42 schreef Ville Syrjälä: > > On Tue, Feb 09, 2016 at 11:27:44AM +0100, Maarten Lankhorst wrote: > >> Hey, > >> > >> Op 13-01-16 om 13:58 schreef Ville Syrjälä: > >>> On Mon, Jan 11, 2016 at 01:27:42PM +0100,

Re: [Intel-gfx] [PATCH v2 2/9] drm/i915: Kill off intel_crtc->atomic.wait_vblank, v3.

2016-01-13 Thread Ville Syrjälä
On Mon, Jan 11, 2016 at 01:27:42PM +0100, Maarten Lankhorst wrote: > Currently we perform our own wait in post_plane_update, > but the atomic core performs another one in wait_for_vblanks. > This means that 2 vblanks are done when a fb is changed, > which is a bit overkill. > > Merge them by

[Intel-gfx] [PATCH v2 2/9] drm/i915: Kill off intel_crtc->atomic.wait_vblank, v3.

2016-01-11 Thread Maarten Lankhorst
Currently we perform our own wait in post_plane_update, but the atomic core performs another one in wait_for_vblanks. This means that 2 vblanks are done when a fb is changed, which is a bit overkill. Merge them by creating a helper function that takes a crtc mask for the planes to wait on. The