After fixing up some of the AR8035 PHY registers, issue a reset. This seems

to fix autonegotiation at 100BaseTX.  Drop the bit of code that powers up
the PHY as the reset should already take care of that.

ok patrick@

diff --git a/sys/arch/armv7/imx/if_fec.c b/sys/arch/armv7/imx/if_fec.c
index 32b8729..ad06d31 100644 (file)
--- a/sys/arch/armv7/imx/if_fec.c
+++ b/sys/arch/armv7/imx/if_fec.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: if_fec.c,v 1.11 2016/07/27 11:45:02 patrick Exp $ */
+/* $OpenBSD: if_fec.c,v 1.12 2016/08/04 14:14:05 kettenis Exp $ */
 /*
  * Copyright (c) 2012-2013 Patrick Wildt <patrick@blueri.se>
  *
@@ -486,10 +486,7 @@ fec_phy_init(struct fec_softc *sc, struct mii_softc *child)
                reg = fec_miibus_readreg(dev, phy, 0x1e);
                fec_miibus_writereg(dev, phy, 0x1e, reg | 0x0100);
 
-               /* phy power */
-               reg = fec_miibus_readreg(dev, phy, 0x00);
-               if (reg & 0x0800)
-                       fec_miibus_writereg(dev, phy, 0x00, reg & ~0x0800);
+               PHY_RESET(child);
        }
 
        if (child->mii_oui == MII_OUI_MICREL &&