net: phy: initialize rc to zero to avoid returning garbage value
authorColin Ian King <colin.king@canonical.com>
Fri, 12 Aug 2016 20:29:24 +0000 (21:29 +0100)
committerDavid S. Miller <davem@davemloft.net>
Sat, 13 Aug 2016 22:51:49 +0000 (15:51 -0700)
In the case where phydev->interrupts is not PHY_INTERRUPT_ENABLED
function vsc85xx_ack_interrupt is returning an uninitialized
garbage value.  Fix this by initializing rc to zero.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/phy/mscc.c

index 2e1de53e3548c7ece5af7d2cc0f0f0a045dacd78..ad33390b382ad9644a0f09533b993f08ea36e4b6 100644 (file)
@@ -84,7 +84,7 @@ static int vsc85xx_config_init(struct phy_device *phydev)
 
 static int vsc85xx_ack_interrupt(struct phy_device *phydev)
 {
-       int rc;
+       int rc = 0;
 
        if (phydev->interrupts == PHY_INTERRUPT_ENABLED)
                rc = phy_read(phydev, MII_VSC85XX_INT_STATUS);