From dee94462b97ff24a993ef911ab770d47d4b8cc93 Mon Sep 17 00:00:00 2001 From: ChanWoo Lee Date: Tue, 7 Sep 2021 13:41:11 +0900 Subject: [PATCH] UPSTREAM: scsi: ufs: ufs-qcom: Remove unneeded variable 'err' 'err' is never set in this functon. Remove the declaration and just return 0. Link: https://lore.kernel.org/r/20210907044111.29632-1-cw9316.lee@samsung.com Signed-off-by: ChanWoo Lee Signed-off-by: Martin K. Petersen Bug: 234653003 (cherry picked from commit c4adf171e834dab8ca72676ca7264c5b50a542a8) Change-Id: I8f38ceac838d543e6d2e89d9c8cdd3bd1b785694 Signed-off-by: Bart Van Assche --- drivers/scsi/ufs/ufs-qcom.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/scsi/ufs/ufs-qcom.c b/drivers/scsi/ufs/ufs-qcom.c index 82cc55744afc..c6e61410020f 100644 --- a/drivers/scsi/ufs/ufs-qcom.c +++ b/drivers/scsi/ufs/ufs-qcom.c @@ -892,7 +892,6 @@ static int ufs_qcom_setup_clocks(struct ufs_hba *hba, bool on, enum ufs_notify_change_status status) { struct ufs_qcom_host *host = ufshcd_get_variant(hba); - int err = 0; /* * In case ufs_qcom_init() is not yet done, simply ignore. @@ -920,7 +919,7 @@ static int ufs_qcom_setup_clocks(struct ufs_hba *hba, bool on, break; } - return err; + return 0; } static int