Re: [PATCH 2/7] drm/i915: Fix computation of last_adjustment for RPS autotuning

Chris Wilson
2015-03-18T09:06:03Z

On Wed, Mar 18, 2015 at 12:26:49PM +0530, Deepak S wrote:
> On Friday 06 March 2015 08:36 PM, Chris Wilson wrote:
> I think we should modify adj in GEN6_PM_RP_UP_EI_EXPIRED?
> if not not we might request higher freq since we add adj to new_delay before request freq.

Oh yeah. That branch didn't exist when I first wrote the patch, and
didn't exist at the end of the series so I never saw it!

For this point in the patch series, yes, we do need to set adj=0.
-Chris

--
Chris Wilson, Intel Open Source Technology Centre