Commit fa8d3272 authored by Dan Carpenter's avatar Dan Carpenter Committed by Martin K. Petersen
Browse files

scsi: ufs: ufs-qcom: Remove impossible check 

The "dev_req_params" pointer points to inside the middle of a struct so it
can't be NULL.  Removing this impossible condition is nice because now we
don't need to consider the correct error code for that situation.

Link: https://lore.kernel.org/r/Y/yA3niWUcGYgBU8@kili


Fixes: f06fcc71 ("scsi: ufs-qcom: add QUniPro hardware support and power optimizations")
Signed-off-by: default avatarDan Carpenter <error27@gmail.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 2ebe1615

drivers/ufs/host/ufs-qcom.c

+1 −1
Original line number Diff line number Diff line
@@ -1177,7 +1177,7 @@ static int ufs_qcom_clk_scale_notify(struct ufs_hba *hba, 
			err = ufs_qcom_clk_scale_down_post_change(hba);





		if (err || !dev_req_params) {

		if (err) {

			ufshcd_uic_hibern8_exit(hba);

			return err;

		}