The negation makes it a bool before the comparison and hence it will never evaluate to true. Signed-off-by: Roel Kluin <roel.kluin@gmail.com> Signed-off-by: David S. Miller <davem@davemloft.net>
@@ -596,7 +596,8 @@ static int receive(struct net_device *dev, int cnt)
state = 0;
/* not flag received */
- else if (!(bitstream & (0x1fe << j)) != (0x0fc << j)) {
+ else if ((bitstream & (0x1fe << j)) !=
+ (0x0fc << j)) {
if (state)
do_rxpacket(dev);
bc->hdlcrx.bufcnt = 0;