Re: [Intel-gfx] [PATCH 2/2] drm/i915: Ignore stale wm register values on resume on ilk-bdw

2016-05-13 Thread Ville Syrjälä
On Fri, May 13, 2016 at 10:35:16AM -0700, Matt Roper wrote: > On Fri, May 13, 2016 at 08:18:18PM +0300, Ville Syrjälä wrote: > > On Fri, May 13, 2016 at 09:58:55AM -0700, Matt Roper wrote: > > > On Fri, May 13, 2016 at 05:55:18PM +0300, ville.syrj...@linux.intel.com > > > wrote: > > > > From:

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Ignore stale wm register values on resume on ilk-bdw

2016-05-13 Thread Matt Roper
On Fri, May 13, 2016 at 08:18:18PM +0300, Ville Syrjälä wrote: > On Fri, May 13, 2016 at 09:58:55AM -0700, Matt Roper wrote: > > On Fri, May 13, 2016 at 05:55:18PM +0300, ville.syrj...@linux.intel.com > > wrote: > > > From: Ville Syrjälä > > > > > > When we resume

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Ignore stale wm register values on resume on ilk-bdw

2016-05-13 Thread Ville Syrjälä
On Fri, May 13, 2016 at 09:58:55AM -0700, Matt Roper wrote: > On Fri, May 13, 2016 at 05:55:18PM +0300, ville.syrj...@linux.intel.com wrote: > > From: Ville Syrjälä > > > > When we resume the watermark register may contain some BIOS leftovers, > > or just the

[Intel-gfx] [PATCH 2/2] drm/i915: Ignore stale wm register values on resume on ilk-bdw (v2)

2016-05-13 Thread Matt Roper
From: Ville Syrjälä When we resume the watermark register may contain some BIOS leftovers, or just the hardware reset values. We should ignore those as the pipes will be off anyway, and so frobbing around with intermediate watermarks doesn't make much sense. In

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Ignore stale wm register values on resume on ilk-bdw

2016-05-13 Thread Matt Roper
On Fri, May 13, 2016 at 05:55:18PM +0300, ville.syrj...@linux.intel.com wrote: > From: Ville Syrjälä > > When we resume the watermark register may contain some BIOS leftovers, > or just the hardware reset values. We should ignore those as the > pipes will be off

[Intel-gfx] [PATCH 2/2] drm/i915: Ignore stale wm register values on resume on ilk-bdw

2016-05-13 Thread ville . syrjala
From: Ville Syrjälä When we resume the watermark register may contain some BIOS leftovers, or just the hardware reset values. We should ignore those as the pipes will be off anyway, and so frobbing around with intermediate watermarks doesn't make much sense. In