Re: [PATCH v6] drm/i915/vlv: WA for Turbo and RC6 to work together.

From: Jesse Barnes <jbarnes@virtuousgeek.org>
To: deepak.s@linux.intel.com
Cc: intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH v6] drm/i915/vlv: WA for Turbo and RC6 to work together.
Date: Tue, 13 May 2014 15:12:46 -0700	[thread overview]
Message-ID: <20140513151246.31362e27@jbarnes-desktop> (raw)
In-Reply-To: <1396160928-2793-1-git-send-email-deepak.s@linux.intel.com>

On Sun, 30 Mar 2014 11:58:48 +0530
deepak.s@linux.intel.com wrote:

> @@ -843,6 +849,8 @@ struct intel_gen6_power_mgmt {
>  	bool rp_up_masked;
>  	bool rp_down_masked;
>  
> +	u32 ei_interrupt_count;
> +
>  	int last_adj;
>  	enum { LOW_POWER, BETWEEN, HIGH_POWER } power;
>  
> @@ -1403,6 +1411,13 @@ typedef struct drm_i915_private {
>  	/* gen6+ rps state */
>  	struct intel_gen6_power_mgmt rps;
>  
> +	/* rps wa up ei calculation */
> +	struct intel_rps_ei_calc rps_up_ei;
> +
> +	/* rps wa down ei calculation */
> +	struct intel_rps_ei_calc rps_down_ei;
> +

I think Chris meant that these bits belonged in intel_gen6_power_mgmt
too.

Other than that it looks like Ville has given this his r-b so it ought
to be fine.

-- 
Jesse Barnes, Intel Open Source Technology Center