On Tue, Jun 10, 2014 at 08:09:29PM +0200, Matthias Beyer wrote: > - if (psIntfAdapter->bSuspended == TRUE || > psIntfAdapter->bPreparingForBusSuspend == TRUE) { > - BCM_DEBUG_PRINT(psIntfAdapter->psAdapter, DBG_TYPE_OTHERS, RDM, > DBG_LVL_ALL, "Bus is in suspended states hence RDM not allowed.."); > + if (psIntfAdapter->bSuspended == TRUE > + || psIntfAdapter->bPreparingForBusSuspend == TRUE) {
For conditions, put the "||" or "&&" on the first line and line up the arguments like this: if (psIntfAdapter->bSuspended == TRUE || psIntfAdapter->bPreparingForBusSuspend == TRUE) { [tab][space][space][space][space]psIntfAdapter->... regards, dan carpenter _______________________________________________ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel