On Tue, Apr 10, 2012 at 04:33:38PM +0100, Chris Wilson wrote: > On Tue, 10 Apr 2012 17:29:17 +0200, daniel.vetter at ffwll.ch wrote: > > From: Daniel Vetter <daniel.vetter at ffwll.ch> > > > > We don't need the pt_addr for the !dmar case, so drop the else and > > move the if (dmar) condition out of the loop. > > > > v2: Fixup whitespace damage noticed by Chris Wilson. > > > > v3: Collapse the two identical if blocks. Chris Wilson makes me look > > like a moron right now ... > > > > Noticed-by: Konstantin Belousov <kostikbel at gmail.com> > > Signed-Off-by: Daniel Vetter <daniel.vetter at ffwll.ch> > > Reviewed-by: Chris Wilson <chris at chris-wislon.co.uk> Queued for -next, thanks for the review. -Daniel -- Daniel Vetter Mail: daniel at ffwll.ch Mobile: +41 (0)79 365 57 48