I removed a misplace semicolon. It is clear from the indentation that TxPwrTracking87SE() was only supposed to be called if CheckTxPwrTracking() returned true. Signed-off-by: Dan Carpenter <error27@xxxxxxxxx> --- orig/drivers/staging/rtl8187se/r8180_core.c 2010-01-18 08:27:05.000000000 +0300 +++ devel/drivers/staging/rtl8187se/r8180_core.c 2010-01-18 08:27:40.000000000 +0300 @@ -2566,7 +2566,7 @@ void watch_dog_adaptive(unsigned long da // Tx Power Tracking on 87SE. #ifdef TX_TRACK //if( priv->bTxPowerTrack ) //lzm mod 080826 - if( CheckTxPwrTracking((struct net_device *)data)); + if (CheckTxPwrTracking((struct net_device *)data)) TxPwrTracking87SE((struct net_device *)data); #endif _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel