]> git.baikalelectronics.ru Git - kernel.git/commit
net: phy: relax error checking when creating sysfs link netdev->phydev
authorGrygorii Strashko <grygorii.strashko@ti.com>
Fri, 16 Mar 2018 22:08:35 +0000 (17:08 -0500)
committerDavid S. Miller <davem@davemloft.net>
Tue, 20 Mar 2018 01:14:27 +0000 (21:14 -0400)
commitea4b0119e156d24424ec5fc4ba0a4b83bc398eaa
treeebb85cbfdbb58eb932a8f01b6f95b83fcc060ecf
parent79492b201818c742a4f9f68faa44e56f9a20cdc1
net: phy: relax error checking when creating sysfs link netdev->phydev

Some ethernet drivers (like TI CPSW) may connect and manage >1 Net PHYs per
one netdevice, as result such drivers will produce warning during system
boot and fail to connect second phy to netdevice when PHYLIB framework
will try to create sysfs link netdev->phydev for second PHY
in phy_attach_direct(), because sysfs link with the same name has been
created already for the first PHY. As result, second CPSW external
port will became unusable.

Fix it by relaxing error checking when PHYLIB framework is creating sysfs
link netdev->phydev in phy_attach_direct(), suppressing warning by using
sysfs_create_link_nowarn() and adding error message instead.
After this change links (phy->netdev and netdev->phy) creation failure is not
fatal any more and system can continue working, which fixes TI CPSW issue.

Cc: Florian Fainelli <f.fainelli@gmail.com>
Cc: Andrew Lunn <andrew@lunn.ch>
Fixes: 5a0cf3cd0565 ("net: phy: Relax error checking on sysfs_create_link()")
Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/phy/phy_device.c