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ä > > > > Replace the valleyview_set_rps() and gen6_set_rps() calls with > > intel_set_rps() which itself does the IS_VALLEYVIEW() c

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ä > > 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

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

2015-02-02 Thread ville . syrjala
From: Ville Syrjälä 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 the following semantic patch: @@