Re: [Intel-gfx] [PATCH 2/3] drm/i915: Remove atomic.pre_disable_primary.

2016-02-04 Thread Maarten Lankhorst
Op 03-02-16 om 18:39 schreef Ville Syrjälä: > On Wed, Feb 03, 2016 at 06:23:13PM +0100, Maarten Lankhorst wrote: >> Op 03-02-16 om 17:07 schreef Ville Syrjälä: >>> On Wed, Feb 03, 2016 at 04:53:24PM +0100, Maarten Lankhorst wrote: This can be derived from the atomic state in pre_plane_update,

[Intel-gfx] [PATCH 2/3] drm/i915: Remove atomic.pre_disable_primary.

2016-02-03 Thread Maarten Lankhorst
This can be derived from the atomic state in pre_plane_update, which makes it more clear when it's supposed to be called. Reviewed-by: Ander Conselvan de Oliveira Signed-off-by: Maarten Lankhorst --- drivers/gpu/drm/i915/intel_display.c

Re: [Intel-gfx] [PATCH 2/3] drm/i915: Remove atomic.pre_disable_primary.

2016-02-03 Thread Ville Syrjälä
On Wed, Feb 03, 2016 at 04:53:24PM +0100, Maarten Lankhorst wrote: > This can be derived from the atomic state in pre_plane_update, > which makes it more clear when it's supposed to be called. > > Reviewed-by: Ander Conselvan de Oliveira > Signed-off-by: Maarten Lankhorst

Re: [Intel-gfx] [PATCH 2/3] drm/i915: Remove atomic.pre_disable_primary.

2016-02-03 Thread Maarten Lankhorst
Op 03-02-16 om 17:07 schreef Ville Syrjälä: > On Wed, Feb 03, 2016 at 04:53:24PM +0100, Maarten Lankhorst wrote: >> This can be derived from the atomic state in pre_plane_update, >> which makes it more clear when it's supposed to be called. >> >> Reviewed-by: Ander Conselvan de Oliveira

Re: [Intel-gfx] [PATCH 2/3] drm/i915: Remove atomic.pre_disable_primary.

2016-02-03 Thread Ville Syrjälä
On Wed, Feb 03, 2016 at 06:23:13PM +0100, Maarten Lankhorst wrote: > Op 03-02-16 om 17:07 schreef Ville Syrjälä: > > On Wed, Feb 03, 2016 at 04:53:24PM +0100, Maarten Lankhorst wrote: > >> This can be derived from the atomic state in pre_plane_update, > >> which makes it more clear when it's