net: dsa: lan9303: fix variable 'res' set but not used
authorAndrew Lunn <andrew@lunn.ch>
Sun, 5 Jul 2020 20:55:55 +0000 (22:55 +0200)
committerDavid S. Miller <davem@davemloft.net>
Sun, 5 Jul 2020 22:48:19 +0000 (15:48 -0700)
Since lan9303_adjust_link() is a void function, there is no option to
return an error. So just remove the variable and lets any errors be
discarded.

Cc: Egil Hjelmeland <privat@egil-hjelmeland.no>
Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/dsa/lan9303-core.c

index cc17a44..aa1142d 100644 (file)
@@ -1042,7 +1042,7 @@ static void lan9303_adjust_link(struct dsa_switch *ds, int port,
                                struct phy_device *phydev)
 {
        struct lan9303 *chip = ds->priv;
-       int ctl, res;
+       int ctl;
 
        if (!phy_is_pseudo_fixed_link(phydev))
                return;
@@ -1063,15 +1063,14 @@ static void lan9303_adjust_link(struct dsa_switch *ds, int port,
        else
                ctl &= ~BMCR_FULLDPLX;
 
-       res =  lan9303_phy_write(ds, port, MII_BMCR, ctl);
+       lan9303_phy_write(ds, port, MII_BMCR, ctl);
 
        if (port == chip->phy_addr_base) {
                /* Virtual Phy: Remove Turbo 200Mbit mode */
                lan9303_read(chip->regmap, LAN9303_VIRT_SPECIAL_CTRL, &ctl);
 
                ctl &= ~LAN9303_VIRT_SPECIAL_TURBO;
-               res =  regmap_write(chip->regmap,
-                                   LAN9303_VIRT_SPECIAL_CTRL, ctl);
+               regmap_write(chip->regmap, LAN9303_VIRT_SPECIAL_CTRL, ctl);
        }
 }