Re: [PATCH v2] Staging: bcm: Shortened lines

[Dan Carpenter <dan.carpenter@xxxxxxxxxx>]

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
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel