On Tue, Jun 10, 2014 at 09:22:03PM +0200, Matthias Beyer wrote: > The inner if-statement was aligned just like the outer one. Why? > > This indention was introduced by > > f34c488c3894968e8cdbdc3b1ed617d78315cace > > which is a indention-fix patch itself. That's why I'm curious about it. > > I did not merge these nested if-statements, as I don't know if I'm > destroying logical seperated checks with it. Your patch is correct. The wrong indenting here was introduced in f34c488c3894 ('Staging: bcm: DDRInit: fixed issues w/ indentation.') regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel