scsi: qla2xxx: Introduce a switch/case statement in qlt_xmit_tm_rsp()

This patch improves code readability but does not change any functionality.

Cc: Himanshu Madhani <himanshu.madhani@cavium.com>
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
Acked-by: Himanshu Madhani <himanshu.madhani@cavium.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>

diff --git a/drivers/scsi/qla2xxx/qla_target.c b/drivers/scsi/qla2xxx/qla_target.c
index bceb8e8..510337e 100644 (file)
--- a/drivers/scsi/qla2xxx/qla_target.c
+++ b/drivers/scsi/qla2xxx/qla_target.c
@@ -2379,20 +2379,20 @@ void qlt_xmit_tm_rsp(struct qla_tgt_mgmt_cmd *mcmd)
        }
 
        if (mcmd->flags == QLA24XX_MGMT_SEND_NACK) {
-               if (mcmd->orig_iocb.imm_ntfy.u.isp24.status_subcode ==
-                   ELS_LOGO ||
-                   mcmd->orig_iocb.imm_ntfy.u.isp24.status_subcode ==
-                   ELS_PRLO ||
-                   mcmd->orig_iocb.imm_ntfy.u.isp24.status_subcode ==
-                   ELS_TPRLO) {
+               switch (mcmd->orig_iocb.imm_ntfy.u.isp24.status_subcode) {
+               case ELS_LOGO:
+               case ELS_PRLO:
+               case ELS_TPRLO:
                        ql_dbg(ql_dbg_disc, vha, 0x2106,
                            "TM response logo %phC status %#x state %#x",
                            mcmd->sess->port_name, mcmd->fc_tm_rsp,
                            mcmd->flags);
                        qlt_schedule_sess_for_deletion(mcmd->sess);
-               } else {
+                       break;
+               default:
                        qlt_send_notify_ack(vha->hw->base_qpair,
                            &mcmd->orig_iocb.imm_ntfy, 0, 0, 0, 0, 0, 0);
+                       break;
                }
        } else {
                if (mcmd->orig_iocb.atio.u.raw.entry_type == ABTS_RECV_24XX) {