[PATCH] drivers/staging/winbond boolean negation and bitwise operation in wrong order

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Diego Liziero <diegoliz@xxxxxxxxx>

The semantic patch that makes this change is:
(http://www.emn.fr/x-info/coccinelle/)

@@ expression E; constant C; @@
(
- !E == C
+ E != C
)

Signed-off-by: Diego Liziero <diegoliz@xxxxxxxxx>

diff = 
--- ./drivers/staging/winbond/mds.c	2009-04-13 01:21:29.000000000 +0200
+++ /tmp/cocci-output-30079-9c4ec4-mds.c	2009-04-13 20:44:20.000000000 +0200
@@ -432,7 +432,7 @@ Mds_Tx(struct wbsoft_priv * adapter)
 		return;
 
 	//Only one thread can be run here
-	if (!atomic_inc_return(&pMds->TxThreadCount) == 1)
+	if (atomic_inc_return(&pMds->TxThreadCount) != 1)
 		goto cleanup;
 
 	// Start to fill the data
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux