Re: [PATCH -next] scsi: libsas: simplify the return expression of sas_discover_end_dev

From: John Garry
Date: Mon Sep 21 2020 - 09:06:30 EST


On 21/09/2020 09:24, Liu Shixin wrote:
Simplify the return expression.

Signed-off-by: Liu Shixin <liushixin2@xxxxxxxxxx>
---
drivers/scsi/libsas/sas_discover.c | 8 +-------
1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/drivers/scsi/libsas/sas_discover.c b/drivers/scsi/libsas/sas_discover.c
index d0f9e90e3279..161c9b387da7 100644
--- a/drivers/scsi/libsas/sas_discover.c
+++ b/drivers/scsi/libsas/sas_discover.c
@@ -278,13 +278,7 @@ static void sas_resume_devices(struct work_struct *work)
*/
int sas_discover_end_dev(struct domain_device *dev)
{
- int res;
-
- res = sas_notify_lldd_dev_found(dev);
- if (res)
- return res;
-
- return 0;
+ return sas_notify_lldd_dev_found(dev);
}
/* ---------- Device registration and unregistration ---------- */


You can make a similar change at the end of sas_discover_data() [and include in the same patch]