From b888195fee831938b67565fd901fb8629b87775d Mon Sep 17 00:00:00 2001 From: Dan Carpenter Date: Wed, 1 Jun 2022 12:41:51 -0700 Subject: [PATCH] UPSTREAM: scsi: ufs: ufshcd: Delete unnecessary NULL check The 'info' pointer points to somewhere in the middle of the 'hba' struct. It can't possibly be NULL. Delete the NULL check. Link: https://lore.kernel.org/r/YotFotj43TkB8Rid@kili Signed-off-by: Dan Carpenter Signed-off-by: Martin K. Petersen Bug: 234653003 (cherry picked from commit 476e45923b5df19534178d0ed5c87c517dd6f669) Change-Id: I4eb49e0cd3a3fa6c3e885324fc69ba8b2a04aba8 Signed-off-by: Bart Van Assche --- drivers/ufs/core/ufshcd.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/ufs/core/ufshcd.c b/drivers/ufs/core/ufshcd.c index 47ec1aaaf956..22881084937f 100644 --- a/drivers/ufs/core/ufshcd.c +++ b/drivers/ufs/core/ufshcd.c @@ -8517,10 +8517,7 @@ static int ufshcd_init_hba_vreg(struct ufs_hba *hba) { struct ufs_vreg_info *info = &hba->vreg_info; - if (info) - return ufshcd_get_vreg(hba->dev, info->vdd_hba); - - return 0; + return ufshcd_get_vreg(hba->dev, info->vdd_hba); } static int ufshcd_setup_clocks(struct ufs_hba *hba, bool on)