Re: [PATCH for-next] RDMA/hns: Remove unused variables and definitions

[liweihang <liweihang@huawei.com>]

On 2020/9/28 19:55, Jason Gunthorpe wrote:
> On Sun, Sep 27, 2020 at 05:52:21PM +0800, Weihang Li wrote:
>> From: Lang Cheng <chenglang@huawei.com>
>>
>> Some code was removed but the variables were still there, and some
>> parameters have been changed to be queried from firmware. So the
>> definitions of them are no longer needed.
>>
>> Signed-off-by: Lang Cheng <chenglang@huawei.com>
>> Signed-off-by: Weihang Li <liweihang@huawei.com>
>> ---
>>  drivers/infiniband/hw/hns/hns_roce_device.h | 8 --------
>>  drivers/infiniband/hw/hns/hns_roce_qp.c     | 2 --
>>  2 files changed, 10 deletions(-)
> 
> Should have a fixes for the patch that removed the code
> 
> Jason
> 

Hi Jason,

Thanks for the comment. But I'm confused about when we should add
fixes tag.

For example, The only purpose of this patch is to remove redundant
macro definitions, the macros to be removed belong to 4 different
former patches, so we have to add 4 lines of fixes. It seems difficult
to merge this one back to previous versions of kernel.

Should I split this patch into 4 patches and add a fixes tag for each
one, or just put all these fixes tag in this patch?

Thanks
Weihang