On Mon, Jan 02, 2012 at 11:17:16PM -0500, Kevin McKinney wrote: > case eDeletePHSRule: > { > - BCM_DEBUG_PRINT(Adapter,DBG_TYPE_OTHERS, CONN_MSG, DBG_LVL_ALL,"PHS DSC Action = Delete PHS Rule \n"); > + BCM_DEBUG_PRINT(Adapter, DBG_TYPE_OTHERS, CONN_MSG, DBG_LVL_ALL, "PHS DSC Action = Delete PHS Rule\n"); > > - if(psfCSType->cPhsRule.u8PHSI) > - { > - PhsDeletePHSRule( > - &Adapter->stBCMPhsContext, > - uVCID, > - psfCSType->cCPacketClassificationRule.u8AssociatedPHSI); > - } > + if (psfCSType->cPhsRule.u8PHSI) > + PhsDeletePHSRule(&Adapter->stBCMPhsContext, uVCID, psfCSType->cCPacketClassificationRule.u8AssociatedPHSI); > else > - { > - //BCM_DEBUG_PRINT(CONN_MSG,("Error CPHSRule.PHSI is ZERO \n")); > - } > + /* BCM_DEBUG_PRINT(CONN_MSG,("Error CPHSRule.PHSI is ZERO\n")); */ > + > break; This is broken. You removed the empty curly braces so now onle the else statement breaks. You have to remove the else statement as well. regards, dan carpenter
Attachment:
signature.asc
Description: Digital signature
_______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel