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 /include/linux/mlx5/device.h | |
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 'include/linux/mlx5/device.h')
-rw-r--r-- | include/linux/mlx5/device.h | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/include/linux/mlx5/device.h b/include/linux/mlx5/device.h index 3406cfb1267a..334947151dfc 100644 --- a/include/linux/mlx5/device.h +++ b/include/linux/mlx5/device.h | |||
@@ -456,9 +456,6 @@ struct mlx5_eqe_cq_err { | |||
456 | u8 syndrome; | 456 | u8 syndrome; |
457 | }; | 457 | }; |
458 | 458 | ||
459 | struct mlx5_eqe_dropped_packet { | ||
460 | }; | ||
461 | |||
462 | struct mlx5_eqe_port_state { | 459 | struct mlx5_eqe_port_state { |
463 | u8 reserved0[8]; | 460 | u8 reserved0[8]; |
464 | u8 port; | 461 | u8 port; |
@@ -498,7 +495,6 @@ union ev_data { | |||
498 | struct mlx5_eqe_comp comp; | 495 | struct mlx5_eqe_comp comp; |
499 | struct mlx5_eqe_qp_srq qp_srq; | 496 | struct mlx5_eqe_qp_srq qp_srq; |
500 | struct mlx5_eqe_cq_err cq_err; | 497 | struct mlx5_eqe_cq_err cq_err; |
501 | struct mlx5_eqe_dropped_packet dp; | ||
502 | struct mlx5_eqe_port_state port; | 498 | struct mlx5_eqe_port_state port; |
503 | struct mlx5_eqe_gpio gpio; | 499 | struct mlx5_eqe_gpio gpio; |
504 | struct mlx5_eqe_congestion cong; | 500 | struct mlx5_eqe_congestion cong; |