[Intel-gfx] [PATCH 7/8] drm/i915: move sandybridge RC6 enable in resume after ring initialization

2011-03-21 Thread Zhenyu Wang
Move RC6 enable after we reset rings for all regines, as RC6 on Sandybridge would setup ring idle state, so I assume we should have setup rings already. Signed-off-by: Zhenyu Wang --- drivers/gpu/drm/i915/i915_drv.c |4 drivers/gpu/drm/i915/i915_suspend.c |3 --- 2 files changed

Re: [Intel-gfx] [PATCH 7/8] drm/i915: move sandybridge RC6 enable in resume after ring initialization

2011-03-21 Thread Jesse Barnes
On Mon, 21 Mar 2011 17:27:18 +0800 Zhenyu Wang wrote: > Move RC6 enable after we reset rings for all regines, as RC6 on > Sandybridge would setup ring idle state, so I assume we should have > setup rings already. So you're seeing the idle value get clobbered? Otherwise it's it better to set the

Re: [Intel-gfx] [PATCH 7/8] drm/i915: move sandybridge RC6 enable in resume after ring initialization

2011-03-21 Thread Zhenyu Wang
On 2011.03.21 10:46:53 -0700, Jesse Barnes wrote: > On Mon, 21 Mar 2011 17:27:18 +0800 > Zhenyu Wang wrote: > > > Move RC6 enable after we reset rings for all regines, as RC6 on > > Sandybridge would setup ring idle state, so I assume we should have > > setup rings already. > > So you're seeing