From 4201223de8e6bac403213ad57769dfa723db36e3 Mon Sep 17 00:00:00 2001 From: Tom Rini Date: Mon, 8 May 2017 22:14:32 -0400 Subject: [PATCH] net: phy: mv88e61xx: Fix uninitialized variable warning The variable 'res' may be unused uninitialized if our call to mv88e61xx_port_read (register read) fails and we goto the error handling section. In this case we set 'res' to -EIO to indicate why we failed. Cc: Joe Hershberger Cc: Chris Packham Cc: Kevin Smith Cc: Prafulla Wadaskar Signed-off-by: Tom Rini --- drivers/net/phy/mv88e61xx.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/net/phy/mv88e61xx.c b/drivers/net/phy/mv88e61xx.c index a2fd1686fc..3d2f6b98ad 100644 --- a/drivers/net/phy/mv88e61xx.c +++ b/drivers/net/phy/mv88e61xx.c @@ -655,8 +655,10 @@ static int mv88e61xx_read_port_config(struct phy_device *phydev, u8 port) do { val = mv88e61xx_port_read(phydev, port, PORT_REG_STATUS); - if (val < 0) + if (val < 0) { + res = -EIO; goto unforce; + } if (val & PORT_REG_STATUS_LINK) break; } while (--timeout); -- 2.39.5