Re: [PATCH 5/6] qla2xxx_nvmet: Add SysFS node for FC-NVMe Target

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Bart, 

> On Sep 15, 2018, at 9:57 PM, Bart Van Assche <bvanassche@xxxxxxx> wrote:
> 
> External Email
> 
> On 09/14/18 14:28, Himanshu Madhani wrote:
>> @@ -686,7 +686,7 @@ static void qla_nvmet_send_resp_ctio(struct qla_qpair *qpair,
>>                      ctio->u.nvme_status_mode1.transfer_len =
>>                                              cpu_to_be32(ersp->xfrd_len);
>> 
>> -                     ql_log(ql_log_info, vha, 0x1100f,
>> +                      ql_dbg(ql_dbg_nvme, vha, 0x1100f,
>>                              "op: %#x, rsplen: %#x\n", rsp_buf->op,
>>                              rsp_buf->rsplen);
>>              } else
> 
> Have you verified this patch series with checkpatch? I think the above
> change introduces a whitespace violation.
> 
Looks like I missed responding to this question. 

Its odd that actual patch does not have whitespace violation
nor did checkpatch reported one.

I’ll fix it up during v2 submission 

> Bart.
> 

Thanks,
- Himanshu





[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]

  Powered by Linux