svn commit: r190586 - head/sys/dev/ed
Warner Losh
imp at FreeBSD.org
Mon Mar 30 19:50:42 PDT 2009
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);
More information about the svn-src-all
mailing list