On Fri, 2017-10-06 at 14:26 +0200, Bernard Metzler wrote:
> Sorry. Yes, done now, will resend!
Hello Bernard,
On Linux kernel mailing lists it is expected that replies are posted below the
original e-mail instead of above. Please follow that style.
Another convention you may not be aware of is that every cover letter for v2 and
later versions of a patch series is expected to include a changelog that explains
what changes have been made in each version.
Regarding checkpatch and the soft-iWarp driver, checkpatch still reports a large
number of complaints against v2 of which I think most are easy to address. A few
examples:
WARNING: 'vaild' may be misspelled - perhaps 'valid'?
#1281: FILE: drivers/infiniband/sw/siw/siw_verbs.c:1052:
+ /* make SQE only vaild after completely written */
ERROR: trailing whitespace
#2422: FILE: include/uapi/rdma/siw_user.h:133:
+enum siw_wqe_flags { $
Please also make yourself familiar with the static analyzer called 'sparse'. That
static analyzer reports a large number of complaints about the soft-iWarp driver,
complaints of which I think these should be addressed. A few examples:
$ make M=drivers/infiniband/sw/siw C=2
[ ... ]
drivers/infiniband/sw/siw/siw_cm.c:207:6: warning: symbol 'siw_sk_assign_rtr_upcalls' was not declared. Should it be static?
drivers/infiniband/sw/siw/siw_cm.c:464:23: warning: incorrect type in assignment (different base types)
drivers/infiniband/sw/siw/siw_cm.c:464:23: expected restricted __be32 [addressable] [usertype] term_ctrl
drivers/infiniband/sw/siw/siw_cm.c:464:23: got unsigned int
drivers/infiniband/sw/siw/siw_cm.c:466:25: warning: cast from restricted __be32
drivers/infiniband/sw/siw/siw_cm.c:466:25: warning: incorrect type in argument 1 (different base types)
drivers/infiniband/sw/siw/siw_cm.c:466:25: expected unsigned int [unsigned] [usertype] val
drivers/infiniband/sw/siw/siw_cm.c:466:25: got restricted __be32 [addressable] [usertype] term_ctrl
[ ... ]
Thanks,
Bart.��.n��������+%������w��{.n�����{���fk��ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f
