Re: [Intel-gfx] [PATCH 06/12] drm/i915: remove rps local variables

2014-03-20 Thread Chris Wilson
On Wed, Mar 19, 2014 at 06:31:13PM -0700, Ben Widawsky wrote:
 With the renamed RPS struct members, it's easier to skip the local
 variables which no longer clarify anything, and if anything just make
 the code harder to read.
 
 The real motivation for this patch is actually the next patch, which
 attempts to consolidate some of the functionality.
 
 Cc: Jeff McGee jeff.mc...@intel.com
 Signed-off-by: Ben Widawsky b...@bwidawsk.net
Reviewed-by: Chris Wilson ch...@chris-wilson.co.uk
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx


Re: [Intel-gfx] [PATCH 06/12] drm/i915: remove rps local variables

2014-03-20 Thread Daniel Vetter
On Thu, Mar 20, 2014 at 07:30:52AM +, Chris Wilson wrote:
 On Wed, Mar 19, 2014 at 06:31:13PM -0700, Ben Widawsky wrote:
  With the renamed RPS struct members, it's easier to skip the local
  variables which no longer clarify anything, and if anything just make
  the code harder to read.
  
  The real motivation for this patch is actually the next patch, which
  attempts to consolidate some of the functionality.
  
  Cc: Jeff McGee jeff.mc...@intel.com
  Signed-off-by: Ben Widawsky b...@bwidawsk.net
 Reviewed-by: Chris Wilson ch...@chris-wilson.co.uk

Merged up to this one to not have a badly diverging codebase between 3.15
and later kernels. Thanks for patchesreview.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx