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

Zhenyu Wang
2011-03-22T01:48:47Z

[-- Attachment #1.1: Type: text/plain, Size: 670 bytes --]

On 2011.03.21 10:46:53 -0700, Jesse Barnes wrote:
> On Mon, 21 Mar 2011 17:27:18 +0800
> Zhenyu Wang <zhenyuw@linux.intel.com> 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 ring idle value before actually starting the ring?
>

Don't know, but I can ask if we should do like that or not. I had more
stable test result with this.

--
Open Source Technology Center, Intel ltd.

$gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

[-- Attachment #2: Type: text/plain, Size: 159 bytes --]

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org