Re: [Intel-gfx] [PATCH 5/6] drm/i915: add i915.pc8_timeout function

2013-08-21 Thread Rodrigo Vivi
Reviewed-by: Rodrigo Vivi On Mon, Aug 19, 2013 at 1:18 PM, Paulo Zanoni wrote: > From: Paulo Zanoni > > We currently only enter PC8+ after all its required conditions are > met, there's no rendering, and we stay like that for at least 5 > seconds. > > I chose "5 seconds" because this value is c

[Intel-gfx] [PATCH 5/6] drm/i915: add i915.pc8_timeout function

2013-08-19 Thread Paulo Zanoni
From: Paulo Zanoni We currently only enter PC8+ after all its required conditions are met, there's no rendering, and we stay like that for at least 5 seconds. I chose "5 seconds" because this value is conservative and won't make us enter/leave PC8+ thousands of times after the screen is off: som