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
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to