staging: rts5208: remove redundant assignment to retval
authorColin Ian King <colin.king@canonical.com>
Sun, 18 Aug 2019 18:46:49 +0000 (19:46 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 21 Aug 2019 12:48:06 +0000 (05:48 -0700)
Variable retval is initialized to a value that is never read and it
is re-assigned later. The initialization is redundant and can be
removed.

Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Link: https://lore.kernel.org/r/20190818184649.13828-1-colin.king@canonical.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rts5208/ms.c

index 1128eec..e853fa9 100644 (file)
@@ -3842,7 +3842,7 @@ int mg_set_leaf_id(struct scsi_cmnd *srb, struct rtsx_chip *chip)
 
 int mg_get_local_EKB(struct scsi_cmnd *srb, struct rtsx_chip *chip)
 {
-       int retval = STATUS_FAIL;
+       int retval;
        int bufflen;
        unsigned int lun = SCSI_LUN(srb);
        u8 *buf = NULL;