From: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx> Date: Fri, 21 Apr 2017 19:56:54 +0200 Several update suggestions were taken into account from static source code analysis. Markus Elfring (17): Use kcalloc() in mlx4_ib_alloc_pv_bufs() Use kmalloc_array() in six functions Improve size determinations in six functions Fix a typo in a comment line Delete four unnecessary return statements Delete an unnecessary check before kfree() in free_pv_object() Move an assignment out of a check in forward_trap() Enclose 46 expressions for sizeof by parentheses Split a condition check in six functions Delete an unnecessary variable in __mlx4_ib_query_gid() Delete an unnecessary variable initialisation in mlx4_ib_add() Delete an unnecessary variable assignment in mlx4_ib_add() Delete an error message for a failed memory allocation in mlx4_ib_add() Delete unnecessary braces in mlx4_ib_add() Delete unwanted spaces behind usages of the sizeof operator Add spaces for better code readability Less function calls in create_kernel_qp() after error detection --- v2: Changes were rebased on source files from Linux next-20170421. Some of them were recombined as requested by Doug Ledford. drivers/infiniband/hw/mlx4/mad.c | 72 ++++++++-------- drivers/infiniband/hw/mlx4/main.c | 164 ++++++++++++++++++++---------------- drivers/infiniband/hw/mlx4/qp.c | 173 +++++++++++++++++++------------------- drivers/infiniband/hw/mlx5/qp.c | 66 ++++++++++----- drivers/infiniband/hw/mlx5/srq.c | 5 +- 5 files changed, 261 insertions(+), 219 deletions(-) -- 2.12.2 -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html