OSDN Git Service

scsi: pm80xx: Initialize variable used as return status
authorJohn Sperbeck <jsperbeck@google.com>
Thu, 14 Nov 2019 10:09:00 +0000 (15:39 +0530)
committerMartin K. Petersen <martin.petersen@oracle.com>
Wed, 20 Nov 2019 02:37:33 +0000 (21:37 -0500)
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.

Link: https://lore.kernel.org/r/20191114100910.6153-4-deepak.ukey@microchip.com
Acked-by: Jack Wang <jinpu.wang@cloud.ionos.com>
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>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/pm8001/pm8001_sas.c

index 7e48154..81160e9 100644 (file)
@@ -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) {