Re: [PATCH v2 for-next 0/9] RDMA/hns: Cleanups for hip08

Weihang Li
2019-11-09T02:18:32Z

On 2019/11/9 4:45, Jason Gunthorpe wrote:
> On Tue, Nov 05, 2019 at 07:07:53PM +0800, Weihang Li wrote:
>> These series just make cleanups without changing code logic.
>>
>> [patch 1/9 ~ 3/9] remove unused variables and structures.
>> [patch 4/9 ~ 5/9] modify field and function names.
>> [patch 6/9 ~ 7/9] remove dead codes to simplify functions.
>> [patch 8/9] replaces non-standard return value with linux error codes.
>> [patch 9/9] does some fixes on printings.
>>
>> Changelog
>> v1->v2: Remove "{topost}" in titles.
>>
>> Lang Cheng (3):
>> RDMA/hns: Remove unnecessary structure hns_roce_sqp
>> RDMA/hns: Simplify doorbell initialization code
>> RDMA/hns: Modify hns_roce_hw_v2_get_cfg to simplify the code
>>
>> Wenpeng Liang (1):
>> RDMA/hns: Modify appropriate printings
>>
>> Yixian Liu (4):
>> RDMA/hns: Delete unnecessary variable max_post
>> RDMA/hns: Delete unnecessary uar from hns_roce_cq
>> RDMA/hns: Modify fields of struct hns_roce_srq
>> RDMA/hns: Fix non-standard error codes
>>
>> Yixing Liu (1):
>> RDMA/hns: Replace not intuitive function/macro names
>
> Something should probably be done about that custom bitmap stuff Leon
> noted, but that isn't related to these patches, so applied to for-next
>

Thank you, Jason. We will continue to pay attention to this bitmap stuff
and discuss further with Leon.

Weihang

> Thanks,
> Jason
>
>