[<prev] [next>] [day] [month] [year] [list]
Message-Id: <20220905090037.335055-1-ye.xingchen@zte.com.cn>
Date: Mon, 5 Sep 2022 09:00:37 +0000
From: cgel.zte@...il.com
To: sathya.prakash@...adcom.com
Cc: sreekanth.reddy@...adcom.com,
suganath-prabu.subramani@...adcom.com,
MPT-FusionLinux.pdl@...adcom.com, linux-scsi@...r.kernel.org,
linux-kernel@...r.kernel.org, ye xingchen <ye.xingchen@....com.cn>,
Zeal Robot <zealci@....com.cn>
Subject: [PATCH linux-next] scsi: message: Remove the unneeded result variable
From: ye xingchen <ye.xingchen@....com.cn>
Return the value mptctl_do_mpt_command() directly instead of storing it in
another redundant variable.
Reported-by: Zeal Robot <zealci@....com.cn>
Signed-off-by: ye xingchen <ye.xingchen@....com.cn>
---
drivers/message/fusion/mptctl.c | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/drivers/message/fusion/mptctl.c b/drivers/message/fusion/mptctl.c
index f9ee957072c3..1e13c41fd58e 100644
--- a/drivers/message/fusion/mptctl.c
+++ b/drivers/message/fusion/mptctl.c
@@ -1702,8 +1702,6 @@ mptctl_mpt_command (MPT_ADAPTER *ioc, unsigned long arg)
{
struct mpt_ioctl_command __user *uarg = (void __user *) arg;
struct mpt_ioctl_command karg;
- int rc;
-
if (copy_from_user(&karg, uarg, sizeof(struct mpt_ioctl_command))) {
printk(KERN_ERR MYNAM "%s@%d::mptctl_mpt_command - "
@@ -1712,9 +1710,7 @@ mptctl_mpt_command (MPT_ADAPTER *ioc, unsigned long arg)
return -EFAULT;
}
- rc = mptctl_do_mpt_command (ioc, karg, &uarg->MF);
-
- return rc;
+ return mptctl_do_mpt_command(ioc, karg, &uarg->MF);
}
/*=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=*/
--
2.25.1
Powered by blists - more mailing lists