Re: [PATCH] drm/i915: Treat crtc->mode.clock == 0 as disabled

2013-01-07 Thread Daniel Vetter
On Mon, Jan 07, 2013 at 10:11:40AM +, Chris Wilson wrote: > Prevent a divide-by-zero by consistently treating an 'active' CRTC > without a mode set as actually disabled. > > This looks to have been first introduced with > > commit 24929352481f085c5f85d4d4cbc919ddf106d381 > Author: Daniel Vett

[PATCH] drm/i915: Treat crtc->mode.clock == 0 as disabled

2013-01-07 Thread Chris Wilson
Prevent a divide-by-zero by consistently treating an 'active' CRTC without a mode set as actually disabled. This looks to have been first introduced with commit 24929352481f085c5f85d4d4cbc919ddf106d381 Author: Daniel Vetter Date: Mon Jul 2 20:28:59 2012 +0200 drm/i915: read out the modese

Re: [PATCH] drm/i915: Treat crtc->mode.clock == 0 as disabled

2013-01-06 Thread Daniel Vetter
On Fri, Jan 4, 2013 at 12:39 PM, Chris Wilson wrote: > Prevent a divide-by-zero by consistently treating an 'active' CRTC > without a mode set as actually disabled. Can you please resend with sob/tested-by/lkml-references and regression sha1 citation and maybe a small comment in the code that thi

Re: [PATCH] drm/i915: Treat crtc->mode.clock == 0 as disabled

2013-01-04 Thread Alexey Zaytsev
On Fri, Jan 4, 2013 at 1:39 PM, Chris Wilson wrote: > Prevent a divide-by-zero by consistently treating an 'active' CRTC > without a mode set as actually disabled. > --- Works fine here, thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to

[PATCH] drm/i915: Treat crtc->mode.clock == 0 as disabled

2013-01-04 Thread Chris Wilson
Prevent a divide-by-zero by consistently treating an 'active' CRTC without a mode set as actually disabled. --- drivers/gpu/drm/i915/intel_pm.c | 22 +- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel