mbox series

[v2,for-next,0/6] RDMA/hns: Codes optimization

Message ID 1586938475-37049-1-git-send-email-liweihang@huawei.com (mailing list archive)
Headers show
Series RDMA/hns: Codes optimization | expand

Message

Weihang Li April 15, 2020, 8:14 a.m. UTC
This series optimize some codes in hns drivers. The first two patches are
mainly to remove unnecessary memset(), and the others use map table to
simplify the conversion of values.

Previous version can be found at:
https://patchwork.kernel.org/cover/11485099/

Changes since v1:
- Fix comments from Jason that the arrays should be defined in type of
  "static const" in patch #3 ~ #6.

Lang Cheng (4):
  RDMA/hns: Simplify the qp state convert code
  RDMA/hns: Simplify the cqe code of poll cq
  RDMA/hns: Simplify the state judgment code of qp
  RDMA/hns: Simplify the status judgment code of hns_roce_v1_m_qp()

Lijun Ou (2):
  RDMA/hns: Optimize hns_roce_config_link_table()
  RDMA/hns: Optimize hns_roce_v2_set_mac()

 drivers/infiniband/hw/hns/hns_roce_hw_v1.c |  42 +++--
 drivers/infiniband/hw/hns/hns_roce_hw_v2.c | 269 +++++++++++++----------------
 2 files changed, 150 insertions(+), 161 deletions(-)

Comments

Jason Gunthorpe April 24, 2020, 1:24 p.m. UTC | #1
On Wed, Apr 15, 2020 at 04:14:29PM +0800, Weihang Li wrote:
> This series optimize some codes in hns drivers. The first two patches are
> mainly to remove unnecessary memset(), and the others use map table to
> simplify the conversion of values.
> 
> Previous version can be found at:
> https://patchwork.kernel.org/cover/11485099/
> 
> Changes since v1:
> - Fix comments from Jason that the arrays should be defined in type of
>   "static const" in patch #3 ~ #6.
> 
> Lang Cheng (4):
>   RDMA/hns: Simplify the qp state convert code
>   RDMA/hns: Simplify the cqe code of poll cq
>   RDMA/hns: Simplify the state judgment code of qp
>   RDMA/hns: Simplify the status judgment code of hns_roce_v1_m_qp()
> 
> Lijun Ou (2):
>   RDMA/hns: Optimize hns_roce_config_link_table()
>   RDMA/hns: Optimize hns_roce_v2_set_mac()

applied to for-next, thanks

Jason