Commit 7ebb9db0 authored by Tom Rix's avatar Tom Rix Committed by Jakub Kicinski

net: remove unneeded break

A break is not needed if it is preceded by a return or goto
Signed-off-by: default avatarTom Rix <trix@redhat.com>
Link: https://lore.kernel.org/r/20201019172607.31622-1-trix@redhat.comSigned-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent df6afe2f
...@@ -1163,7 +1163,6 @@ int aq_nic_set_link_ksettings(struct aq_nic_s *self, ...@@ -1163,7 +1163,6 @@ int aq_nic_set_link_ksettings(struct aq_nic_s *self,
default: default:
err = -1; err = -1;
goto err_exit; goto err_exit;
break;
} }
if (!(self->aq_nic_cfg.aq_hw_caps->link_speed_msk & rate)) { if (!(self->aq_nic_cfg.aq_hw_caps->link_speed_msk & rate)) {
err = -1; err = -1;
......
...@@ -434,7 +434,6 @@ static int enic_grxclsrule(struct enic *enic, struct ethtool_rxnfc *cmd) ...@@ -434,7 +434,6 @@ static int enic_grxclsrule(struct enic *enic, struct ethtool_rxnfc *cmd)
break; break;
default: default:
return -EINVAL; return -EINVAL;
break;
} }
fsp->h_u.tcp_ip4_spec.ip4src = flow_get_u32_src(&n->keys); fsp->h_u.tcp_ip4_spec.ip4src = flow_get_u32_src(&n->keys);
......
...@@ -350,7 +350,6 @@ static s32 ixgbe_calc_eeprom_checksum_X540(struct ixgbe_hw *hw) ...@@ -350,7 +350,6 @@ static s32 ixgbe_calc_eeprom_checksum_X540(struct ixgbe_hw *hw)
if (ixgbe_read_eerd_generic(hw, pointer, &length)) { if (ixgbe_read_eerd_generic(hw, pointer, &length)) {
hw_dbg(hw, "EEPROM read failed\n"); hw_dbg(hw, "EEPROM read failed\n");
return IXGBE_ERR_EEPROM; return IXGBE_ERR_EEPROM;
break;
} }
/* Skip pointer section if length is invalid. */ /* Skip pointer section if length is invalid. */
......
...@@ -89,17 +89,13 @@ __be16 lmc_proto_type(lmc_softc_t *sc, struct sk_buff *skb) /*FOLD00*/ ...@@ -89,17 +89,13 @@ __be16 lmc_proto_type(lmc_softc_t *sc, struct sk_buff *skb) /*FOLD00*/
switch(sc->if_type){ switch(sc->if_type){
case LMC_PPP: case LMC_PPP:
return hdlc_type_trans(skb, sc->lmc_device); return hdlc_type_trans(skb, sc->lmc_device);
break;
case LMC_NET: case LMC_NET:
return htons(ETH_P_802_2); return htons(ETH_P_802_2);
break;
case LMC_RAW: /* Packet type for skbuff kind of useless */ case LMC_RAW: /* Packet type for skbuff kind of useless */
return htons(ETH_P_802_2); return htons(ETH_P_802_2);
break;
default: default:
printk(KERN_WARNING "%s: No protocol set for this interface, assuming 802.2 (which is wrong!!)\n", sc->name); printk(KERN_WARNING "%s: No protocol set for this interface, assuming 802.2 (which is wrong!!)\n", sc->name);
return htons(ETH_P_802_2); return htons(ETH_P_802_2);
break;
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment