diff options
| author | Zhu Yanjun <yanjun.zhu@oracle.com> | 2017-01-05 09:56:08 +0100 |
|---|---|---|
| committer | Doug Ledford <dledford@redhat.com> | 2017-01-24 22:20:37 +0100 |
| commit | f7534f45dcbc1a2ecb486b019db0443188e957d6 (patch) | |
| tree | e9bd287abadd5bf983cf261d6fff6a6eb0e35f22 /drivers/infiniband/ulp/ipoib/ipoib.h | |
| parent | IB/isert: fix spelling mistake: "teminating" -> "terminating" (diff) | |
| download | linux-f7534f45dcbc1a2ecb486b019db0443188e957d6.tar.xz linux-f7534f45dcbc1a2ecb486b019db0443188e957d6.zip | |
IB/ipoib: Remove unnecessary returned value check
In the function ipoib_set_dev_features, the returned value is always 0.
As such, it is not necessary to check the returned value.
This is not a bug. It is a trivial problem.
Reviewed-by: Guanglei Li <guanglei.li@oracle.com>
Signed-off-by: Zhu Yanjun <yanjun.zhu@oracle.com>
Reviewed-by: Yuval Shaia <yuval.shaia@oracle.com>
Reviewed-by: Leon Romanovsky <leonro@mellanox.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
Diffstat (limited to 'drivers/infiniband/ulp/ipoib/ipoib.h')
| -rw-r--r-- | drivers/infiniband/ulp/ipoib/ipoib.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/infiniband/ulp/ipoib/ipoib.h b/drivers/infiniband/ulp/ipoib/ipoib.h index da12717a3eb7..f5680642960a 100644 --- a/drivers/infiniband/ulp/ipoib/ipoib.h +++ b/drivers/infiniband/ulp/ipoib/ipoib.h @@ -593,7 +593,7 @@ void ipoib_pkey_open(struct ipoib_dev_priv *priv); void ipoib_drain_cq(struct net_device *dev); void ipoib_set_ethtool_ops(struct net_device *dev); -int ipoib_set_dev_features(struct ipoib_dev_priv *priv, struct ib_device *hca); +void ipoib_set_dev_features(struct ipoib_dev_priv *priv, struct ib_device *hca); #define IPOIB_FLAGS_RC 0x80 #define IPOIB_FLAGS_UC 0x40 |