OSDN Git Service

atlx: remove unnecessary break after return
authorFabian Frederick <fabf@skynet.be>
Sun, 20 Jul 2014 14:27:53 +0000 (16:27 +0200)
committerDavid S. Miller <davem@davemloft.net>
Mon, 21 Jul 2014 04:30:19 +0000 (21:30 -0700)
Signed-off-by: Fabian Frederick <fabf@skynet.be>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/atheros/atlx/atl1.c
drivers/net/ethernet/atheros/atlx/atl2.c

index b460db7..1546d55 100644 (file)
@@ -910,7 +910,6 @@ static s32 atl1_get_speed_and_duplex(struct atl1_hw *hw, u16 *speed, u16 *duplex
                if (netif_msg_hw(adapter))
                        dev_dbg(&pdev->dev, "error getting speed\n");
                return ATLX_ERR_PHY_SPEED;
-               break;
        }
        if (phy_data & MII_ATLX_PSSR_DPLX)
                *duplex = FULL_DUPLEX;
index 6746bd7..c194bc6 100644 (file)
@@ -2493,7 +2493,6 @@ static s32 atl2_get_speed_and_duplex(struct atl2_hw *hw, u16 *speed,
                break;
        default:
                return ATLX_ERR_PHY_SPEED;
-               break;
        }
 
        if (phy_data & MII_ATLX_PSSR_DPLX)
@@ -2933,11 +2932,9 @@ static int atl2_validate_option(int *value, struct atl2_option *opt)
                case OPTION_ENABLED:
                        printk(KERN_INFO "%s Enabled\n", opt->name);
                        return 0;
-                       break;
                case OPTION_DISABLED:
                        printk(KERN_INFO "%s Disabled\n", opt->name);
                        return 0;
-                       break;
                }
                break;
        case range_option: