diff mbox series

scsi: arcmsr: Remove the superfluous break

Message ID 20200918093230.49050-1-jingxiangfeng@huawei.com
State New
Headers show
Series scsi: arcmsr: Remove the superfluous break | expand

Commit Message

Jing Xiangfeng Sept. 18, 2020, 9:32 a.m. UTC
Remove the superfluous break, as there is a 'return' before it.

Signed-off-by: Jing Xiangfeng <jingxiangfeng@huawei.com>
---
 drivers/scsi/arcmsr/arcmsr_hba.c | 2 --
 1 file changed, 2 deletions(-)

Comments

Gustavo A. R. Silva Sept. 18, 2020, 2:56 p.m. UTC | #1
On Fri, Sep 18, 2020 at 05:32:30PM +0800, Jing Xiangfeng wrote:
> Remove the superfluous break, as there is a 'return' before it.
>

Apparently, the change is correct. Please, just add a proper Fixes tag by
yourself this time.

Thanks
--
Gustavo

> Signed-off-by: Jing Xiangfeng <jingxiangfeng@huawei.com>
> ---
>  drivers/scsi/arcmsr/arcmsr_hba.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/drivers/scsi/arcmsr/arcmsr_hba.c b/drivers/scsi/arcmsr/arcmsr_hba.c
> index ec895d0319f0..74add6d247d5 100644
> --- a/drivers/scsi/arcmsr/arcmsr_hba.c
> +++ b/drivers/scsi/arcmsr/arcmsr_hba.c
> @@ -2699,10 +2699,8 @@ static irqreturn_t arcmsr_interrupt(struct AdapterControlBlock *acb)
>  	switch (acb->adapter_type) {
>  	case ACB_ADAPTER_TYPE_A:
>  		return arcmsr_hbaA_handle_isr(acb);
> -		break;
>  	case ACB_ADAPTER_TYPE_B:
>  		return arcmsr_hbaB_handle_isr(acb);
> -		break;
>  	case ACB_ADAPTER_TYPE_C:
>  		return arcmsr_hbaC_handle_isr(acb);
>  	case ACB_ADAPTER_TYPE_D:
> -- 
> 2.17.1
>
Joe Perches Sept. 18, 2020, 5:13 p.m. UTC | #2
On Fri, 2020-09-18 at 09:56 -0500, Gustavo A. R. Silva wrote:
> On Fri, Sep 18, 2020 at 05:32:30PM +0800, Jing Xiangfeng wrote:

> > Remove the superfluous break, as there is a 'return' before it.

> > 

> 

> Apparently, the change is correct. Please, just add a proper Fixes tag by

> yourself this time.


There's no need for a "Fixes:" here, it's purely cosmetic.

btw:

There are at least 150 instances of
	return foo(...);
	break;
still in the kernel tree:

$ grep-2.5.4 --include=*.[ch] -rP -n "\breturn\s+\w+\s*\([^\)]+\)\s*;\s*break\s*;" * | grep return
arch/mips/loongson2ef/common/cs5536/cs5536_isa.c:275:		return pci_isa_read_bar(0);
arch/mips/loongson2ef/common/cs5536/cs5536_isa.c:277:		return pci_isa_read_bar(1);
arch/mips/loongson2ef/common/cs5536/cs5536_isa.c:279:		return pci_isa_read_bar(2);
arch/mips/loongson2ef/common/cs5536/cs5536_isa.c:283:		return pci_isa_read_bar(4);
arch/mips/loongson2ef/common/cs5536/cs5536_isa.c:285:		return pci_isa_read_bar(5);
arch/sparc/kernel/viohs.c:334:			return handshake_failure(vio);
arch/sparc/kernel/perf_event.c:1442:			return PTR_ERR(pmap);
arch/x86/kvm/emulate.c:1857:			return segmented_write(ctxt,
arch/x86/kvm/emulate.c:1859:		return segmented_write(ctxt,
arch/x86/kvm/emulate.c:2922:			return emulate_gp(ctxt, 0);
arch/x86/kvm/emulate.c:4232:			return emulate_gp(ctxt, 0);
arch/x86/kvm/emulate.c:4255:			return emulate_gp(ctxt, 0);
arch/x86/kvm/emulate.c:4261:			return emulate_gp(ctxt, 0);
arch/x86/kernel/kgdb.c:535:				return single_step_cont(regs, args);
arch/x86/kernel/cpu/bugs.c:1676:			return l1tf_show_state(buf);
arch/x86/kernel/paravirt_patch.c:96:			return PATCH(lock, queued_spin_unlock, insn_buff, len);
arch/x86/kernel/paravirt_patch.c:100:			return PATCH(lock, vcpu_is_preempted, insn_buff, len);
arch/arm/nwfpe/fpa11_cprt.c:37:		return PerformFLT(opcode);
arch/arm/nwfpe/fpa11_cprt.c:39:		return PerformFIX(opcode);
arch/powerpc/kvm/book3s_pr_papr.c:394:			return kvmppc_h_pr_xics_hcall(vcpu, cmd);
block/blk-merge.c:1000:			return bio_attempt_back_merge(rq, bio, nr_segs);
block/blk-merge.c:1003:			return bio_attempt_front_merge(rq, bio, nr_segs);
drivers/net/macvtap.c:180:			return notifier_from_errno(err);
drivers/net/wireless/realtek/rtlwifi/rtl8821ae/phy.c:2091:		return __rtl8821ae_phy_config_with_headerfile(hw,
drivers/net/wireless/realtek/rtlwifi/rtl8821ae/phy.c:2093:		return __rtl8821ae_phy_config_with_headerfile(hw,
drivers/net/wireless/realtek/rtlwifi/rtl8821ae/phy.c:2116:		return __rtl8821ae_phy_config_with_headerfile(hw,
drivers/net/wireless/marvell/mwifiex/uap_event.c:246:			return mwifiex_handle_event_ext_scan_report(priv,
drivers/net/wireless/marvell/mwifiex/cfg80211.c:1210:			return mwifiex_change_vif_to_sta_adhoc(dev, curr_iftype,
drivers/net/wireless/marvell/mwifiex/cfg80211.c:1247:			return mwifiex_change_vif_to_sta_adhoc(dev, curr_iftype,
drivers/net/wireless/ath/ath10k/htt_rx.c:3878:		return ath10k_htt_rx_proc_rx_frag_ind(htt,
drivers/net/wireless/intel/iwlwifi/mvm/mac-ctxt.c:1200:		return iwl_mvm_mac_ctxt_cmd_sta(mvm, vif, action,
drivers/net/wireless/intel/iwlwifi/mvm/mac-ctxt.c:1205:			return iwl_mvm_mac_ctxt_cmd_go(mvm, vif, action);
drivers/net/ipvlan/ipvtap.c:173:			return notifier_from_errno(err);
drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c:4837:			return mvpp2_set_ts_config(port, ifr);
drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c:4841:			return mvpp2_get_ts_config(port, ifr);
drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c:3624:			return cxgb4_tc_matchall_stats(dev, cls_matchall);
drivers/net/ethernet/mellanox/mlxsw/spectrum_router.c:1792:			return mlxsw_sp_netdevice_ipip_ul_vrf_event(mlxsw_sp,
drivers/net/ethernet/mellanox/mlxsw/spectrum_router.c:6831:			return PTR_ERR(rif);
drivers/net/ethernet/mellanox/mlx5/core/accel/tls.h:74:			return MLX5_CAP_TLS(mdev,  tls_1_2_aes_gcm_128);
drivers/net/wan/lmc/lmc_proto.c:103:	    return hdlc_type_trans(skb, sc->lmc_device);
drivers/net/wan/lmc/lmc_proto.c:105:        return htons(ETH_P_802_2);
drivers/net/wan/lmc/lmc_proto.c:107:        return htons(ETH_P_802_2);
drivers/net/wan/lmc/lmc_proto.c:110:        return htons(ETH_P_802_2);
drivers/net/hyperv/netvsc.c:1275:		return netvsc_receive(ndev, net_device, nvchan,
drivers/target/iscsi/iscsi_target.c:2098:			return iscsit_add_reject_cmd(cmd,
drivers/crypto/marvell/octeontx/otx_cptvf_algs.c:527:				return PTR_ERR(ctx->hashalg);
drivers/crypto/marvell/octeontx/otx_cptvf_algs.c:533:				return PTR_ERR(ctx->hashalg);
drivers/crypto/marvell/octeontx/otx_cptvf_algs.c:539:				return PTR_ERR(ctx->hashalg);
drivers/crypto/marvell/octeontx/otx_cptvf_algs.c:545:				return PTR_ERR(ctx->hashalg);
drivers/mfd/da9052-i2c.c:69:			return regmap_read(da9052->regmap,
drivers/staging/vme/devices/vme_user.c:357:			return vme_master_set(image[minor].resource,
drivers/staging/vme/devices/vme_user.c:393:			return vme_slave_set(image[minor].resource,
drivers/staging/iio/addac/adt7316.c:1158:			return sprintf(buf, "%d\n", data);
drivers/staging/comedi/drivers/ni_mio_common.c:5571:			return NISTC_ATRIG_ETC_GPFO_0_SEL_TO_SRC(reg);
drivers/staging/comedi/drivers/ni_mio_common.c:5574:			return NISTC_ATRIG_ETC_GPFO_1_SEL_TO_SRC(reg);
drivers/firmware/google/vpd_decode.c:89:			return callback(key, key_len, value, value_len,
drivers/media/pci/solo6x10/solo6x10-v4l2-enc.c:1105:			return solo_set_motion_block(solo_dev, solo_enc->ch,
drivers/media/dvb-frontends/si21xx.c:466:		return si21_writereg(state, LNB_CTRL_REG_1, val | 0x40);
drivers/media/usb/ttusb-dec/ttusb_dec.c:1104:		return ttusb_dec_start_ts_feed(dvbdmxfeed);
drivers/media/usb/ttusb-dec/ttusb_dec.c:1107:		return ttusb_dec_start_sec_feed(dvbdmxfeed);
drivers/media/usb/ttusb-dec/ttusb_dec.c:1157:		return ttusb_dec_stop_ts_feed(dvbdmxfeed);
drivers/media/usb/ttusb-dec/ttusb_dec.c:1160:		return ttusb_dec_stop_sec_feed(dvbdmxfeed);
drivers/media/v4l2-core/v4l2-fwnode.c:1264:				return PTR_ERR(fwnode);
drivers/media/test-drivers/vivid/vivid-vid-out.c:1204:			return v4l2_src_change_event_subscribe(fh, sub);
drivers/media/cec/core/cec-adap.c:2078:			return cec_feature_abort(adap, msg);
drivers/scsi/bfa/bfa_ioc.h:1000:		return bfi_image_cb_get_chunk(off);
drivers/scsi/bfa/bfa_ioc.h:1002:		return bfi_image_ct_get_chunk(off);
drivers/scsi/bfa/bfa_ioc.h:1004:		return bfi_image_ct2_get_chunk(off);
drivers/scsi/be2iscsi/be_mgmt.c:1245:		return snprintf(buf, PAGE_SIZE,
drivers/scsi/be2iscsi/be_mgmt.c:1248:		return snprintf(buf, PAGE_SIZE, "BE3-R Adapter Family\n");
drivers/scsi/be2iscsi/be_mgmt.c:1250:		return snprintf(buf, PAGE_SIZE, "Skyhawk-R Adapter Family\n");
drivers/scsi/be2iscsi/be_mgmt.c:1252:		return snprintf(buf, PAGE_SIZE,
drivers/scsi/arcmsr/arcmsr_hba.c:2701:		return arcmsr_hbaA_handle_isr(acb);
drivers/scsi/arcmsr/arcmsr_hba.c:2703:		return arcmsr_hbaB_handle_isr(acb);
drivers/iio/adc/nau7802.c:370:				return nau7802_set_gain(st, i);
drivers/iio/adc/meson_saradc.c:594:		return meson_sar_adc_get_sample(indio_dev, chan, NO_AVERAGING,
drivers/iio/adc/meson_saradc.c:597:		return meson_sar_adc_get_sample(indio_dev, chan,
drivers/infiniband/hw/mlx5/fs.c:2235:			return MLX5_CAP_FLOWTABLE(ibdev->mdev,
drivers/infiniband/hw/mlx5/fs.c:2238:			return MLX5_CAP_FLOWTABLE_NIC_TX(ibdev->mdev,
drivers/infiniband/hw/mlx5/fs.c:2241:			return MLX5_CAP_FLOWTABLE_NIC_RX(ibdev->mdev,
drivers/infiniband/hw/mlx5/fs.c:2244:			return MLX5_CAP_FLOWTABLE_NIC_RX(ibdev->mdev, decap);
drivers/char/ipmi/ipmi_devintf.c:492:		return ipmi_set_my_address(priv->user, val.channel, val.value);
drivers/s390/scsi/zfcp_erp.c:1096:			return zfcp_erp_port_strategy_close(erp_action);
drivers/s390/char/tape_34xx.c:653:					return tape_34xx_done(request);
drivers/s390/char/tape_3590.c:945:		return tape_3590_erp_failed(device, request, irb, -EIO);
drivers/nfc/st21nfca/core.c:792:		return nfc_hci_send_cmd_async(hdev, target->hci_reader_gate,
drivers/gpu/drm/mgag200/mgag200_mode.c:796:		return mga_g200se_set_plls(mdev, clock);
drivers/gpu/drm/mgag200/mgag200_mode.c:799:		return mga_g200wb_set_plls(mdev, clock);
drivers/gpu/drm/mgag200/mgag200_mode.c:801:		return mga_g200ev_set_plls(mdev, clock);
drivers/gpu/drm/mgag200/mgag200_mode.c:804:		return mga_g200eh_set_plls(mdev, clock);
drivers/gpu/drm/mgag200/mgag200_mode.c:806:		return mga_g200er_set_plls(mdev, clock);
drivers/gpu/drm/i915/display/intel_display.c:12599:				return PTR_ERR(linked_state);
drivers/gpu/drm/amd/amdgpu/amdgpu_dma_buf.c:321:			return ERR_PTR(r);
drivers/gpu/drm/amd/display/dc/bios/bios_parser.c:953:				return get_ss_info_from_tbl(bp, clk_id_ss,
drivers/gpu/drm/gma500/mdfld_dsi_output.c:451:			return PTR_ERR(gpiod);
drivers/gpu/drm/gma500/mdfld_dsi_output.c:455:			return PTR_ERR(gpiod);
drivers/gpu/drm/bridge/tc358767.c:405:			return tc_aux_read_data(tc, msg->buffer, size);
drivers/tty/vt/vt_ioctl.c:1002:			return vt_disallocate(--arg);
drivers/tty/tty_io.c:1868:			return ERR_PTR(-ENODEV);
drivers/platform/x86/acer-wmi.c:794:			return ec_write(0x83, value);
drivers/mtd/mtdchar.c:883:		return mtd_block_isbad(mtd, offs);
drivers/mtd/mtdchar.c:892:		return mtd_block_markbad(mtd, offs);
drivers/tee/optee/supp.c:251:				return PTR_ERR(req);
drivers/acpi/acpi_lpss.c:1292:			return sysfs_create_group(&pdev->dev.kobj,
drivers/hid/hid-lg-g15.c:616:			return lg_g15_event(g15, data, size);
drivers/hid/hid-lg-g15.c:619:			return lg_g15_v2_event(g15, data, size);
drivers/hid/hid-lg-g15.c:625:			return lg_g510_leds_event(g15, data, size);
drivers/ata/libata-sff.c:1477:			return ata_sff_idle_irq(ap);
fs/ubifs/find.c:923:			return PTR_ERR(lp);
fs/nfsd/nfs4state.c:5724:			return find_writeable_file(s->sc_file);
fs/ioctl.c:732:			return file_ioctl(filp, cmd, argp);
include/net/netfilter/nf_queue.h:92:			return hash_v4(iph, initval);
include/net/netfilter/nf_queue.h:97:			return hash_v6(ip6h, initval);
kernel/exit.c:1519:			return PTR_ERR(pid);
lib/test_objagg.c:777:			return PTR_ERR(objagg_obj);
net/wireless/core.c:1439:			return notifier_from_errno(-ERFKILL);
net/wireless/util.c:86:			return MHZ_TO_KHZ(2407 + chan * 5);
net/wireless/util.c:91:			return MHZ_TO_KHZ(5000 + chan * 5);
net/wireless/util.c:97:			return MHZ_TO_KHZ(5950 + chan * 5);
net/wireless/util.c:100:			return MHZ_TO_KHZ(56160 + chan * 2160);
net/wireless/chan.c:496:			return BIT(chandef->width);
net/wireless/nl80211.c:6363:			return PTR_ERR(params.vlan);
net/netlabel/netlabel_kapi.c:1444:			return cipso_v4_cache_add(ptr, secattr);
net/netlabel/netlabel_kapi.c:1449:			return calipso_cache_add(ptr, secattr);
net/8021q/vlan.c:510:			return notifier_from_errno(err);
net/8021q/vlan.c:519:			return notifier_from_errno(err);
net/ipv4/ipmr.c:844:			return PTR_ERR(dev);
net/bridge/br.c:71:			return notifier_from_errno(err);
net/bridge/br.c:117:			return notifier_from_errno(err);
net/xfrm/xfrm_output.c:384:			return xfrm6_prepare_output(x, skb);
net/xfrm/xfrm_output.c:389:			return xfrm6_transport_output(x, skb);
net/xfrm/xfrm_input.c:341:			return xfrm6_remove_beet_encap(x, skb);
net/xfrm/xfrm_input.c:346:			return xfrm6_remove_tunnel_encap(x, skb);
net/xfrm/xfrm_input.c:444:			return xfrm6_transport_input(x, skb);
net/packet/af_packet.c:3529:			return dev_mc_del(dev, i->addr);
net/packet/af_packet.c:3540:			return dev_uc_del(dev, i->addr);
net/sunrpc/clnt.c:191:			return PTR_ERR(dentry);
sound/soc/intel/skylake/skl-pcm.c:504:		return skl_run_pipe(skl, mconfig->pipe);
sound/oss/dmasound/dmasound_core.c:1005:		return IOCTL_OUT(arg, fmt);
sound/oss/dmasound/dmasound_core.c:1021:		return IOCTL_OUT(arg, size);
sound/oss/dmasound/dmasound_core.c:1123:		return IOCTL_OUT(arg, data);
sound/aoa/soundbus/i2sbus/control.c:119:			return pmf_call_one(i2sdev->cell_disable, &args);
sound/aoa/soundbus/i2sbus/control.c:122:			return pmf_call_one(i2sdev->cell_enable, &args);
sound/aoa/soundbus/i2sbus/control.c:160:			return pmf_call_one(i2sdev->clock_disable, &args);
sound/aoa/soundbus/i2sbus/control.c:163:			return pmf_call_one(i2sdev->clock_enable, &args);
sound/usb/quirks.c:1372:			return snd_usb_motu_microbookii_boot_quirk(dev);
sound/core/seq/oss/seq_oss_event.c:72:			return snd_seq_oss_midi_putc(dp, q->s.dev, q->s.parm1, ev);
tools/perf/util/sort.c:2168:				return scnprintf(hpp->buf, hpp->size,
tools/perf/util/intel-pt-decoder/intel-pt-pkt-decoder.c:547:			return intel_pt_get_bip_4(buf, len, packet);
tools/perf/util/intel-pt-decoder/intel-pt-pkt-decoder.c:550:			return intel_pt_get_bip_8(buf, len, packet);
tools/perf/util/evsel.c:2536:			return scnprintf(msg, size, "%s",
tools/perf/util/evsel.c:2569:			return scnprintf(msg, size, "The 'aux_output' feature is not supported, update the kernel.");
tools/perf/ui/stdio/hist.c:392:		return callchain__fprintf_graph(fp, &he->sorted_chain, total_samples,
tools/perf/ui/stdio/hist.c:394:		return callchain__fprintf_graph(fp, &he->sorted_chain, total_samples,
tools/perf/ui/stdio/hist.c:396:		return callchain__fprintf_flat(fp, &he->sorted_chain, total_samples);
tools/perf/ui/stdio/hist.c:398:		return callchain__fprintf_folded(fp, &he->sorted_chain, total_samples);
Jens Axboe Sept. 18, 2020, 5:17 p.m. UTC | #3
On 9/18/20 11:13 AM, Joe Perches wrote:
> On Fri, 2020-09-18 at 09:56 -0500, Gustavo A. R. Silva wrote:
>> On Fri, Sep 18, 2020 at 05:32:30PM +0800, Jing Xiangfeng wrote:
>>> Remove the superfluous break, as there is a 'return' before it.
>>>
>>
>> Apparently, the change is correct. Please, just add a proper Fixes tag by
>> yourself this time.
> 
> There's no need for a "Fixes:" here, it's purely cosmetic.
> 
> btw:
> 
> There are at least 150 instances of
> 	return foo(...);
> 	break;
> still in the kernel tree:

A lot of these are false positives, since they follow a pattern of:

if (some_condition)
	return func();
break;
Dan Carpenter Sept. 18, 2020, 6 p.m. UTC | #4
Smatch just ignores these because they're often done deliberately.

regards,
dan carpenter
Colin King Sept. 18, 2020, 6:03 p.m. UTC | #5
On 18/09/2020 19:00, Dan Carpenter wrote:
> Smatch just ignores these because they're often done deliberately.
> 
> regards,
> dan carpenter
> 

And I ignore fixing them when coverity reports them because life is too
short.

Colin
Joe Perches Sept. 18, 2020, 7:07 p.m. UTC | #6
On Fri, 2020-09-18 at 11:17 -0600, Jens Axboe wrote:
> On 9/18/20 11:13 AM, Joe Perches wrote:
> > On Fri, 2020-09-18 at 09:56 -0500, Gustavo A. R. Silva wrote:
> > > On Fri, Sep 18, 2020 at 05:32:30PM +0800, Jing Xiangfeng wrote:
> > > > Remove the superfluous break, as there is a 'return' before it.
> > > > 
> > > 
> > > Apparently, the change is correct. Please, just add a proper Fixes tag by
> > > yourself this time.
> > 
> > There's no need for a "Fixes:" here, it's purely cosmetic.
> > 
> > btw:
> > 
> > There are at least 150 instances of
> > 	return foo(...);
> > 	break;
> > still in the kernel tree:
> 
> A lot of these are false positives, since they follow a pattern of:
> 
> if (some_condition)
> 	return func();
> break;

Ah, right. Thanks.

Refining the test a bit to make sure the number of tabs before the
return and before the break are the same appears to show 42 instances.

$ grep-2.5.4 -rP --include=*.[ch] -n "^([\t]+)\breturn\s+\w+\s*\([^\)]+\)\s*;[ \t]*\n\1break\s*;\s*(case\s+\w+|default)\s*:" * | grep return | wc -l
42

(the line numbering seems screwed up below,
 but the instances listed seem right)

$ grep-2.5.4 -rP --include=*.[ch] -n "^([\t]+)\breturn\s+\w+\s*\([^\)]+\)\s*;[ \t]*\n\1break\s*;\s*(case\s+\w+|default)\s*:" * | grep return
arch/mips/loongson2ef/common/cs5536/cs5536_isa.c:275:		return pci_isa_read_bar(0);
arch/mips/loongson2ef/common/cs5536/cs5536_isa.c:276:		return pci_isa_read_bar(1);
arch/mips/loongson2ef/common/cs5536/cs5536_isa.c:277:		return pci_isa_read_bar(2);
arch/mips/loongson2ef/common/cs5536/cs5536_isa.c:280:		return pci_isa_read_bar(4);
arch/mips/loongson2ef/common/cs5536/cs5536_isa.c:281:		return pci_isa_read_bar(5);
arch/x86/kvm/emulate.c:1863:		return segmented_write(ctxt,
drivers/net/wireless/realtek/rtlwifi/rtl8821ae/phy.c:2091:		return __rtl8821ae_phy_config_with_headerfile(hw,
drivers/net/wireless/realtek/rtlwifi/rtl8821ae/phy.c:2092:		return __rtl8821ae_phy_config_with_headerfile(hw,
drivers/net/wireless/realtek/rtlwifi/rtl8821ae/phy.c:2114:		return __rtl8821ae_phy_config_with_headerfile(hw,
drivers/net/wireless/marvell/mwifiex/cfg80211.c:1210:			return mwifiex_change_vif_to_sta_adhoc(dev, curr_iftype,
drivers/net/wireless/marvell/mwifiex/cfg80211.c:1246:			return mwifiex_change_vif_to_sta_adhoc(dev, curr_iftype,
drivers/net/wireless/intel/iwlwifi/mvm/mac-ctxt.c:1200:		return iwl_mvm_mac_ctxt_cmd_sta(mvm, vif, action,
drivers/net/hyperv/netvsc.c:1275:		return netvsc_receive(ndev, net_device, nvchan,
drivers/media/dvb-frontends/si21xx.c:466:		return si21_writereg(state, LNB_CTRL_REG_1, val | 0x40);
drivers/media/usb/ttusb-dec/ttusb_dec.c:1104:		return ttusb_dec_start_ts_feed(dvbdmxfeed);
drivers/media/usb/ttusb-dec/ttusb_dec.c:1105:		return ttusb_dec_start_sec_feed(dvbdmxfeed);
drivers/media/usb/ttusb-dec/ttusb_dec.c:1153:		return ttusb_dec_stop_ts_feed(dvbdmxfeed);
drivers/scsi/bfa/bfa_ioc.h:1000:		return bfi_image_cb_get_chunk(off);
drivers/scsi/bfa/bfa_ioc.h:1001:		return bfi_image_ct_get_chunk(off);
drivers/scsi/bfa/bfa_ioc.h:1002:		return bfi_image_ct2_get_chunk(off);
drivers/scsi/be2iscsi/be_mgmt.c:1245:		return snprintf(buf, PAGE_SIZE,
drivers/scsi/be2iscsi/be_mgmt.c:1247:		return snprintf(buf, PAGE_SIZE, "BE3-R Adapter Family\n");
drivers/scsi/be2iscsi/be_mgmt.c:1248:		return snprintf(buf, PAGE_SIZE, "Skyhawk-R Adapter Family\n");
drivers/scsi/arcmsr/arcmsr_hba.c:2701:		return arcmsr_hbaA_handle_isr(acb);
drivers/scsi/arcmsr/arcmsr_hba.c:2702:		return arcmsr_hbaB_handle_isr(acb);
drivers/iio/adc/meson_saradc.c:594:		return meson_sar_adc_get_sample(indio_dev, chan, NO_AVERAGING,
drivers/iio/adc/meson_saradc.c:595:		return meson_sar_adc_get_sample(indio_dev, chan,
drivers/s390/char/tape_34xx.c:653:					return tape_34xx_done(request);
drivers/s390/char/tape_3590.c:945:		return tape_3590_erp_failed(device, request, irb, -EIO);
drivers/nfc/st21nfca/core.c:792:		return nfc_hci_send_cmd_async(hdev, target->hci_reader_gate,
drivers/gpu/drm/mgag200/mgag200_mode.c:796:		return mga_g200se_set_plls(mdev, clock);
drivers/gpu/drm/mgag200/mgag200_mode.c:798:		return mga_g200wb_set_plls(mdev, clock);
drivers/gpu/drm/mgag200/mgag200_mode.c:799:		return mga_g200ev_set_plls(mdev, clock);
drivers/gpu/drm/mgag200/mgag200_mode.c:801:		return mga_g200eh_set_plls(mdev, clock);
sound/soc/intel/skylake/skl-pcm.c:504:		return skl_run_pipe(skl, mconfig->pipe);
sound/oss/dmasound/dmasound_core.c:1005:		return IOCTL_OUT(arg, fmt);
sound/oss/dmasound/dmasound_core.c:1020:		return IOCTL_OUT(arg, size);
sound/oss/dmasound/dmasound_core.c:1121:		return IOCTL_OUT(arg, data);
tools/perf/ui/stdio/hist.c:392:		return callchain__fprintf_graph(fp, &he->sorted_chain, total_samples,
tools/perf/ui/stdio/hist.c:393:		return callchain__fprintf_graph(fp, &he->sorted_chain, total_samples,
tools/perf/ui/stdio/hist.c:394:		return callchain__fprintf_flat(fp, &he->sorted_chain, total_samples);
tools/perf/ui/stdio/hist.c:395:		return callchain__fprintf_folded(fp, &he->sorted_chain, total_samples);
Jing Xiangfeng Sept. 19, 2020, 1:52 a.m. UTC | #7
On 2020/9/18 22:56, Gustavo A. R. Silva wrote:
> On Fri, Sep 18, 2020 at 05:32:30PM +0800, Jing Xiangfeng wrote:

>> Remove the superfluous break, as there is a 'return' before it.

>>

>

> Apparently, the change is correct. Please, just add a proper Fixes tag by

> yourself this time.


Thanks for comments! I'll resend with the following Fixes tag:

Fixes: 6b3937227479 ("arcmsr: fix command timeout under heavy load")
>

> Thanks

> --

> Gustavo

>

>> Signed-off-by: Jing Xiangfeng <jingxiangfeng@huawei.com>

>> ---

>>   drivers/scsi/arcmsr/arcmsr_hba.c | 2 --

>>   1 file changed, 2 deletions(-)

>>

>> diff --git a/drivers/scsi/arcmsr/arcmsr_hba.c b/drivers/scsi/arcmsr/arcmsr_hba.c

>> index ec895d0319f0..74add6d247d5 100644

>> --- a/drivers/scsi/arcmsr/arcmsr_hba.c

>> +++ b/drivers/scsi/arcmsr/arcmsr_hba.c

>> @@ -2699,10 +2699,8 @@ static irqreturn_t arcmsr_interrupt(struct AdapterControlBlock *acb)

>>   	switch (acb->adapter_type) {

>>   	case ACB_ADAPTER_TYPE_A:

>>   		return arcmsr_hbaA_handle_isr(acb);

>> -		break;

>>   	case ACB_ADAPTER_TYPE_B:

>>   		return arcmsr_hbaB_handle_isr(acb);

>> -		break;

>>   	case ACB_ADAPTER_TYPE_C:

>>   		return arcmsr_hbaC_handle_isr(acb);

>>   	case ACB_ADAPTER_TYPE_D:

>> --

>> 2.17.1

>>

> .

>
diff mbox series

Patch

diff --git a/drivers/scsi/arcmsr/arcmsr_hba.c b/drivers/scsi/arcmsr/arcmsr_hba.c
index ec895d0319f0..74add6d247d5 100644
--- a/drivers/scsi/arcmsr/arcmsr_hba.c
+++ b/drivers/scsi/arcmsr/arcmsr_hba.c
@@ -2699,10 +2699,8 @@  static irqreturn_t arcmsr_interrupt(struct AdapterControlBlock *acb)
 	switch (acb->adapter_type) {
 	case ACB_ADAPTER_TYPE_A:
 		return arcmsr_hbaA_handle_isr(acb);
-		break;
 	case ACB_ADAPTER_TYPE_B:
 		return arcmsr_hbaB_handle_isr(acb);
-		break;
 	case ACB_ADAPTER_TYPE_C:
 		return arcmsr_hbaC_handle_isr(acb);
 	case ACB_ADAPTER_TYPE_D: