On Thu, Jun 25, 2015 at 08:49:30AM -0700, Juston Li wrote: > I placed it above the else to make it consistent with the if statement with > similar comment that the else corresponds to: > > /* Set the primary display control */ > - if (!ctrl)+ if (!ctrl) { Once you add the else statement, then that kind of doesn't make sense. Sudip is right. It should be: } else { /* commentary about else side */ regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel