diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2016-03-15 03:18:49 -0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2016-03-18 18:28:11 -0400 |
| commit | 977bc20cf66367a530d17696f7e1783510a115b7 (patch) | |
| tree | 8ace513e42a984a3dc1b3a4cb38d389f703feb9a | |
| parent | fe3a5f6c795810edb1646a840fec3c8c350c2a4e (diff) | |
mediatek: checking for IS_ERR() instead of NULL
of_phy_connect() returns NULL on error, it never returns error pointers.
Fixes: 656e705243fd ('net-next: mediatek: add support for MT7623 ethernet')
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: John Crispin <blogic@openwrt.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | drivers/net/ethernet/mediatek/mtk_eth_soc.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/ethernet/mediatek/mtk_eth_soc.c b/drivers/net/ethernet/mediatek/mtk_eth_soc.c index 1e6c5498bba9..fd19d2d163b8 100644 --- a/drivers/net/ethernet/mediatek/mtk_eth_soc.c +++ b/drivers/net/ethernet/mediatek/mtk_eth_soc.c | |||
| @@ -186,9 +186,9 @@ static int mtk_phy_connect_node(struct mtk_eth *eth, struct mtk_mac *mac, | |||
| 186 | 186 | ||
| 187 | phydev = of_phy_connect(eth->netdev[mac->id], phy_node, | 187 | phydev = of_phy_connect(eth->netdev[mac->id], phy_node, |
| 188 | mtk_phy_link_adjust, 0, phy_mode); | 188 | mtk_phy_link_adjust, 0, phy_mode); |
| 189 | if (IS_ERR(phydev)) { | 189 | if (!phydev) { |
| 190 | dev_err(eth->dev, "could not connect to PHY\n"); | 190 | dev_err(eth->dev, "could not connect to PHY\n"); |
| 191 | return PTR_ERR(phydev); | 191 | return -ENODEV; |
| 192 | } | 192 | } |
| 193 | 193 | ||
| 194 | dev_info(eth->dev, | 194 | dev_info(eth->dev, |
