diff options
| author | Uwe Kleine-König <u.kleine-koenig@pengutronix.de> | 2016-09-30 22:10:08 +0200 |
|---|---|---|
| committer | Sascha Hauer <s.hauer@pengutronix.de> | 2016-10-04 08:15:01 +0200 |
| commit | 84a4369d338cc99eaa346080a2cd51af3586c16e (patch) | |
| tree | 61e3f6b71c4f22f58f6698a2ba1ee7ccea21a34e | |
| parent | 38c0c1105a27cd8aa370cfec41846e7db7504e55 (diff) | |
| download | barebox-84a4369d338cc99eaa346080a2cd51af3586c16e.tar.gz barebox-84a4369d338cc99eaa346080a2cd51af3586c16e.tar.xz | |
net/phy: marvell: fix error handling
Without first assigning to ret it doesn't make sense to check it.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
| -rw-r--r-- | drivers/net/phy/marvell.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/net/phy/marvell.c b/drivers/net/phy/marvell.c index 6409f14ae2..9a963f6d5e 100644 --- a/drivers/net/phy/marvell.c +++ b/drivers/net/phy/marvell.c @@ -198,7 +198,8 @@ static int m88e1121_config_init(struct phy_device *phydev) if (ret < 0) return ret; - phy_write(phydev, MII_MARVELL_PHY_PAGE, MII_MARVELL_PHY_DEFAULT_PAGE); + ret = phy_write(phydev, MII_MARVELL_PHY_PAGE, + MII_MARVELL_PHY_DEFAULT_PAGE); if (ret < 0) return ret; |