diff options
| author | Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se> | 2014-06-07 07:26:37 -0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2014-06-11 03:13:16 -0400 |
| commit | f05d435bde588ac26ba75f7f6893b2a63ae71381 (patch) | |
| tree | fa4cf2b2306cdc957408e6a8eeedbfc3b3559109 | |
| parent | 655aa393063b0cca03d1c4d1d2971b9b9770525b (diff) | |
net: wimax: i2400m: control.c: Cleaning up conjunction always evaluates to false
Logical conjunction always evaluates to false: minor < 2 && minor > 1
I guess what you wanted is rather: minor > 2 || minor < 1
This was partly found using a static code analysis program called cppcheck.
Signed-off-by: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | drivers/net/wimax/i2400m/control.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wimax/i2400m/control.c b/drivers/net/wimax/i2400m/control.c index 4a01e5c7fe09..4c417903e9be 100644 --- a/drivers/net/wimax/i2400m/control.c +++ b/drivers/net/wimax/i2400m/control.c | |||
| @@ -1061,7 +1061,7 @@ int i2400m_firmware_check(struct i2400m *i2400m) | |||
| 1061 | goto error_bad_major; | 1061 | goto error_bad_major; |
| 1062 | } | 1062 | } |
| 1063 | result = 0; | 1063 | result = 0; |
| 1064 | if (minor < I2400M_HDIv_MINOR_2 && minor > I2400M_HDIv_MINOR) | 1064 | if (minor > I2400M_HDIv_MINOR_2 || minor < I2400M_HDIv_MINOR) |
| 1065 | dev_warn(dev, "untested minor fw version %u.%u.%u\n", | 1065 | dev_warn(dev, "untested minor fw version %u.%u.%u\n", |
| 1066 | major, minor, branch); | 1066 | major, minor, branch); |
| 1067 | /* Yes, we ignore the branch -- we don't have to track it */ | 1067 | /* Yes, we ignore the branch -- we don't have to track it */ |