Re: [Intel-gfx] [PATCH v2 3/7] drm/i915: Fix fbdev sprite disable code

2013-06-03 Thread Ville Syrjälä
On Mon, Jun 03, 2013 at 02:00:00PM +0200, Daniel Vetter wrote: > On Fri, May 31, 2013 at 08:07:03PM +0300, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä > > > > plane->enabled is never set, so this code didn't do anything. > > > > Fix the code for sprites by calling the new drm_pl

Re: [Intel-gfx] [PATCH v2 3/7] drm/i915: Fix fbdev sprite disable code

2013-06-03 Thread Daniel Vetter
On Fri, May 31, 2013 at 08:07:03PM +0300, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > plane->enabled is never set, so this code didn't do anything. > > Fix the code for sprites by calling the new drm_plane_force_disable() > function. That means the plane will remain off until

Re: [Intel-gfx] [PATCH v2 3/7] drm/i915: Fix fbdev sprite disable code

2013-06-03 Thread Chris Wilson
On Fri, May 31, 2013 at 08:07:03PM +0300, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > plane->enabled is never set, so this code didn't do anything. > > Fix the code for sprites by calling the new drm_plane_force_disable() > function. That means the plane will remain off until

[Intel-gfx] [PATCH v2 3/7] drm/i915: Fix fbdev sprite disable code

2013-05-31 Thread ville . syrjala
From: Ville Syrjälä plane->enabled is never set, so this code didn't do anything. Fix the code for sprites by calling the new drm_plane_force_disable() function. That means the plane will remain off until someone explicitly turns it back on. And do the same for cursors and the old video overlay