Fixes the following W=1 kernel build warning(s):

 drivers/scsi/myrs.c: In function ‘consistency_check_show’:
 drivers/scsi/myrs.c:1193:16: warning: variable ‘status’ set but not used 
[-Wunused-but-set-variable]
 drivers/scsi/myrs.c: In function ‘myrs_get_resync’:
 drivers/scsi/myrs.c:1984:5: warning: variable ‘status’ set but not used 
[-Wunused-but-set-variable]

Cc: Hannes Reinecke <h...@kernel.org>
Cc: "James E.J. Bottomley" <j...@linux.ibm.com>
Cc: "Martin K. Petersen" <martin.peter...@oracle.com>
Cc: Linux GmbH <h...@suse.com>
Cc: "Leonard N. Zubkoff" <l...@dandelion.com>
Cc: linux-s...@vger.kernel.org
Signed-off-by: Lee Jones <lee.jo...@linaro.org>
---
 drivers/scsi/myrs.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/scsi/myrs.c b/drivers/scsi/myrs.c
index 4adf9ded296aa..48e399f057d5c 100644
--- a/drivers/scsi/myrs.c
+++ b/drivers/scsi/myrs.c
@@ -1190,7 +1190,6 @@ static ssize_t consistency_check_show(struct device *dev,
        struct myrs_hba *cs = shost_priv(sdev->host);
        struct myrs_ldev_info *ldev_info;
        unsigned short ldev_num;
-       unsigned char status;
 
        if (sdev->channel < cs->ctlr_info->physchan_present)
                return snprintf(buf, 32, "physical device - not checking\n");
@@ -1199,7 +1198,7 @@ static ssize_t consistency_check_show(struct device *dev,
        if (!ldev_info)
                return -ENXIO;
        ldev_num = ldev_info->ldev_num;
-       status = myrs_get_ldev_info(cs, ldev_num, ldev_info);
+       myrs_get_ldev_info(cs, ldev_num, ldev_info);
        if (ldev_info->cc_active)
                return snprintf(buf, 32, "checking block %zu of %zu\n",
                                (size_t)ldev_info->cc_lba,
@@ -1981,14 +1980,13 @@ myrs_get_resync(struct device *dev)
        struct myrs_hba *cs = shost_priv(sdev->host);
        struct myrs_ldev_info *ldev_info = sdev->hostdata;
        u64 percent_complete = 0;
-       u8 status;
 
        if (sdev->channel < cs->ctlr_info->physchan_present || !ldev_info)
                return;
        if (ldev_info->rbld_active) {
                unsigned short ldev_num = ldev_info->ldev_num;
 
-               status = myrs_get_ldev_info(cs, ldev_num, ldev_info);
+               myrs_get_ldev_info(cs, ldev_num, ldev_info);
                percent_complete = ldev_info->rbld_lba * 100;
                do_div(percent_complete, ldev_info->cfg_devsize);
        }
-- 
2.27.0

Reply via email to