summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorYue Hu <huyue2@yulong.com>2021-03-10 11:27:41 +0300
committerMartin K. Petersen <martin.petersen@oracle.com>2021-03-17 07:04:40 +0300
commit2a8561b78e377f48aa638d7cd203628b6dfc2519 (patch)
treede9f10bf1fca3953dc52a59aba8a80b9882393a7
parent4517e77eb83d875c61e11dec01dec4ad49e4c5e7 (diff)
downloadlinux-2a8561b78e377f48aa638d7cd203628b6dfc2519.tar.xz
scsi: ufs: core: Remove unnecessary ret in ufshcd_populate_vreg()
The local variable 'ret' is always zero. Remove it and return 0 at the end of the function. Link: https://lore.kernel.org/r/20210310082741.647-1-zbestahu@gmail.com Signed-off-by: Yue Hu <huyue2@yulong.com> Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
-rw-r--r--drivers/scsi/ufs/ufshcd-pltfrm.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/scsi/ufs/ufshcd-pltfrm.c b/drivers/scsi/ufs/ufshcd-pltfrm.c
index 1a69949a4ea1..fedb7deb3304 100644
--- a/drivers/scsi/ufs/ufshcd-pltfrm.c
+++ b/drivers/scsi/ufs/ufshcd-pltfrm.c
@@ -106,7 +106,6 @@ out:
static int ufshcd_populate_vreg(struct device *dev, const char *name,
struct ufs_vreg **out_vreg)
{
- int ret = 0;
char prop_name[MAX_PROP_SIZE];
struct ufs_vreg *vreg = NULL;
struct device_node *np = dev->of_node;
@@ -135,9 +134,8 @@ static int ufshcd_populate_vreg(struct device *dev, const char *name,
vreg->max_uA = 0;
}
out:
- if (!ret)
- *out_vreg = vreg;
- return ret;
+ *out_vreg = vreg;
+ return 0;
}
/**