This patch seems to be good. Please consider this patch as Acked-by: "Sreekanth Reddy" <Sreekanth.Reddy@xxxxxxxxxxxxx> Regards, Sreekanth On Thu, Dec 4, 2014 at 4:26 PM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote: > The "issue_reset" variable can be used uninitialized. It should be set > to false at the start. > > Also I cleaned up the types by using bool. > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > --- > v2: style fix > > diff --git a/drivers/scsi/mpt2sas/mpt2sas_base.c b/drivers/scsi/mpt2sas/mpt2sas_base.c > index 58e4521..3fb01d1 100644 > --- a/drivers/scsi/mpt2sas/mpt2sas_base.c > +++ b/drivers/scsi/mpt2sas/mpt2sas_base.c > @@ -3236,7 +3236,7 @@ mpt2sas_base_sas_iounit_control(struct MPT2SAS_ADAPTER *ioc, > u16 smid; > u32 ioc_state; > unsigned long timeleft; > - u8 issue_reset; > + bool issue_reset = false; > int rc; > void *request; > u16 wait_state_count; > @@ -3300,7 +3300,7 @@ mpt2sas_base_sas_iounit_control(struct MPT2SAS_ADAPTER *ioc, > _debug_dump_mf(mpi_request, > sizeof(Mpi2SasIoUnitControlRequest_t)/4); > if (!(ioc->base_cmds.status & MPT2_CMD_RESET)) > - issue_reset = 1; > + issue_reset = true; > goto issue_host_reset; > } > if (ioc->base_cmds.status & MPT2_CMD_REPLY_VALID) > @@ -3341,7 +3341,7 @@ mpt2sas_base_scsi_enclosure_processor(struct MPT2SAS_ADAPTER *ioc, > u16 smid; > u32 ioc_state; > unsigned long timeleft; > - u8 issue_reset; > + bool issue_reset = false; > int rc; > void *request; > u16 wait_state_count; > @@ -3398,7 +3398,7 @@ mpt2sas_base_scsi_enclosure_processor(struct MPT2SAS_ADAPTER *ioc, > _debug_dump_mf(mpi_request, > sizeof(Mpi2SepRequest_t)/4); > if (!(ioc->base_cmds.status & MPT2_CMD_RESET)) > - issue_reset = 1; > + issue_reset = true; > goto issue_host_reset; > } > if (ioc->base_cmds.status & MPT2_CMD_REPLY_VALID) -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
