Re: [Intel-gfx] debugging Haswell eDP black screen after S3

2013-05-17 Thread Ben Guthro
On Thu, May 16, 2013 at 9:24 AM, Ben Guthro wrote: > On Wed, May 15, 2013 at 4:42 PM, Ben Guthro wrote: >> On Tue, May 14, 2013 at 5:01 PM, Ben Guthro wrote: >>> I am attempting to debug an issue with some Haswell laptop systems >>> which do not restore their screen after resuming from S3 when r

Re: [Intel-gfx] debugging Haswell eDP black screen after S3

2013-05-17 Thread Ben Guthro
On Fri, May 17, 2013 at 7:52 AM, Ben Guthro wrote: > On Thu, May 16, 2013 at 9:24 AM, Ben Guthro wrote: >> On Wed, May 15, 2013 at 4:42 PM, Ben Guthro wrote: >>> On Tue, May 14, 2013 at 5:01 PM, Ben Guthro wrote: I am attempting to debug an issue with some Haswell laptop systems which

Re: [Intel-gfx] [PATCH 1/2 V3] drm/915: Add private api for power well usage

2013-05-17 Thread Jesse Barnes
A few comments and questions below. On Thu, 16 May 2013 15:52:36 +0800 Wang Xingchao wrote: > Haswell Display audio depends on power well in graphic side, it should > request power well before use it and release power well after use. > I915 will not shutdown power well if it detects audio is usi

Re: [Intel-gfx] [PATCH 2/5] drm/i915: track ring progression using seqnos

2013-05-17 Thread Ben Widawsky
On Mon, May 13, 2013 at 04:32:10PM +0300, Mika Kuoppala wrote: > Instead of relying in acthd, track ring seqno progression > to detect if ring has hung. > > v2: put hangcheck stuff inside struct (Chris Wilson) > > Signed-off-by: Mika Kuoppala > --- > drivers/gpu/drm/i915/i915_drv.h |

Re: [Intel-gfx] [PATCH 1/3] drm/i915: implement IPS feature

2013-05-17 Thread Rodrigo Vivi
Reviewed-by: Rodrigo Vivi On Thu, May 16, 2013 at 4:54 PM, Paulo Zanoni wrote: > From: Paulo Zanoni > > Intermediate Pixel Storage is a feature that should reduce the number > of times the display engine wakes up memory to read pixels, so it > should allow deeper PC states. IPS can only be enab