]> git.proxmox.com Git - mirror_ubuntu-jammy-kernel.git/commitdiff
scsi: qla2xxx: Make qla_set_ini_mode() return void
authorJason Yan <yanaijie@huawei.com>
Wed, 29 Apr 2020 14:09:52 +0000 (22:09 +0800)
committerMartin K. Petersen <martin.petersen@oracle.com>
Fri, 8 May 2020 01:54:42 +0000 (21:54 -0400)
The return value is not used by the caller and the local variable 'rc' is
not needed. Make qla_set_ini_mode() return void and remove 'rc'.  This also
fixes the following coccicheck warning:

drivers/scsi/qla2xxx/qla_attr.c:1906:5-7: Unneeded variable: "rc".
Return "0" on line 2180

Link: https://lore.kernel.org/r/20200429140952.8240-1-yanaijie@huawei.com
Signed-off-by: Jason Yan <yanaijie@huawei.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/qla2xxx/qla_attr.c

index 3a5f6f27587efe7276ce9168d433d7bd81213095..e83e44ef7083f55a2f0eb7e80275aa5b5cedff35 100644 (file)
@@ -1925,9 +1925,8 @@ static char *mode_to_str[] = {
 };
 
 #define NEED_EXCH_OFFLOAD(_exchg) ((_exchg) > FW_DEF_EXCHANGES_CNT)
-static int qla_set_ini_mode(scsi_qla_host_t *vha, int op)
+static void qla_set_ini_mode(scsi_qla_host_t *vha, int op)
 {
-       int rc = 0;
        enum {
                NO_ACTION,
                MODE_CHANGE_ACCEPT,
@@ -2200,8 +2199,6 @@ static int qla_set_ini_mode(scsi_qla_host_t *vha, int op)
                    vha->ql2xexchoffld, vha->u_ql2xexchoffld);
                break;
        }
-
-       return rc;
 }
 
 static ssize_t