diff options
| author |   Colin Ian King <colin.king@canonical.com> | 2019-08-13 14:23:49 +0100 |
|---|---|---|
| committer |   Martin K. Petersen <martin.petersen@oracle.com> | 2019-08-19 21:51:00 -0400 |
| commit | 9730ddfb123db3f37ace7571176661257712c99b (patch) | |
| tree | 2f29f8ba00c3c2f198b8ae89eddb863133a468a8 /drivers/scsi/fnic/fnic_debugfs.c | |
| parent | scsi: qla2xxx: cleanup trace buffer initialization (diff) | |
| download | linux-dev-9730ddfb123db3f37ace7571176661257712c99b.tar.xz linux-dev-9730ddfb123db3f37ace7571176661257712c99b.zip | |
scsi: fnic: remove redundant assignment of variable rc
Variable ret is initialized to a value that is never read and it is
re-assigned later and immediately returns. Clean up the code by removing
rc and just returning 0.
[mkp: typo]
Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Acked-by: Karan Tilak Kumar <kartilak@cisco.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Diffstat (limited to '')
| -rw-r--r-- | drivers/scsi/fnic/fnic_debugfs.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/scsi/fnic/fnic_debugfs.c b/drivers/scsi/fnic/fnic_debugfs.c index 21991c99db7c..13f7d88d6e57 100644 --- a/drivers/scsi/fnic/fnic_debugfs.c +++ b/drivers/scsi/fnic/fnic_debugfs.c @@ -52,7 +52,6 @@ static struct fc_trace_flag_type *fc_trc_flag; */ int fnic_debugfs_init(void) { - int rc = -1; fnic_trace_debugfs_root = debugfs_create_dir("fnic", NULL); fnic_stats_debugfs_root = debugfs_create_dir("statistics", @@ -70,8 +69,7 @@ int fnic_debugfs_init(void) fc_trc_flag->fc_clear = 4; } - rc = 0; - return rc; + return 0; } /* |