From: Tom Rix <trix@redhat.com>
Date: Mon, 19 Oct 2020 07:23:33 -0700
Subject: scsi: Remove unneeded break statements
Patch-mainline: v5.11-rc1
Git-commit: 170b7d2de29e6239b8dbc63a88de1fa9789b0d8d
References: bsc#1164780
A break is not needed if it is preceded by a return or goto.
Link: https://lore.kernel.org/r/20201019142333.16584-1-trix@redhat.com
Signed-off-by: Tom Rix <trix@redhat.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Acked-by: Daniel Wagner <dwagner@suse.de>
---
drivers/scsi/aic94xx/aic94xx_task.c | 1 -
drivers/scsi/be2iscsi/be_mgmt.c | 4 ----
drivers/scsi/bnx2fc/bnx2fc_hwi.c | 1 -
drivers/scsi/fcoe/fcoe.c | 1 -
drivers/scsi/hpsa.c | 1 -
drivers/scsi/hptiop.c | 1 -
drivers/scsi/ipr.c | 1 -
drivers/scsi/isci/phy.c | 2 --
drivers/scsi/lpfc/lpfc_debugfs.c | 12 ++++--------
drivers/scsi/lpfc/lpfc_init.c | 1 -
drivers/scsi/lpfc/lpfc_scsi.c | 1 -
drivers/scsi/lpfc/lpfc_sli.c | 3 ---
drivers/scsi/mvumi.c | 1 -
drivers/scsi/pcmcia/nsp_cs.c | 2 --
drivers/scsi/qla2xxx/qla_mbx.c | 1 -
drivers/scsi/st.c | 1 -
drivers/scsi/sym53c8xx_2/sym_hipd.c | 1 -
17 files changed, 4 insertions(+), 31 deletions(-)
--- a/drivers/scsi/aic94xx/aic94xx_task.c
+++ b/drivers/scsi/aic94xx/aic94xx_task.c
@@ -287,7 +287,6 @@ static void asd_task_tasklet_complete(st
case TA_I_T_NEXUS_LOSS:
opcode = dl->status_block[0];
goto Again;
- break;
case TF_INV_CONN_HANDLE:
ts->resp = SAS_TASK_UNDELIVERED;
ts->stat = SAS_DEVICE_UNKNOWN;
--- a/drivers/scsi/be2iscsi/be_mgmt.c
+++ b/drivers/scsi/be2iscsi/be_mgmt.c
@@ -1243,18 +1243,14 @@ beiscsi_adap_family_disp(struct device *
case OC_DEVICE_ID2:
return snprintf(buf, PAGE_SIZE,
"Obsolete/Unsupported BE2 Adapter Family\n");
- break;
case BE_DEVICE_ID2:
case OC_DEVICE_ID3:
return snprintf(buf, PAGE_SIZE, "BE3-R Adapter Family\n");
- break;
case OC_SKH_ID1:
return snprintf(buf, PAGE_SIZE, "Skyhawk-R Adapter Family\n");
- break;
default:
return snprintf(buf, PAGE_SIZE,
"Unknown Adapter Family: 0x%x\n", dev_id);
- break;
}
}
--- a/drivers/scsi/bnx2fc/bnx2fc_hwi.c
+++ b/drivers/scsi/bnx2fc/bnx2fc_hwi.c
@@ -778,7 +778,6 @@ static void bnx2fc_process_unsol_compl(s
} else
printk(KERN_ERR PFX "SRR in progress\n");
goto ret_err_rqe;
- break;
default:
break;
}
--- a/drivers/scsi/fcoe/fcoe.c
+++ b/drivers/scsi/fcoe/fcoe.c
@@ -1907,7 +1907,6 @@ static int fcoe_device_notification(stru
mutex_unlock(&fcoe_config_mutex);
fcoe_ctlr_device_delete(fcoe_ctlr_to_ctlr_dev(ctlr));
goto out;
- break;
case NETDEV_FEAT_CHANGE:
fcoe_netdev_features_change(lport, netdev);
break;
--- a/drivers/scsi/hpsa.c
+++ b/drivers/scsi/hpsa.c
@@ -7372,7 +7372,6 @@ static int find_PCI_BAR_index(struct pci
dev_warn(&pdev->dev,
"base address is invalid\n");
return -1;
- break;
}
}
if (offset == pci_bar_addr - PCI_BASE_ADDRESS_0)
--- a/drivers/scsi/hptiop.c
+++ b/drivers/scsi/hptiop.c
@@ -766,7 +766,6 @@ static void hptiop_finish_scsi_req(struc
scp->result = SAM_STAT_CHECK_CONDITION;
memcpy(scp->sense_buffer, &req->sg_list, SCSI_SENSE_BUFFERSIZE);
goto skip_resid;
- break;
default:
scp->result = DRIVER_INVALID << 24 | DID_ABORT << 16;
--- a/drivers/scsi/ipr.c
+++ b/drivers/scsi/ipr.c
@@ -9507,7 +9507,6 @@ static pci_ers_result_t ipr_pci_error_de
case pci_channel_io_perm_failure:
ipr_pci_perm_failure(pdev);
return PCI_ERS_RESULT_DISCONNECT;
- break;
default:
break;
}
--- a/drivers/scsi/isci/phy.c
+++ b/drivers/scsi/isci/phy.c
@@ -753,7 +753,6 @@ enum sci_status sci_phy_event_handler(st
default:
phy_event_warn(iphy, state, event_code);
return SCI_FAILURE;
- break;
}
return SCI_SUCCESS;
case SCI_PHY_SUB_AWAIT_IAF_UF:
@@ -957,7 +956,6 @@ enum sci_status sci_phy_event_handler(st
default:
phy_event_warn(iphy, state, event_code);
return SCI_FAILURE_INVALID_STATE;
- break;
}
return SCI_SUCCESS;
default:
--- a/drivers/scsi/lpfc/lpfc_debugfs.c
+++ b/drivers/scsi/lpfc/lpfc_debugfs.c
@@ -3348,7 +3348,6 @@ lpfc_idiag_pcicfg_read(struct file *file
break;
case LPFC_PCI_CFG_BROWSE: /* browse all */
goto pcicfg_browse;
- break;
default:
/* illegal count */
len = 0;
@@ -4388,7 +4387,7 @@ lpfc_idiag_queacc_write(struct file *fil
}
}
goto error_out;
- break;
+
case LPFC_IDIAG_CQ:
/* MBX complete queue */
if (phba->sli4_hba.mbx_cq &&
@@ -4440,7 +4439,7 @@ lpfc_idiag_queacc_write(struct file *fil
}
}
goto error_out;
- break;
+
case LPFC_IDIAG_MQ:
/* MBX work queue */
if (phba->sli4_hba.mbx_wq &&
@@ -4454,7 +4453,7 @@ lpfc_idiag_queacc_write(struct file *fil
goto pass_check;
}
goto error_out;
- break;
+
case LPFC_IDIAG_WQ:
/* ELS work queue */
if (phba->sli4_hba.els_wq &&
@@ -4494,9 +4493,8 @@ lpfc_idiag_queacc_write(struct file *fil
}
}
}
-
goto error_out;
- break;
+
case LPFC_IDIAG_RQ:
/* HDR queue */
if (phba->sli4_hba.hdr_rq &&
@@ -4521,10 +4519,8 @@ lpfc_idiag_queacc_write(struct file *fil
goto pass_check;
}
goto error_out;
- break;
default:
goto error_out;
- break;
}
pass_check:
--- a/drivers/scsi/lpfc/lpfc_init.c
+++ b/drivers/scsi/lpfc/lpfc_init.c
@@ -7199,7 +7199,6 @@ lpfc_init_api_table_setup(struct lpfc_hb
"1431 Invalid HBA PCI-device group: 0x%x\n",
dev_grp);
return -ENODEV;
- break;
}
return 0;
}
--- a/drivers/scsi/lpfc/lpfc_scsi.c
+++ b/drivers/scsi/lpfc/lpfc_scsi.c
@@ -4284,7 +4284,6 @@ lpfc_scsi_api_table_setup(struct lpfc_hb
"1418 Invalid HBA PCI-device group: 0x%x\n",
dev_grp);
return -ENODEV;
- break;
}
phba->lpfc_rampdown_queue_depth = lpfc_rampdown_queue_depth;
phba->lpfc_scsi_cmd_iocb_cmpl = lpfc_scsi_cmd_iocb_cmpl;
--- a/drivers/scsi/lpfc/lpfc_sli.c
+++ b/drivers/scsi/lpfc/lpfc_sli.c
@@ -9082,7 +9082,6 @@ lpfc_mbox_api_table_setup(struct lpfc_hb
"1420 Invalid HBA PCI-device group: 0x%x\n",
dev_grp);
return -ENODEV;
- break;
}
return 0;
}
@@ -9965,7 +9964,6 @@ lpfc_sli4_iocb2wqe(struct lpfc_hba *phba
"2014 Invalid command 0x%x\n",
iocbq->iocb.ulpCommand);
return IOCB_ERROR;
- break;
}
if (iocbq->iocb_flag & LPFC_IO_DIF_PASS)
@@ -10127,7 +10125,6 @@ lpfc_sli_api_table_setup(struct lpfc_hba
"1419 Invalid HBA PCI-device group: 0x%x\n",
dev_grp);
return -ENODEV;
- break;
}
phba->lpfc_get_iocb_from_iocbq = lpfc_get_iocb_from_iocbq;
return 0;
--- a/drivers/scsi/mvumi.c
+++ b/drivers/scsi/mvumi.c
@@ -2310,7 +2310,6 @@ static int mvumi_cfg_hw_reg(struct mvumi
break;
default:
return -1;
- break;
}
return 0;
--- a/drivers/scsi/pcmcia/nsp_cs.c
+++ b/drivers/scsi/pcmcia/nsp_cs.c
@@ -1103,8 +1103,6 @@ static irqreturn_t nspintr(int irq, void
nsp_index_write(base, SCSIBUSCTRL, SCSI_ATN | AUTODIRECTION | ACKENB);
return IRQ_HANDLED;
- break;
-
case PH_RESELECT:
//nsp_dbg(NSP_DEBUG_INTR, "phase reselect");
// *sync_neg = SYNC_NOT_YET;
--- a/drivers/scsi/qla2xxx/qla_mbx.c
+++ b/drivers/scsi/qla2xxx/qla_mbx.c
@@ -4031,7 +4031,6 @@ qla24xx_report_id_acquisition(scsi_qla_h
set_bit(N2N_LOGIN_NEEDED, &vha->dpc_flags);
return;
- break;
case TOPO_FL:
ha->current_topology = ISP_CFG_FL;
break;
--- a/drivers/scsi/st.c
+++ b/drivers/scsi/st.c
@@ -2845,7 +2845,6 @@ static int st_int_ioctl(struct scsi_tape
case MTNOP:
DEBC_printk(STp, "No op on tape.\n");
return 0; /* Should do something ? */
- break;
case MTRETEN:
cmd[0] = START_STOP;
if (STp->immediate) {
--- a/drivers/scsi/sym53c8xx_2/sym_hipd.c
+++ b/drivers/scsi/sym53c8xx_2/sym_hipd.c
@@ -4609,7 +4609,6 @@ static void sym_int_sir(struct sym_hcb *
scr_to_cpu(np->lastmsg), np->msgout[0]);
}
goto out_clrack;
- break;
default:
goto out_reject;
}