On Tue, Jun 11, 2013 at 10:33:06AM -0700, De Roo, Steven wrote: > Hi Dan, > > Can you elaborate your statement ? > I really think the logic was wrong... > It was a small style nit. Kernel style for braces is: if (...) { one line; } else { blah; blah; } As opposed to: if (...) one line; else { blah; blah; } This is all documented in CodingStyle. regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel