Re: [Intel-gfx] [PATCH v2] drm/i915: Introduce intel_set_rps()

2015-02-03 Thread Daniel Vetter
On Tue, Feb 03, 2015 at 01:36:50PM +, Chris Wilson wrote: On Mon, Feb 02, 2015 at 07:09:50PM +0200, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä ville.syrj...@linux.intel.com Replace the valleyview_set_rps() and gen6_set_rps() calls with intel_set_rps() which itself does

Re: [Intel-gfx] [PATCH v2] drm/i915: Introduce intel_set_rps()

2015-02-03 Thread Chris Wilson
On Mon, Feb 02, 2015 at 07:09:50PM +0200, ville.syrj...@linux.intel.com wrote: From: Ville Syrjälä ville.syrj...@linux.intel.com Replace the valleyview_set_rps() and gen6_set_rps() calls with intel_set_rps() which itself does the IS_VALLEYVIEW() check. The code becomes simpler since the

[Intel-gfx] [PATCH v2] drm/i915: Introduce intel_set_rps()

2015-02-02 Thread ville . syrjala
From: Ville Syrjälä ville.syrj...@linux.intel.com Replace the valleyview_set_rps() and gen6_set_rps() calls with intel_set_rps() which itself does the IS_VALLEYVIEW() check. The code becomes simpler since the callers don't have to do this check themselves. Most of the change was performe with