On Wed, Feb 10, 2016 at 12:06 AM, Stefan Roese <s...@denx.de> wrote: > With commit a058052c [net: phy: do not read configuration register on > reset], phy_reset() will clear the BMCR register. Resulting in bit 12 > being cleared (A/N enable). This leads to autonegotiation link problems, > at least on the Marvell Armada ClearFog board. I suspect that other > boards using this driver will be affected as well. > > At the of m88e1111s_config(), phy_reset() is called. This is not needed > for the PHY to load the changed configuration, as phy_reset() is called > a few lines before already. So lets call genphy_restart_aneg() here > instead to start the AN correctly. > > Tested on clearfog. > > Signed-off-by: Stefan Roese <s...@denx.de> > Cc: Stefan Agner <ste...@agner.ch> > Cc: Hao Zhang <hzh...@ti.com> > Cc: Michal Simek <mon...@monstr.eu> > Cc: Andy Fleming <aflem...@gmail.com> > Cc: Joe Hershberger <joe.hershber...@ni.com>
Acked-by: Joe Hershberger <joe.hershber...@ni.com> _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
