Re: [Intel-gfx] [PATCH v2 2/2] drm/i915: Don't enable the cursor on a disable pipe

2013-09-17 Thread Paulo Zanoni
2013/9/17 Ville Syrjälä : > On Tue, Sep 17, 2013 at 05:24:01PM +0100, Chris Wilson wrote: >> On Tue, Sep 17, 2013 at 06:33:44PM +0300, ville.syrj...@linux.intel.com >> wrote: >> > From: Ville Syrjälä >> > >> > On HSW enabling a plane on a disabled pipe may hang the entire system. >> > And there's

Re: [Intel-gfx] [PATCH v2 2/2] drm/i915: Don't enable the cursor on a disable pipe

2013-09-17 Thread Ville Syrjälä
On Tue, Sep 17, 2013 at 05:24:01PM +0100, Chris Wilson wrote: > On Tue, Sep 17, 2013 at 06:33:44PM +0300, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä > > > > On HSW enabling a plane on a disabled pipe may hang the entire system. > > And there's no good reason for doing it ever, s

Re: [Intel-gfx] [PATCH v2 2/2] drm/i915: Don't enable the cursor on a disable pipe

2013-09-17 Thread Chris Wilson
On Tue, Sep 17, 2013 at 06:33:44PM +0300, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > On HSW enabling a plane on a disabled pipe may hang the entire system. > And there's no good reason for doing it ever, so just don't. > > v2: Move the crtc active checks to intel_crtc_cursor_

[Intel-gfx] [PATCH v2 2/2] drm/i915: Don't enable the cursor on a disable pipe

2013-09-17 Thread ville . syrjala
From: Ville Syrjälä On HSW enabling a plane on a disabled pipe may hang the entire system. And there's no good reason for doing it ever, so just don't. v2: Move the crtc active checks to intel_crtc_cursor_{set,move} to avoid confusing people during modeset Cc: sta...@vger.kernel.org Signed-