scsi: ufs: Simplify a condition
authorDan Carpenter <dan.carpenter@oracle.com>
Fri, 13 Dec 2019 10:49:35 +0000 (13:49 +0300)
committerMartin K. Petersen <martin.petersen@oracle.com>
Fri, 20 Dec 2019 03:08:53 +0000 (22:08 -0500)
We know that "check_for_bkops" is non-zero on this side of the || because
it was checked on the other side.

Link: https://lore.kernel.org/r/20191213104935.wgpq2epaz6zh5zus@kili.mountain
Reviewed-by: Bart Van Assche <bvanassche@acm.org>
Reviewed-by: Avri Altman <avri.altman@wdc.com>
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/ufs/ufshcd.c

index bf981f0..c299c5f 100644 (file)
@@ -7684,8 +7684,7 @@ static int ufshcd_link_state_transition(struct ufs_hba *hba,
         * turning off the link would also turn off the device.
         */
        else if ((req_link_state == UIC_LINK_OFF_STATE) &&
-                  (!check_for_bkops || (check_for_bkops &&
-                   !hba->auto_bkops_enabled))) {
+                (!check_for_bkops || !hba->auto_bkops_enabled)) {
                /*
                 * Let's make sure that link is in low power mode, we are doing
                 * this currently by putting the link in Hibern8. Otherway to