summaryrefslogtreecommitdiff
path: root/include
diff options
context:
space:
mode:
authorMoshe Lazer <moshel@mellanox.com>2013-07-10 14:31:03 +0300
committerRoland Dreier <roland@purestorage.com>2013-07-11 16:45:08 -0700
commit288dde9f23b6726c1e8147bf635721372bf77b16 (patch)
tree93867e6ea58c82b182c073b1b17e1e4a12b65762 /include
parent582c016e68dc5dfea4d3582512157f165a428149 (diff)
mlx5_core: Adjust hca_cap.uar_page_sz to conform to Connect-IB spec
Sparse reported an endianness bug in the assignment to hca_cap.uar_page_sz. Fix the declaration of this field to be __be16 (which is what is in the firmware spec), renaming the field to log_uar_pg_size to conform to the spec, which fixes the endianness bug reported by sparse. Reported-by: Fengguang Wu <fengguang.wu@intel.com> Signed-off-by: Moshe Lazer <moshel@mellanox.com> Signed-off-by: Or Gerlitz <ogerlitz@mellanox.com> Signed-off-by: Roland Dreier <roland@purestorage.com>
Diffstat (limited to 'include')
-rw-r--r--include/linux/mlx5/device.h4
1 files changed, 2 insertions, 2 deletions
diff --git a/include/linux/mlx5/device.h b/include/linux/mlx5/device.h
index 51390915e53..8de8d8f2238 100644
--- a/include/linux/mlx5/device.h
+++ b/include/linux/mlx5/device.h
@@ -317,8 +317,8 @@ struct mlx5_hca_cap {
u8 log_max_pd;
u8 rsvd25;
u8 log_max_xrcd;
- u8 rsvd26[40];
- __be32 uar_page_sz;
+ u8 rsvd26[42];
+ __be16 log_uar_page_sz;
u8 rsvd27[28];
u8 log_msx_atomic_size_qp;
u8 rsvd28[2];