diff options
| author | Rémi Denis-Courmont <remi.denis-courmont@nokia.com> | 2009-11-02 23:41:28 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2009-11-04 14:06:24 +0100 |
| commit | 4b7673a04a16f1d8faf1e367ae28a6ee1671843d (patch) | |
| tree | 3abd280daad1126007fa7d75d0b69b86ea3b8d60 | |
| parent | trivial: remove duplicated MIN macro from tehuti. (diff) | |
| download | linux-4b7673a04a16f1d8faf1e367ae28a6ee1671843d.tar.xz linux-4b7673a04a16f1d8faf1e367ae28a6ee1671843d.zip | |
Phonet: remove tautologies
These checks don't make sense anymore since rtnl_notify() cannot fail.
Signed-off-by: Rémi Denis-Courmont <remi.denis-courmont@nokia.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | net/phonet/pn_netlink.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/net/phonet/pn_netlink.c b/net/phonet/pn_netlink.c index d8f5d3fb9ee2..609e509b369b 100644 --- a/net/phonet/pn_netlink.c +++ b/net/phonet/pn_netlink.c @@ -53,8 +53,7 @@ void phonet_address_notify(int event, struct net_device *dev, u8 addr) RTNLGRP_PHONET_IFADDR, NULL, GFP_KERNEL); return; errout: - if (err < 0) - rtnl_set_sk_err(dev_net(dev), RTNLGRP_PHONET_IFADDR, err); + rtnl_set_sk_err(dev_net(dev), RTNLGRP_PHONET_IFADDR, err); } static const struct nla_policy ifa_phonet_policy[IFA_MAX+1] = { @@ -212,8 +211,7 @@ void rtm_phonet_notify(int event, struct net_device *dev, u8 dst) RTNLGRP_PHONET_ROUTE, NULL, GFP_KERNEL); return; errout: - if (err < 0) - rtnl_set_sk_err(dev_net(dev), RTNLGRP_PHONET_ROUTE, err); + rtnl_set_sk_err(dev_net(dev), RTNLGRP_PHONET_ROUTE, err); } static const struct nla_policy rtm_phonet_policy[RTA_MAX+1] = { |