]> git.proxmox.com Git - mirror_ubuntu-jammy-kernel.git/commit
net: fec: Fix temporary RMII clock reset on link up
authorLaurent Badel <laurentbadel@eaton.com>
Mon, 25 Jan 2021 10:07:45 +0000 (11:07 +0100)
committerJakub Kicinski <kuba@kernel.org>
Wed, 27 Jan 2021 02:24:39 +0000 (18:24 -0800)
commitc730ab423bfa1ae99b688a9abdedf74477d44955
tree70adc938dc3824de01d88256f27ecbc7dfc68ec4
parentb491e6a7391e3ecdebdd7a097550195cc878924a
net: fec: Fix temporary RMII clock reset on link up

fec_restart() does a hard reset of the MAC module when the link status
changes to up. This temporarily resets the R_CNTRL register which controls
the MII mode of the ENET_OUT clock. In the case of RMII, the clock
frequency momentarily drops from 50MHz to 25MHz until the register is
reconfigured. Some link partners do not tolerate this glitch and
invalidate the link causing failure to establish a stable link when using
PHY polling mode. Since as per IEEE802.3 the criteria for link validity
are PHY-specific, what the partner should tolerate cannot be assumed, so
avoid resetting the MII clock by using software reset instead of hardware
reset when the link is up. This is generally relevant only if the SoC
provides the clock to an external PHY and the PHY is configured for RMII.

Signed-off-by: Laurent Badel <laurentbadel@eaton.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/freescale/fec.h
drivers/net/ethernet/freescale/fec_main.c