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
signature.asc
Description: Digital signature
_______________________________________________ devel mailing list [email protected] http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
