On Mon, Jan 28, 2013 at 12:54:26PM +0200, Imre Deak wrote: > On Fri, 2013-01-25 at 17:53 +0100, Daniel Vetter wrote: > > That way the control flow is clearer, and it prepares the stage > > to extract these ums functions and hide them somewhere. > > > > There's still tons of display stuff outside of these, but that > > requires more work. > > > > Signed-off-by: Daniel Vetter <daniel.vetter at ffwll.ch> > s/modeset/display/ as discussed on IRC. Also would be nice to use > format-patch -C for things like patch 2/4 in the future, as that'd make > reviewing easier imo. Otherwise on the series: > > Reviewed-by: Imre Deak <imre.deak at intel.com> Ok, slurped in the entire series, with your suggestion applied to patch 2 (less conflicts that way to resolve). Thanks for the review. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch