Re: [PATCH 4/6] Staging: bcm: Fixed indention for inner if-block

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux