Message ID | 20191031051241.6762-3-deepak.ukey@microchip.com (mailing list archive) |
---|---|
State | Superseded |
Headers | show |
Series | pm80xx : Updates for the driver version 0.1.39. | expand |
On Thu, Oct 31, 2019 at 6:12 AM Deepak Ukey <deepak.ukey@microchip.com> wrote: > > From: John Sperbeck <jsperbeck@google.com> > > In pm8001_task_exec(), if the PHY is down, then we return the > current value of 'rc'. We need to make sure it's initialized. > > Signed-off-by: John Sperbeck <jsperbeck@google.com> > Signed-off-by: Deepak Ukey <deepak.ukey@microchip.com> > Signed-off-by: Viswas G <Viswas.G@microchip.com> Looks good, thanks. Acked-by: Jack Wang <jinpu.wang@cloud.ionos.com> > --- > drivers/scsi/pm8001/pm8001_sas.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/scsi/pm8001/pm8001_sas.c b/drivers/scsi/pm8001/pm8001_sas.c > index 7e48154e11c3..81160e99c75e 100644 > --- a/drivers/scsi/pm8001/pm8001_sas.c > +++ b/drivers/scsi/pm8001/pm8001_sas.c > @@ -384,7 +384,7 @@ static int pm8001_task_exec(struct sas_task *task, > struct pm8001_port *port = NULL; > struct sas_task *t = task; > struct pm8001_ccb_info *ccb; > - u32 tag = 0xdeadbeef, rc, n_elem = 0; > + u32 tag = 0xdeadbeef, rc = 0, n_elem = 0; > unsigned long flags = 0; > > if (!dev->port) { > -- > 2.16.3 >
diff --git a/drivers/scsi/pm8001/pm8001_sas.c b/drivers/scsi/pm8001/pm8001_sas.c index 7e48154e11c3..81160e99c75e 100644 --- a/drivers/scsi/pm8001/pm8001_sas.c +++ b/drivers/scsi/pm8001/pm8001_sas.c @@ -384,7 +384,7 @@ static int pm8001_task_exec(struct sas_task *task, struct pm8001_port *port = NULL; struct sas_task *t = task; struct pm8001_ccb_info *ccb; - u32 tag = 0xdeadbeef, rc, n_elem = 0; + u32 tag = 0xdeadbeef, rc = 0, n_elem = 0; unsigned long flags = 0; if (!dev->port) {