diff options
| author | arybchik <arybchik@FreeBSD.org> | 2015-05-29 05:58:21 +0000 |
|---|---|---|
| committer | arybchik <arybchik@FreeBSD.org> | 2015-05-29 05:58:21 +0000 |
| commit | 197df5ee5d0b02f68c84ed0429d4e72b907ea024 (patch) | |
| tree | 7f18bcd6dde16bba6f9d78d75c359a5b0052c276 | |
| parent | ec234b9b79b1496b734c8e59f8e5ee2ad1a44463 (diff) | |
| download | FreeBSD-src-197df5ee5d0b02f68c84ed0429d4e72b907ea024.zip FreeBSD-src-197df5ee5d0b02f68c84ed0429d4e72b907ea024.tar.gz | |
MFC: r283599
sfxge: mask ifmedia options (pauses) when looking for matching mode
Sponsored by: Solarflare Communications, Inc.
| -rw-r--r-- | sys/dev/sfxge/sfxge_port.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/sys/dev/sfxge/sfxge_port.c b/sys/dev/sfxge/sfxge_port.c index aaf3da0..9bdfc1c 100644 --- a/sys/dev/sfxge/sfxge_port.c +++ b/sys/dev/sfxge/sfxge_port.c @@ -710,6 +710,8 @@ sfxge_link_mode_to_phy_cap(efx_link_mode_t mode) static int sfxge_phy_cap_mask(struct sfxge_softc *sc, int ifmedia, uint32_t *phy_cap_mask) { + /* Get global options (duplex), type and subtype bits */ + int ifmedia_masked = ifmedia & (IFM_GMASK | IFM_NMASK | IFM_TMASK); efx_phy_media_type_t medium_type; boolean_t mode_found = B_FALSE; uint32_t cap_mask, mode_cap_mask; @@ -725,7 +727,7 @@ sfxge_phy_cap_mask(struct sfxge_softc *sc, int ifmedia, uint32_t *phy_cap_mask) efx_phy_adv_cap_get(sc->enp, EFX_PHY_CAP_PERM, &cap_mask); for (mode = EFX_LINK_10HDX; mode < EFX_LINK_NMODES; mode++) { - if (ifmedia == sfxge_link_mode[medium_type][mode]) { + if (ifmedia_masked == sfxge_link_mode[medium_type][mode]) { mode_found = B_TRUE; break; } @@ -736,8 +738,8 @@ sfxge_phy_cap_mask(struct sfxge_softc *sc, int ifmedia, uint32_t *phy_cap_mask) * If media is not in the table, it must be IFM_AUTO. */ KASSERT((cap_mask & (1 << EFX_PHY_CAP_AN)) && - ifmedia == (IFM_ETHER | IFM_AUTO), - ("%s: no mode for media %d", __func__, ifmedia)); + ifmedia_masked == (IFM_ETHER | IFM_AUTO), + ("%s: no mode for media %#x", __func__, ifmedia)); *phy_cap_mask = (cap_mask & ~(1 << EFX_PHY_CAP_ASYM)); return (0); } |
