[Intel-gfx] [PATCH 3/4] drm/i915: extract constant offset setting

Ben Widawsky ben at bwidawsk.net
Thu Oct 13 19:14:10 CEST 2011


On Thu, Oct 13, 2011 at 10:46:29AM +0200, Daniel Vetter wrote:
> On Wed, Oct 12, 2011 at 03:56:21PM -0700, Ben Widawsky wrote:
> > Simple refactor.
> > 
> > Signed-off-by: Ben Widawsky <ben at bwidawsk.net>
> 
> Imo you can fold this in with the fix. Usually it's good to separate fixes
> from refactoring, but it this special case of just moving around code
> blocks, it doesn't add anything.
> 

I agree, and I had it like this locally, but Chris suggested that he
preferred the separation as it highlighted the fix.

I largely do not care, I just want an r-b. So you two can fight it out.

Ben



More information about the Intel-gfx mailing list