Marvell Yukon 88E8062 - media selection problem

[Pyun YongHyeon]

On Tue, Jul 01, 2008 at 12:32:04PM +0200, Krzysztof J??druczyk wrote:
 > Hi,
 > 
 > Pyun YongHyeon wrote:
 > >The 88E8062 support wasn't tested due to lack of hardware and I just
 > >copied the device id from myk driver. Would you show me the output
 > >of "devinfo -rv| grep oui" ? Also it would be better if I can see
 > >verbosed boot message.
 > >
 > 
 > Here you go:
 > 
 > # devinfo -rv | grep oui
 >                   e1000phy0 pnpinfo oui=0x5043 model=0x9 rev=0x1 at phyno=0
 >                   e1000phy1 pnpinfo oui=0x5043 model=0x9 rev=0x1 at phyno=0
 >                 inphy0 pnpinfo oui=0xaa00 model=0x15 rev=0x4 at phyno=1
 > 
 > And dmesg with '-v' in boot.config and 'verbose_loading="YES"' in 
 > loader.conf:
 > 

Would you try attached patch?
The patch is just for checking code path of fiber media.

Btw, it looks like you have dual port controller, right?
-- 
Regards,
Pyun YongHyeon
-------------- next part --------------
--- sys/dev/mii/e1000phy.c.orig	2007-11-21 14:51:55.000000000 +0900
+++ sys/dev/mii/e1000phy.c	2008-07-02 09:57:39.000000000 +0900
@@ -148,10 +148,13 @@
 	esc->mii_model = MII_MODEL(ma->mii_id2);
 	switch (esc->mii_model) {
 	case MII_MODEL_MARVELL_E1011:
-	case MII_MODEL_MARVELL_E1112:
 		if (PHY_READ(sc, E1000_ESSR) & E1000_ESSR_FIBER_LINK)
 			sc->mii_flags |= MIIF_HAVEFIBER;
 		break;
+	case MII_MODEL_MARVELL_E1112:
+		/* XXX Should have a way to get instance info. */
+		sc->mii_flags |= MIIF_HAVEFIBER;
+		break;
 	case MII_MODEL_MARVELL_E3082:
 		/* 88E3082 10/100 Fast Ethernet PHY. */
 		sc->mii_anegticks = MII_ANEGTICKS;