On Sat, 2019-08-03 at 16:45 +0800, Lijun Ou wrote:
> Here are some updates for hns driver based 5.3-rc2, mainly
> include some codes optimization and comments style modification.
>
> Change from V2:
> 1. Remove the unncessary memset opertion for the tenth patch
>
> Change from V1:
> 1. Fix the checkpatch warning
> 2. Use ibdev print interface instead of dev print interface in
> this patchset.
I need you to separate the ibdev changes from other changes. I have
other comments on the patches that I'll make on the individual patches,
but just in general, do one single patch to switch to using ibdev prints
and have it cover the entire driver. You can make it the first or last
patch, I don't care. But don't mix anything else in with the ibdev
transition patch. You shouldn't be mixing things like fixing an
incorrect print with a switch to ibdev print in the same patch because
it makes it very difficult on people that might be backporting these
patches to take the fix if they haven't also taken the ibdev print
patchset.
>
> Lang Cheng (6):
> RDMA/hns: Clean up unnecessary initial assignment
> RDMA/hns: Update some comments style
> RDMA/hns: Handling the error return value of hem function
> RDMA/hns: Split bool statement and assign statement
> RDMA/hns: Refactor irq request code
> RDMA/hns: Remove unnecessary kzalloc
>
> Lijun Ou (2):
> RDMA/hns: Encapsulate some lines for setting sq size in user mode
> RDMA/hns: Optimize hns_roce_modify_qp function
>
> Weihang Li (2):
> RDMA/hns: Remove redundant print in hns_roce_v2_ceq_int()
> RDMA/hns: Disable alw_lcl_lpbk of SSU
>
> Yangyang Li (1):
> RDMA/hns: Refactor hns_roce_v2_set_hem for hip08
>
> Yixian Liu (2):
> RDMA/hns: Update the prompt message for creating and destroy qp
> RDMA/hns: Remove unnessary init for cmq reg
>
> drivers/infiniband/hw/hns/hns_roce_device.h | 65 +++++----
> drivers/infiniband/hw/hns/hns_roce_hem.c | 15 +-
> drivers/infiniband/hw/hns/hns_roce_hem.h | 6 +-
> drivers/infiniband/hw/hns/hns_roce_hw_v2.c | 211 ++++++++++++++-----
> ---------
> drivers/infiniband/hw/hns/hns_roce_hw_v2.h | 2 -
> drivers/infiniband/hw/hns/hns_roce_mr.c | 1 -
> drivers/infiniband/hw/hns/hns_roce_qp.c | 178 ++++++++++++++-----
> ----
> 7 files changed, 260 insertions(+), 218 deletions(-)
>
--
Doug Ledford <dledford@xxxxxxxxxx>
GPG KeyID: B826A3330E572FDD
Fingerprint = AE6B 1BDA 122B 23B4 265B 1274 B826 A333 0E57 2FDD
Attachment:
signature.asc
Description: This is a digitally signed message part
