| From b3e487c0cf425369a48049251af75593a5652dc1 Mon Sep 17 00:00:00 2001 |
| From: "David S. Miller" <davem@davemloft.net> |
| Date: Mon, 22 Jul 2019 20:44:48 -0700 |
| Subject: Revert "net: hns: fix LED configuration for marvell phy" |
| |
| From: David S. Miller <davem@davemloft.net> |
| |
| commit b3e487c0cf425369a48049251af75593a5652dc1 upstream. |
| |
| This reverts commit f4e5f775db5a4631300dccd0de5eafb50a77c131. |
| |
| Andrew Lunn says this should be handled another way. |
| |
| Signed-off-by: David S. Miller <davem@davemloft.net> |
| Cc: Yonglong Liu <liuyonglong@huawei.com> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| |
| --- |
| drivers/net/ethernet/hisilicon/hns/hns_enet.c | 23 +---------------------- |
| 1 file changed, 1 insertion(+), 22 deletions(-) |
| |
| --- a/drivers/net/ethernet/hisilicon/hns/hns_enet.c |
| +++ b/drivers/net/ethernet/hisilicon/hns/hns_enet.c |
| @@ -11,7 +11,6 @@ |
| #include <linux/io.h> |
| #include <linux/ip.h> |
| #include <linux/ipv6.h> |
| -#include <linux/marvell_phy.h> |
| #include <linux/module.h> |
| #include <linux/phy.h> |
| #include <linux/platform_device.h> |
| @@ -1150,13 +1149,6 @@ static void hns_nic_adjust_link(struct n |
| } |
| } |
| |
| -static int hns_phy_marvell_fixup(struct phy_device *phydev) |
| -{ |
| - phydev->dev_flags |= MARVELL_PHY_LED0_LINK_LED1_ACTIVE; |
| - |
| - return 0; |
| -} |
| - |
| /** |
| *hns_nic_init_phy - init phy |
| *@ndev: net device |
| @@ -1182,16 +1174,6 @@ int hns_nic_init_phy(struct net_device * |
| if (h->phy_if != PHY_INTERFACE_MODE_XGMII) { |
| phy_dev->dev_flags = 0; |
| |
| - /* register the PHY fixup (for Marvell 88E1510) */ |
| - ret = phy_register_fixup_for_uid(MARVELL_PHY_ID_88E1510, |
| - MARVELL_PHY_ID_MASK, |
| - hns_phy_marvell_fixup); |
| - /* we can live without it, so just issue a warning */ |
| - if (ret) |
| - netdev_warn(ndev, |
| - "Cannot register PHY fixup, ret=%d\n", |
| - ret); |
| - |
| ret = phy_connect_direct(ndev, phy_dev, hns_nic_adjust_link, |
| h->phy_if); |
| } else { |
| @@ -2447,11 +2429,8 @@ static int hns_nic_dev_remove(struct pla |
| hns_nic_uninit_ring_data(priv); |
| priv->ring_data = NULL; |
| |
| - if (ndev->phydev) { |
| - phy_unregister_fixup_for_uid(MARVELL_PHY_ID_88E1510, |
| - MARVELL_PHY_ID_MASK); |
| + if (ndev->phydev) |
| phy_disconnect(ndev->phydev); |
| - } |
| |
| if (!IS_ERR_OR_NULL(priv->ae_handle)) |
| hnae_put_handle(priv->ae_handle); |