diff options
| author | Bean Huo <beanhuo@micron.com> | 2021-01-18 21:12:33 +0100 |
|---|---|---|
| committer | Martin K. Petersen <martin.petersen@oracle.com> | 2021-01-21 03:45:20 +0100 |
| commit | 60ec37555d05b023721a08005f5dbf716ab5394f (patch) | |
| tree | 0d0a62c783443ffb24653b45adfbd461cfaed97c /drivers/scsi/ufs/ufshcd.c | |
| parent | scsi: ibmvfc: Fix spelling mistake "succeded" -> "succeeded" (diff) | |
| download | linux-60ec37555d05b023721a08005f5dbf716ab5394f.tar.xz linux-60ec37555d05b023721a08005f5dbf716ab5394f.zip | |
scsi: ufs: Delete redundant if statement in ufshcd_intr()
Once going into while-do loop, intr_status is already true, this
if-statement is redundant, remove it.
Link: https://lore.kernel.org/r/20210118201233.3043-1-huobean@gmail.com
Reviewed-by: Avri Altman <avri.altman@wdc.com>
Signed-off-by: Bean Huo <beanhuo@micron.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Diffstat (limited to '')
| -rw-r--r-- | drivers/scsi/ufs/ufshcd.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/scsi/ufs/ufshcd.c b/drivers/scsi/ufs/ufshcd.c index 5fd45f312dbe..c22e751bb030 100644 --- a/drivers/scsi/ufs/ufshcd.c +++ b/drivers/scsi/ufs/ufshcd.c @@ -6315,8 +6315,7 @@ static irqreturn_t ufshcd_intr(int irq, void *__hba) while (intr_status && retries--) { enabled_intr_status = intr_status & ufshcd_readl(hba, REG_INTERRUPT_ENABLE); - if (intr_status) - ufshcd_writel(hba, intr_status, REG_INTERRUPT_STATUS); + ufshcd_writel(hba, intr_status, REG_INTERRUPT_STATUS); if (enabled_intr_status) retval |= ufshcd_sl_intr(hba, enabled_intr_status); |