diff options
author | Vladimir Oltean <vladimir.oltean@nxp.com> | 2023-04-02 15:37:50 +0300 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2023-04-03 12:04:26 +0300 |
commit | 1193db2a55b6b04f296f03affdfa80c16ecc3814 (patch) | |
tree | 95db1d8a10f5e5cbd3406fc9e99f986cd3d96f60 /net/core/dev_ioctl.c | |
parent | 00d521b39307c3232bdc685c2e9fd82ed973ac24 (diff) | |
download | linux-1193db2a55b6b04f296f03affdfa80c16ecc3814.tar.xz |
net: simplify handling of dsa_ndo_eth_ioctl() return code
In the expression "x == 0 || x != -95", the term "x == 0" does not
change the expression's logical value, because 0 != -95, and so,
if x is 0, the expression would still be true by virtue of the second
term. If x is non-zero, the expression depends on the truth value of
the second term anyway. As such, the first term is redundant and can
be deleted.
Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/core/dev_ioctl.c')
-rw-r--r-- | net/core/dev_ioctl.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/core/dev_ioctl.c b/net/core/dev_ioctl.c index 1c0256ea522f..b299fb23fcfa 100644 --- a/net/core/dev_ioctl.c +++ b/net/core/dev_ioctl.c @@ -249,7 +249,7 @@ static int dev_eth_ioctl(struct net_device *dev, int err; err = dsa_ndo_eth_ioctl(dev, ifr, cmd); - if (err == 0 || err != -EOPNOTSUPP) + if (err != -EOPNOTSUPP) return err; if (ops->ndo_eth_ioctl) { |