Re: [Intel-gfx] [PATCH 07/24] drm/i915: Remove useless checks from primary enable/disable

2014-04-07 Thread Paulo Zanoni
2014-03-10 8:57 GMT-03:00 Daniel Vetter dan...@ffwll.ch: On Mon, Mar 10, 2014 at 12:20 PM, Ville Syrjälä ville.syrj...@linux.intel.com wrote: On Fri, Mar 07, 2014 at 10:29:26PM +0100, Daniel Vetter wrote: On Fri, Mar 07, 2014 at 06:32:14PM +0200, ville.syrj...@linux.intel.com wrote: From:

Re: [Intel-gfx] [PATCH 07/24] drm/i915: Remove useless checks from primary enable/disable

2014-03-10 Thread Ville Syrjälä
On Fri, Mar 07, 2014 at 10:29:26PM +0100, Daniel Vetter wrote: On Fri, Mar 07, 2014 at 06:32:14PM +0200, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä ville.syrj...@linux.intel.com We won't be calling intel_enable_primary_plane() or intel_disable_primary_plane() with the

Re: [Intel-gfx] [PATCH 07/24] drm/i915: Remove useless checks from primary enable/disable

2014-03-10 Thread Daniel Vetter
On Mon, Mar 10, 2014 at 12:20 PM, Ville Syrjälä ville.syrj...@linux.intel.com wrote: On Fri, Mar 07, 2014 at 10:29:26PM +0100, Daniel Vetter wrote: On Fri, Mar 07, 2014 at 06:32:14PM +0200, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä ville.syrj...@linux.intel.com We won't be

[Intel-gfx] [PATCH 07/24] drm/i915: Remove useless checks from primary enable/disable

2014-03-07 Thread ville . syrjala
From: Ville Syrjälä ville.syrj...@linux.intel.com We won't be calling intel_enable_primary_plane() or intel_disable_primary_plane() with the primary plane in the wrong state. So remove the useless DISPLAY_PLANE_ENABLE checks. Signed-off-by: Ville Syrjälä ville.syrj...@linux.intel.com ---

Re: [Intel-gfx] [PATCH 07/24] drm/i915: Remove useless checks from primary enable/disable

2014-03-07 Thread Daniel Vetter
On Fri, Mar 07, 2014 at 06:32:14PM +0200, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä ville.syrj...@linux.intel.com We won't be calling intel_enable_primary_plane() or intel_disable_primary_plane() with the primary plane in the wrong state. So remove the useless