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