On 2020-04-03 01:40, Nilesh Javali wrote: > @@ -2547,6 +2546,7 @@ ql_dbg(uint level, scsi_qla_host_t *vha, uint id, const char *fmt, ...) > vaf.va = &va; > > if (!ql_mask_match(level)) { > + char pbuf[64]; > if (vha != NULL) { > const struct pci_dev *pdev = vha->hw->pdev; > /* <module-name> <msg-id>:<host> Message */ Has this patch been verified witch checkpatch? Did checkpatch report "Missing a blank line after declarations"? Thanks, Bart.