Author: imp
Date: Tue Mar 31 02:50:41 2009
New Revision: 190586
URL: http://svn.freebsd.org/changeset/base/190586

Log:
  Go back to filtering all PHY addresses above 16 since at least two
  cards still have issues with them.  Maybe this is a silicon rev?  In
  any case, doing the filtering only for the AX88790 for the moment.

Modified:
  head/sys/dev/ed/if_ed_pccard.c

Modified: head/sys/dev/ed/if_ed_pccard.c
==============================================================================
--- head/sys/dev/ed/if_ed_pccard.c      Tue Mar 31 02:50:18 2009        
(r190585)
+++ head/sys/dev/ed/if_ed_pccard.c      Tue Mar 31 02:50:41 2009        
(r190586)
@@ -1101,8 +1101,14 @@ ed_miibus_readreg(device_t dev, int phy,
         * the internal PHY, a bit has to be set, when accessing the external
         * PHYs, it must be clear.  See Errata 1, page 51, in the AX88790
         * datasheet for more details.
+        *
+        * Also, PHYs above 16 appear to be phantoms on some cards, but not
+        * others.  Registers read for this are often the same as prior values
+        * read.  Filter all register requests to 17-31.
         */
        if (sc->chip_type == ED_CHIP_TYPE_AX88790) {
+               if (phy > 0x10)
+                       return (0);
                if (phy == 0x10)
                        ed_asic_outb(sc, ED_AX88X90_GPIO,
                            ED_AX88X90_GPIO_INT_PHY);
@@ -1130,15 +1136,10 @@ ed_miibus_writereg(device_t dev, int phy
        struct ed_softc *sc;
 
        sc = device_get_softc(dev);
-       /*
-        * The AX88790 has an interesting quirk.  It has an internal phy that
-        * needs a special bit set to access, but can also have additional
-        * external PHYs set for things like HomeNET media.  When accessing
-        * the internal PHY, a bit has to be set, when accessing the external
-        * PHYs, it must be clear.  See Errata 1, page 51, in the AX88790
-        * datasheet for more details.
-        */
+       /* See ed_miibus_readreg for details */
        if (sc->chip_type == ED_CHIP_TYPE_AX88790) {
+               if (phy > 0x10)
+                       return (0);
                if (phy == 0x10)
                        ed_asic_outb(sc, ED_AX88X90_GPIO,
                            ED_AX88X90_GPIO_INT_PHY);
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to