Re: [Intel-gfx] [PATCH 4/5] drm/i915: check the power well on i915_pipe_enabled

2013-05-07 Thread Daniel Vetter
On Tue, May 7, 2013 at 4:18 PM, Paulo Zanoni wrote: > 2013/5/7 Daniel Vetter : >> On Fri, May 03, 2013 at 12:15:39PM -0300, Paulo Zanoni wrote: >>> From: Paulo Zanoni >>> >>> This fixes "unclaimed register" messages when the power well is >>> disabled and there's a GPU hang. >> >> How does this b

Re: [Intel-gfx] [PATCH 4/5] drm/i915: check the power well on i915_pipe_enabled

2013-05-07 Thread Paulo Zanoni
2013/5/7 Daniel Vetter : > On Fri, May 03, 2013 at 12:15:39PM -0300, Paulo Zanoni wrote: >> From: Paulo Zanoni >> >> This fixes "unclaimed register" messages when the power well is >> disabled and there's a GPU hang. > > How does this blow up in a gpu hang? Afaics pipe_enabled is mostly called > i

Re: [Intel-gfx] [PATCH 4/5] drm/i915: check the power well on i915_pipe_enabled

2013-05-07 Thread Daniel Vetter
On Fri, May 03, 2013 at 12:15:39PM -0300, Paulo Zanoni wrote: > From: Paulo Zanoni > > This fixes "unclaimed register" messages when the power well is > disabled and there's a GPU hang. How does this blow up in a gpu hang? Afaics pipe_enabled is mostly called in the vblank code, so this should b

Re: [Intel-gfx] [PATCH 4/5] drm/i915: check the power well on i915_pipe_enabled

2013-05-07 Thread Damien Lespiau
On Fri, May 03, 2013 at 12:15:39PM -0300, Paulo Zanoni wrote: > From: Paulo Zanoni > > This fixes "unclaimed register" messages when the power well is > disabled and there's a GPU hang. > > v2: Use the new intel_display_power_enabled(). > v3: Use the new domains for intel_display_power_enabled()

[Intel-gfx] [PATCH 4/5] drm/i915: check the power well on i915_pipe_enabled

2013-05-03 Thread Paulo Zanoni
From: Paulo Zanoni This fixes "unclaimed register" messages when the power well is disabled and there's a GPU hang. v2: Use the new intel_display_power_enabled(). v3: Use the new domains for intel_display_power_enabled(). Signed-off-by: Paulo Zanoni --- drivers/gpu/drm/i915/i915_irq.c |4