diff options
| author |   Zhu Yanjun <yanjun.zhu@oracle.com> | 2017-01-05 03:56:08 -0500 |
|---|---|---|
| committer |   Doug Ledford <dledford@redhat.com> | 2017-01-24 16:20:37 -0500 |
| commit | f7534f45dcbc1a2ecb486b019db0443188e957d6 (patch) | |
| tree | e9bd287abadd5bf983cf261d6fff6a6eb0e35f22 /drivers/infiniband/ulp/ipoib/ipoib_vlan.c | |
| parent | IB/isert: fix spelling mistake: "teminating" -> "terminating" (diff) | |
| download | linux-dev-f7534f45dcbc1a2ecb486b019db0443188e957d6.tar.xz linux-dev-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_vlan.c')
| -rw-r--r-- | drivers/infiniband/ulp/ipoib/ipoib_vlan.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/infiniband/ulp/ipoib/ipoib_vlan.c b/drivers/infiniband/ulp/ipoib/ipoib_vlan.c index d9dab4a109f7..deedb6fc1b05 100644 --- a/drivers/infiniband/ulp/ipoib/ipoib_vlan.c +++ b/drivers/infiniband/ulp/ipoib/ipoib_vlan.c @@ -61,9 +61,7 @@ int __ipoib_vlan_add(struct ipoib_dev_priv *ppriv, struct ipoib_dev_priv *priv, priv->parent = ppriv->dev; set_bit(IPOIB_FLAG_SUBINTERFACE, &priv->flags); - result = ipoib_set_dev_features(priv, ppriv->ca); - if (result) - goto err; + ipoib_set_dev_features(priv, ppriv->ca); priv->pkey = pkey; |