diff options
| author |   Parav Pandit <parav@mellanox.com> | 2018-03-19 07:59:59 +0200 |
|---|---|---|
| committer |   Jason Gunthorpe <jgg@mellanox.com> | 2018-03-19 11:41:40 -0600 |
| commit | 6d5b2047fe62ed83a90f0ecdf1cc9b4ae6fcc974 (patch) | |
| tree | e5dc4fd30e77f805df720eedd2cec6d8c4728580 /drivers/infiniband/core/cache.c | |
| parent | RDMA/bnxt: Fix structure layout for bnxt_re_pd_resp (diff) | |
| download | linux-dev-6d5b2047fe62ed83a90f0ecdf1cc9b4ae6fcc974.tar.xz linux-dev-6d5b2047fe62ed83a90f0ecdf1cc9b4ae6fcc974.zip | |
IB/core: Use rdma_is_port_valid()
Use rdma_is_port_valid() which performs port validity check instead of
open coding the same check.
Signed-off-by: Parav Pandit <parav@mellanox.com>
Signed-off-by: Leon Romanovsky <leonro@mellanox.com>
Reviewed-by: Yuval Shaia <yuval.shaia@oracle.com>
Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
Diffstat (limited to '')
| -rw-r--r-- | drivers/infiniband/core/cache.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/infiniband/core/cache.c b/drivers/infiniband/core/cache.c index 31def0f2ac49..5b9416af825b 100644 --- a/drivers/infiniband/core/cache.c +++ b/drivers/infiniband/core/cache.c @@ -937,8 +937,7 @@ int ib_get_cached_subnet_prefix(struct ib_device *device, unsigned long flags; int p; - if (port_num < rdma_start_port(device) || - port_num > rdma_end_port(device)) + if (!rdma_is_port_valid(device, port_num)) return -EINVAL; p = port_num - rdma_start_port(device); @@ -1048,7 +1047,7 @@ int ib_get_cached_port_state(struct ib_device *device, unsigned long flags; int ret = 0; - if (port_num < rdma_start_port(device) || port_num > rdma_end_port(device)) + if (!rdma_is_port_valid(device, port_num)) return -EINVAL; read_lock_irqsave(&device->cache.lock, flags); |