> On Sep 29, 2020, at 9:25 PM, Ye Bin <yebin10@xxxxxxxxxx> wrote: > > Fix follow warnings: > [drivers/scsi/qla2xxx/tcm_qla2xxx.c:884]: (warning) %u in format string (no. 1) > requires 'unsigned int' but the argument type is 'signed int'. > [drivers/scsi/qla2xxx/tcm_qla2xxx.c:885]: (warning) %u in format string (no. 1) > requires 'unsigned int' but the argument type is 'signed int'. > [drivers/scsi/qla2xxx/tcm_qla2xxx.c:886]: (warning) %u in format string (no. 1) > requires 'unsigned int' but the argument type is 'signed int'. > [drivers/scsi/qla2xxx/tcm_qla2xxx.c:887]: (warning) %u in format string (no. 1) > requires 'unsigned int' but the argument type is 'signed int'. > [drivers/scsi/qla2xxx/tcm_qla2xxx.c:888]: (warning) %u in format string (no. 1) > requires 'unsigned int' but the argument type is 'signed int'. > > Reported-by: Hulk Robot <hulkci@xxxxxxxxxx> > Signed-off-by: Ye Bin <yebin10@xxxxxxxxxx> > --- > drivers/scsi/qla2xxx/tcm_qla2xxx.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/scsi/qla2xxx/tcm_qla2xxx.c b/drivers/scsi/qla2xxx/tcm_qla2xxx.c > index 44bfe162654a..61017acd3458 100644 > --- a/drivers/scsi/qla2xxx/tcm_qla2xxx.c > +++ b/drivers/scsi/qla2xxx/tcm_qla2xxx.c > @@ -850,7 +850,7 @@ static ssize_t tcm_qla2xxx_tpg_attrib_##name##_show( \ > struct tcm_qla2xxx_tpg *tpg = container_of(se_tpg, \ > struct tcm_qla2xxx_tpg, se_tpg); \ > \ > - return sprintf(page, "%u\n", tpg->tpg_attrib.name); \ > + return sprintf(page, "%d\n", tpg->tpg_attrib.name); \ > } \ > \ > static ssize_t tcm_qla2xxx_tpg_attrib_##name##_store( \ > -- > 2.25.4 > Looks good. Reviewed-by: Himanshu Madhani <himanshu.madhani@xxxxxxxxxx> -- Himanshu Madhani Oracle Linux Engineering
