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
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel