Messages in this thread | | | Date | Wed, 06 Aug 2014 09:21:26 +0200 | From | Hannes Reinecke <> | Subject | Re: [PATCH 10/15] scsi: fix decimal printf format specifiers prefixed with 0x |
| |
On 08/06/2014 06:43 AM, Hans Wennborg wrote: > The prefix suggests the number should be printed in hex, so use > the %x specifier to do that. > > Found by using regex suggested by Joe Perches. > > Signed-off-by: Hans Wennborg <hans@hanshq.net> > --- > drivers/scsi/cxgbi/libcxgbi.c | 2 +- > drivers/scsi/nsp32.c | 2 +- > drivers/scsi/qla2xxx/qla_target.c | 2 +- > 3 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/scsi/cxgbi/libcxgbi.c b/drivers/scsi/cxgbi/libcxgbi.c > index d65df6d..4cc9787 100644 > --- a/drivers/scsi/cxgbi/libcxgbi.c > +++ b/drivers/scsi/cxgbi/libcxgbi.c > @@ -1807,7 +1807,7 @@ static void csk_return_rx_credits(struct cxgbi_sock *csk, int copied) > u32 credits; > > log_debug(1 << CXGBI_DBG_PDU_RX, > - "csk 0x%p,%u,0x%lu,%u, seq %u, wup %u, thre %u, %u.\n", > + "csk 0x%p,%u,0x%lx,%u, seq %u, wup %u, thre %u, %u.\n", > csk, csk->state, csk->flags, csk->tid, csk->copied_seq, > csk->rcv_wup, cdev->rx_credit_thres, > cdev->rcv_win); > diff --git a/drivers/scsi/nsp32.c b/drivers/scsi/nsp32.c > index 50b086a..198f754 100644 > --- a/drivers/scsi/nsp32.c > +++ b/drivers/scsi/nsp32.c > @@ -915,7 +915,7 @@ static int nsp32_queuecommand_lck(struct scsi_cmnd *SCpnt, void (*done)(struct s > int ret; > > nsp32_dbg(NSP32_DEBUG_QUEUECOMMAND, > - "enter. target: 0x%x LUN: 0x%llu cmnd: 0x%x cmndlen: 0x%x " > + "enter. target: 0x%x LUN: 0x%llx cmnd: 0x%x cmndlen: 0x%x " > "use_sg: 0x%x reqbuf: 0x%lx reqlen: 0x%x", > SCpnt->device->id, SCpnt->device->lun, SCpnt->cmnd[0], SCpnt->cmd_len, > scsi_sg_count(SCpnt), scsi_sglist(SCpnt), scsi_bufflen(SCpnt)); > diff --git a/drivers/scsi/qla2xxx/qla_target.c b/drivers/scsi/qla2xxx/qla_target.c > index e632e14..aebe62c 100644 > --- a/drivers/scsi/qla2xxx/qla_target.c > +++ b/drivers/scsi/qla2xxx/qla_target.c > @@ -2503,7 +2503,7 @@ qlt_handle_dif_error(struct scsi_qla_host *vha, struct qla_tgt_cmd *cmd, > "iocb(s) %p Returned STATUS.\n", sts); > > ql_dbg(ql_dbg_tgt, vha, 0xf075, > - "dif check TGT cdb 0x%x lba 0x%llu: [Actual|Expected] Ref Tag[0x%x|0x%x], App Tag [0x%x|0x%x], Guard [0x%x|0x%x]\n", > + "dif check TGT cdb 0x%x lba 0x%llx: [Actual|Expected] Ref Tag[0x%x|0x%x], App Tag [0x%x|0x%x], Guard [0x%x|0x%x]\n", > cmd->atio.u.isp24.fcp_cmnd.cdb[0], lba, > a_ref_tag, e_ref_tag, a_app_tag, e_app_tag, a_guard, e_guard); > > Reviewed-by: Hannes Reinecke <hare@suse.de>
Cheers,
Hannes -- Dr. Hannes Reinecke zSeries & Storage hare@suse.de +49 911 74053 688 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: J. Hawn, J. Guild, F. Imendörffer, HRB 16746 (AG Nürnberg) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |