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
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to