[for-next,0/5] RDMA/hns: Cleanup for removing redundant statements and judging return values

Message

Wenpeng Liang April 9, 2022, 8:32 a.m. UTC

Most static warnings are detected by tools, mainly about:

(1) #1~3: Remove redundant statements.
(2) #4~5: About the return value of the function.

Chengchang Tang (1):
  RDMA/hns: Remove unnecessary check for the sgid_attr when modifying QP

Guofeng Yue (1):
  RDMA/hns: Remove redundant variable "ret"

Haoyue Xu (1):
  RDMA/hns: Init the variable at the suitable place

Wenpeng Liang (1):
  RDMA/hns: Add judgment on the execution result of CMDQ that free vf
    resource

Yixing Liu (1):
  RDMA/hns: Remove unused function to_hns_roce_state()

 drivers/infiniband/hw/hns/hns_roce_device.h | 11 --------
 drivers/infiniband/hw/hns/hns_roce_hw_v2.c  | 29 ++++++++++++---------
 drivers/infiniband/hw/hns/hns_roce_qp.c     | 20 --------------
 3 files changed, 17 insertions(+), 43 deletions(-)

--
2.33.0

Comments

Jason Gunthorpe April 12, 2022, 1:42 p.m. UTC | #1

On Sat, Apr 09, 2022 at 04:32:49PM +0800, Wenpeng Liang wrote:
> Most static warnings are detected by tools, mainly about:
> 
> (1) #1~3: Remove redundant statements.
> (2) #4~5: About the return value of the function.
> 
> Chengchang Tang (1):
>   RDMA/hns: Remove unnecessary check for the sgid_attr when modifying QP
> 
> Guofeng Yue (1):
>   RDMA/hns: Remove redundant variable "ret"
> 
> Haoyue Xu (1):
>   RDMA/hns: Init the variable at the suitable place
> 
> Wenpeng Liang (1):
>   RDMA/hns: Add judgment on the execution result of CMDQ that free vf
>     resource
> 
> Yixing Liu (1):
>   RDMA/hns: Remove unused function to_hns_roce_state()
> 
>  drivers/infiniband/hw/hns/hns_roce_device.h | 11 --------
>  drivers/infiniband/hw/hns/hns_roce_hw_v2.c  | 29 ++++++++++++---------
>  drivers/infiniband/hw/hns/hns_roce_qp.c     | 20 --------------
>  3 files changed, 17 insertions(+), 43 deletions(-)

Applied to for-next, thanks

Jason