Commit da8badd7 authored by Bart Van Assche's avatar Bart Van Assche Committed by Martin K. Petersen

scsi: ufs: Simplify ufshcd_clear_cmd()

Remove the local variable 'err'. This patch does not change any
functionality.

Link: https://lore.kernel.org/r/20220613214442.212466-2-bvanassche@acm.orgReviewed-by: default avatarStanley Chu <stanley.chu@mediatek.com>
Reviewed-by: default avatarAdrian Hunter <adrian.hunter@intel.com>
Signed-off-by: default avatarBart Van Assche <bvanassche@acm.org>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent f6eed15f
...@@ -2866,7 +2866,6 @@ static int ufshcd_compose_dev_cmd(struct ufs_hba *hba, ...@@ -2866,7 +2866,6 @@ static int ufshcd_compose_dev_cmd(struct ufs_hba *hba,
static int static int
ufshcd_clear_cmd(struct ufs_hba *hba, int tag) ufshcd_clear_cmd(struct ufs_hba *hba, int tag)
{ {
int err = 0;
unsigned long flags; unsigned long flags;
u32 mask = 1 << tag; u32 mask = 1 << tag;
...@@ -2879,11 +2878,8 @@ ufshcd_clear_cmd(struct ufs_hba *hba, int tag) ...@@ -2879,11 +2878,8 @@ ufshcd_clear_cmd(struct ufs_hba *hba, int tag)
* wait for h/w to clear corresponding bit in door-bell. * wait for h/w to clear corresponding bit in door-bell.
* max. wait is 1 sec. * max. wait is 1 sec.
*/ */
err = ufshcd_wait_for_register(hba, return ufshcd_wait_for_register(hba, REG_UTP_TRANSFER_REQ_DOOR_BELL,
REG_UTP_TRANSFER_REQ_DOOR_BELL, mask, ~mask, 1000, 1000);
mask, ~mask, 1000, 1000);
return err;
} }
static int static int
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment