staging:vt6656: Removed assignment in if statement.
authorChi Pham <fempsci@gmail.com>
Sun, 9 Mar 2014 13:11:11 +0000 (14:11 +0100)
committerPeter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
Mon, 10 Mar 2014 21:56:07 +0000 (14:56 -0700)
The following coccinelle script found the match:

@@
expression E0, E1, E2;
statement S0;
@@
- if ((E1 = E2) == E0)
+ E1 = E2;
+ if (E1 == E0)
S0

Signed-off-by: Chi Pham <fempsci@gmail.com>
Acked-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@intel.com>
drivers/staging/vt6656/iwctl.c

index bedc8104e835bc893427911818e3456e4da62b7f..cf4c06a4288093992959e6d26725ef6ca8b8970d 100644 (file)
@@ -1395,7 +1395,8 @@ int iwctl_giwpower(struct net_device *dev, struct iw_request_info *info,
        if (pMgmt == NULL)
                return -EFAULT;
 
-       if ((wrq->disabled = (mode == WMAC_POWER_CAM)))
+       wrq->disabled = (mode == WMAC_POWER_CAM);
+       if (wrq->disabled)
                return 0;
 
        if ((wrq->flags & IW_POWER_TYPE) == IW_POWER_TIMEOUT) {