net: phy: dp83869: fix return of uninitialized variable ret
authorColin Ian King <colin.king@canonical.com>
Mon, 18 Nov 2019 11:48:35 +0000 (11:48 +0000)
committerDavid S. Miller <davem@davemloft.net>
Tue, 19 Nov 2019 01:23:44 +0000 (17:23 -0800)
In the case where the call to phy_interface_is_rgmii returns zero
the variable ret is left uninitialized and this is returned at
the end of the function dp83869_configure_rgmii.  Fix this by
returning 0 instead of the uninitialized value in ret.

Addresses-Coverity: ("Uninitialized scalar variable")
Fixes: 01db923e8377 ("net: phy: dp83869: Add TI dp83869 phy")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/phy/dp83869.c

index fcd418716c108407ebcdb5448a9b352d618f5310..1c7a7c57dec3d0fcbece4d1062229c0212ea770e 100644 (file)
@@ -239,7 +239,7 @@ static int dp83869_configure_rgmii(struct phy_device *phydev,
                               dp83869->io_impedance &
                               DP83869_IO_MUX_CFG_IO_IMPEDANCE_CTRL);
 
-       return ret;
+       return 0;
 }
 
 static int dp83869_configure_mode(struct phy_device *phydev,