]> git.baikalelectronics.ru Git - kernel.git/commit
net: fec: Prevent unbind operation
authorFabio Estevam <festevam@gmail.com>
Tue, 18 Feb 2020 15:34:44 +0000 (12:34 -0300)
committerDavid S. Miller <davem@davemloft.net>
Tue, 18 Feb 2020 20:30:21 +0000 (12:30 -0800)
commitef80709d4d17e69d47191435835943310f987cdb
tree2263f34e45086c55b380a01f07ca6101f40dba8c
parent1b4a05ac5ac7cc4def3b8bdecc60977970b2259b
net: fec: Prevent unbind operation

After performing an unbind/bind operation the network is no longer
functional on i.MX6 (which has a single FEC instance):

# echo 2188000.ethernet > /sys/bus/platform/drivers/fec/unbind
# echo 2188000.ethernet > /sys/bus/platform/drivers/fec/bind
[   10.756519] pps pps0: new PPS source ptp0
[   10.792626] libphy: fec_enet_mii_bus: probed
[   10.799330] fec 2188000.ethernet eth0: registered PHC device 1
# udhcpc -i eth0
udhcpc: started, v1.31.1
[   14.985211] fec 2188000.ethernet eth0: no PHY, assuming direct connection to switch
[   14.993140] libphy: PHY fixed-0:00 not found
[   14.997643] fec 2188000.ethernet eth0: could not attach to PHY

On SoCs with two FEC instances there are some cases where one FEC instance
depends on the other one being present. One such example is i.MX28, which
has the following FEC dependency as noted in the comments:

/*
 * The i.MX28 dual fec interfaces are not equal.
 * Here are the differences:
 *
 *  - fec0 supports MII & RMII modes while fec1 only supports RMII
 *  - fec0 acts as the 1588 time master while fec1 is slave
 *  - external phys can only be configured by fec0
 *
 * That is to say fec1 can not work independently. It only works
 * when fec0 is working. The reason behind this design is that the
 * second interface is added primarily for Switch mode.
 *
 * Because of the last point above, both phys are attached on fec0
 * mdio interface in board design, and need to be configured by
 * fec0 mii_bus.
 */

Prevent the unbind operation to avoid these issues.

Signed-off-by: Fabio Estevam <festevam@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/freescale/fec_main.c