diff options
author | Jack Morgenstein <jackm@dev.mellanox.co.il> | 2014-07-28 16:30:23 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-07-30 17:00:06 -0400 |
commit | f241e7497ec2d22b83002b17ae91a851d4034cb7 (patch) | |
tree | 5d8179647f85be21bad567cdce578a78ca6de875 /drivers/infiniband/hw/mlx5/main.c | |
parent | 9603b61de1eee92977d74ff42541be20c0c5b1a7 (diff) |
mlx5: minor fixes (mainly avoidance of hidden casts)
There were many places where parameters which should be u8/u16 were
integer type.
Additionally, in 2 places, a check for a non-null pointer was added
before dereferencing the pointer (this is actually a bug fix).
Signed-off-by: Jack Morgenstein <jackm@dev.mellanox.co.il>
Signed-off-by: Eli Cohen <eli@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/infiniband/hw/mlx5/main.c')
-rw-r--r-- | drivers/infiniband/hw/mlx5/main.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/infiniband/hw/mlx5/main.c b/drivers/infiniband/hw/mlx5/main.c index f2cfd363a705..166335a95c59 100644 --- a/drivers/infiniband/hw/mlx5/main.c +++ b/drivers/infiniband/hw/mlx5/main.c | |||
@@ -478,7 +478,7 @@ static struct ib_ucontext *mlx5_ib_alloc_ucontext(struct ib_device *ibdev, | |||
478 | int uuarn; | 478 | int uuarn; |
479 | int err; | 479 | int err; |
480 | int i; | 480 | int i; |
481 | int reqlen; | 481 | size_t reqlen; |
482 | 482 | ||
483 | if (!dev->ib_active) | 483 | if (!dev->ib_active) |
484 | return ERR_PTR(-EAGAIN); | 484 | return ERR_PTR(-EAGAIN); |