scsi: sym53c8xx_2: Delete unnecessary else-if in sym_xerr_cam_status()

If (x_status & XE_PARITY_ERR) is true we set cam_status = DID_PARITY,
othervise cam_status always ends up being DID_ERROR. Delete superfluous
else-if statements.

Link: https://lore.kernel.org/r/20200902061646.576966-1-yebin10@huawei.com
Signed-off-by: Ye Bin <yebin10@huawei.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>

diff --git a/drivers/scsi/sym53c8xx_2/sym_glue.c b/drivers/scsi/sym53c8xx_2/sym_glue.c
index 28edb6e..d9a045f 100644 (file)
--- a/drivers/scsi/sym53c8xx_2/sym_glue.c
+++ b/drivers/scsi/sym53c8xx_2/sym_glue.c
@@ -156,12 +156,8 @@ void sym_xpt_async_bus_reset(struct sym_hcb *np)
 static int sym_xerr_cam_status(int cam_status, int x_status)
 {
        if (x_status) {
-               if      (x_status & XE_PARITY_ERR)
+               if (x_status & XE_PARITY_ERR)
                        cam_status = DID_PARITY;
-               else if (x_status &(XE_EXTRA_DATA|XE_SODL_UNRUN|XE_SWIDE_OVRUN))
-                       cam_status = DID_ERROR;
-               else if (x_status & XE_BAD_PHASE)
-                       cam_status = DID_ERROR;
                else
                        cam_status = DID_ERROR;
        }