Re: [PATCH 1/6] Staging: bcm: Fix coding style issues in CmHost.c

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

 



On Mon, Jan 02, 2012 at 11:17:16PM -0500, Kevin McKinney wrote:
> -			else
> -			{
> -				//This Classifier Already Exists and it is invalid to Add Classifier with existing PCRI
> -				BCM_DEBUG_PRINT( Adapter,DBG_TYPE_OTHERS, CONN_MSG, DBG_LVL_ALL,"CopyToAdapter : Error The Specified Classifier Already Exists \
> -						and attempted To Add Classifier with Same PCRI : 0x%x\n", u16PacketClassificationRuleIndex);
> +				/* Copy the Classifier Rule for this service flow into our Classifier table maintained per SF. */
> +				CopyClassifierRuleToSF(Adapter, psfCSType, uiSearchRuleIndex, nClassifierIndex);
> +			} else {
> +				/* This Classifier Already Exists and it is invalid to Add Classifier with existing PCRI */
> +				BCM_DEBUG_PRINT(Adapter, DBG_TYPE_OTHERS, CONN_MSG, DBG_LVL_ALL,
> +						"CopyToAdapter: Error The Specified Classifier Already Exists and attempted To Add Classifier with Same PCRI : 0x%x\n",
> +						u16PacketClassificationRuleIndex);
>  			}

Btw, I don't know if you noticed, but this is a small bugfix.  The
original code but bogus tabs into the printk.

Was there some automated tool to find these?

regards,
dan carpenter

Attachment: signature.asc
Description: Digital signature

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