diff options
author | Ben Hutchings <bhutchings@solarflare.com> | 2008-12-13 00:44:14 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-12-13 00:57:58 -0500 |
commit | 27dd2caca4eabe7c13a052b7456495ba75535e6a (patch) | |
tree | 514e7844ab78e5110e09d1e4a650e3283cdd32e9 /drivers/net/sfc/mdio_10g.c | |
parent | f31a45d2f45a7667acd6e85ab6613b0910c55ea9 (diff) |
sfc: Add support for MMDs numbered >15
Combine DEVS0 and DEVS1 registers into a 32-bit mask instead of
reading just DEVS0.
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/sfc/mdio_10g.c')
-rw-r--r-- | drivers/net/sfc/mdio_10g.c | 24 |
1 files changed, 13 insertions, 11 deletions
diff --git a/drivers/net/sfc/mdio_10g.c b/drivers/net/sfc/mdio_10g.c index 19e25210b687..1eed48407ed8 100644 --- a/drivers/net/sfc/mdio_10g.c +++ b/drivers/net/sfc/mdio_10g.c | |||
@@ -121,16 +121,18 @@ int mdio_clause45_wait_reset_mmds(struct efx_nic *efx, | |||
121 | int mdio_clause45_check_mmds(struct efx_nic *efx, | 121 | int mdio_clause45_check_mmds(struct efx_nic *efx, |
122 | unsigned int mmd_mask, unsigned int fatal_mask) | 122 | unsigned int mmd_mask, unsigned int fatal_mask) |
123 | { | 123 | { |
124 | int devices, mmd = 0; | 124 | u32 devices; |
125 | int probe_mmd; | 125 | int mmd = 0, probe_mmd; |
126 | 126 | ||
127 | /* Historically we have probed the PHYXS to find out what devices are | 127 | /* Historically we have probed the PHYXS to find out what devices are |
128 | * present,but that doesn't work so well if the PHYXS isn't expected | 128 | * present,but that doesn't work so well if the PHYXS isn't expected |
129 | * to exist, if so just find the first item in the list supplied. */ | 129 | * to exist, if so just find the first item in the list supplied. */ |
130 | probe_mmd = (mmd_mask & MDIO_MMDREG_DEVS0_PHYXS) ? MDIO_MMD_PHYXS : | 130 | probe_mmd = (mmd_mask & MDIO_MMDREG_DEVS_PHYXS) ? MDIO_MMD_PHYXS : |
131 | __ffs(mmd_mask); | 131 | __ffs(mmd_mask); |
132 | devices = mdio_clause45_read(efx, efx->mii.phy_id, | 132 | devices = (mdio_clause45_read(efx, efx->mii.phy_id, |
133 | probe_mmd, MDIO_MMDREG_DEVS0); | 133 | probe_mmd, MDIO_MMDREG_DEVS0) | |
134 | mdio_clause45_read(efx, efx->mii.phy_id, | ||
135 | probe_mmd, MDIO_MMDREG_DEVS1) << 16); | ||
134 | 136 | ||
135 | /* Check all the expected MMDs are present */ | 137 | /* Check all the expected MMDs are present */ |
136 | if (devices < 0) { | 138 | if (devices < 0) { |
@@ -175,14 +177,14 @@ bool mdio_clause45_links_ok(struct efx_nic *efx, unsigned int mmd_mask) | |||
175 | else if (efx_phy_mode_disabled(efx->phy_mode)) | 177 | else if (efx_phy_mode_disabled(efx->phy_mode)) |
176 | return false; | 178 | return false; |
177 | else if (efx->loopback_mode == LOOPBACK_PHYXS) | 179 | else if (efx->loopback_mode == LOOPBACK_PHYXS) |
178 | mmd_mask &= ~(MDIO_MMDREG_DEVS0_PHYXS | | 180 | mmd_mask &= ~(MDIO_MMDREG_DEVS_PHYXS | |
179 | MDIO_MMDREG_DEVS0_PCS | | 181 | MDIO_MMDREG_DEVS_PCS | |
180 | MDIO_MMDREG_DEVS0_PMAPMD); | 182 | MDIO_MMDREG_DEVS_PMAPMD); |
181 | else if (efx->loopback_mode == LOOPBACK_PCS) | 183 | else if (efx->loopback_mode == LOOPBACK_PCS) |
182 | mmd_mask &= ~(MDIO_MMDREG_DEVS0_PCS | | 184 | mmd_mask &= ~(MDIO_MMDREG_DEVS_PCS | |
183 | MDIO_MMDREG_DEVS0_PMAPMD); | 185 | MDIO_MMDREG_DEVS_PMAPMD); |
184 | else if (efx->loopback_mode == LOOPBACK_PMAPMD) | 186 | else if (efx->loopback_mode == LOOPBACK_PMAPMD) |
185 | mmd_mask &= ~MDIO_MMDREG_DEVS0_PMAPMD; | 187 | mmd_mask &= ~MDIO_MMDREG_DEVS_PMAPMD; |
186 | 188 | ||
187 | while (mmd_mask) { | 189 | while (mmd_mask) { |
188 | if (mmd_mask & 1) { | 190 | if (mmd_mask & 1) { |