scsi: qla2xxx: Suppress gcc 7 fall-through warnings
authorBart Van Assche <bart.vanassche@wdc.com>
Fri, 8 Dec 2017 00:02:46 +0000 (16:02 -0800)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 12 Dec 2017 02:50:29 +0000 (21:50 -0500)
Avoid that building with gcc 7 and W=1 triggers warnings similar to the
following:

drivers/scsi/qla2xxx/qla_isr.c:1189:27: warning: this statement may fall through [-Wimplicit-fallthrough=]

Signed-off-by: Bart Van Assche <bart.vanassche@wdc.com>
Cc: Himanshu Madhani <himanshu.madhani@cavium.com>
Cc: Quinn Tran <quinn.tran@cavium.com>
Cc: Hannes Reinecke <hare@suse.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/qla2xxx/qla_gs.c
drivers/scsi/qla2xxx/qla_isr.c
drivers/scsi/qla2xxx/qla_sup.c
drivers/scsi/qla2xxx/qla_target.c

index 7d715e5..07fe17a 100644 (file)
@@ -177,7 +177,7 @@ qla2x00_chk_ms_status(scsi_qla_host_t *vha, ms_iocb_entry_t *ms_pkt,
                        break;
                case CS_TIMEOUT:
                        rval = QLA_FUNCTION_TIMEOUT;
-                       /* drop through */
+                       /* fall through */
                default:
                        ql_dbg(ql_dbg_disc, vha, 0x2033,
                            "%s failed, completion status (%x) on port_id: "
index 8538238..a55bfaa 100644 (file)
@@ -1202,6 +1202,7 @@ global_port_update:
                                qla2xxx_wake_dpc(vha);
                        }
                }
+               /* fall through */
        case MBA_IDC_COMPLETE:
                if (ha->notify_lb_portup_comp && !vha->vp_idx)
                        complete(&ha->lb_portup_comp);
@@ -1769,7 +1770,7 @@ qla24xx_logio_entry(scsi_qla_host_t *vha, struct req_que *req,
                                set_bit(ISP_ABORT_NEEDED, &vha->dpc_flags);
                        qla2xxx_wake_dpc(vha);
                }
-               /* drop through */
+               /* fall through */
        default:
                data[0] = MBS_COMMAND_ERROR;
                break;
@@ -2967,9 +2968,9 @@ process_err:
                                    (response_t *)pkt);
                                break;
                        } else {
-                               /* drop through */
                                qlt_24xx_process_atio_queue(vha, 1);
                        }
+                       /* fall through */
                case ABTS_RESP_24XX:
                case CTIO_TYPE7:
                case CTIO_CRC2:
index b4336e0..d2db86e 100644 (file)
@@ -2461,6 +2461,7 @@ qla2x00_write_optrom_data(struct scsi_qla_host *vha, uint8_t *buf,
                                sec_mask = 0x1e000;
                                break;
                        }
+                       /* fall through */
                default:
                        /* Default to 16 kb sector size. */
                        rest_addr = 0x3fff;
index a23107b..067bcc5 100644 (file)
@@ -450,6 +450,7 @@ void qlt_response_pkt_all_vps(struct scsi_qla_host *vha,
                ql_dbg(ql_dbg_tgt, vha, 0xe073,
                        "qla_target(%d):%s: CRC2 Response pkt\n",
                        vha->vp_idx, __func__);
+               /* fall through */
        case CTIO_TYPE7:
        {
                struct ctio7_from_24xx *entry = (struct ctio7_from_24xx *)pkt;
@@ -4889,7 +4890,7 @@ static int qlt_24xx_handle_els(struct scsi_qla_host *vha,
                        res = 1;
                        break;
                }
-               /* drop through */
+               /* fall through */
        case ELS_LOGO:
        case ELS_PRLO:
                spin_lock_irqsave(&ha->tgt.sess_lock, flags);