On Wed, Feb 25, 2015 at 03:29:05PM +0200, Jani Nikula wrote: > On Wed, 25 Feb 2015, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote: > > Static checkers complain that we should probably add curly braces > > because, from the indenting, it looks like seq_printf() should be inside > > the list_for_each_entry() loop. But the code is actually correct, it's > > just the indenting which is off. > > > > Besides fixing the indenting on seq_printf(), I did add curly braces, > > because generally mult-line indents should have curly braces to make > > them more readable. > > > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > > The unintended indent was left behind and not unindented in > > commit d7f46fc4e7323887494db13f063a8e59861fefb0 > Author: Ben Widawsky <benjamin.widawsky@xxxxxxxxx> > Date: Fri Dec 6 14:10:55 2013 -0800 > > drm/i915: Make pin count per VMA Added to the commit message. > > Reviewed-by: Jani Nikula <jani.nikula@xxxxxxxxx> Queued for -next, thanks for the patch. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
