.. | .. |
---|
| 1 | +// SPDX-License-Identifier: GPL-2.0+ |
---|
1 | 2 | /* |
---|
2 | 3 | * Core PHY library, taken from phy.c |
---|
3 | | - * |
---|
4 | | - * This program is free software; you can redistribute it and/or modify it |
---|
5 | | - * under the terms of the GNU General Public License as published by the |
---|
6 | | - * Free Software Foundation; either version 2 of the License, or (at your |
---|
7 | | - * option) any later version. |
---|
8 | 4 | */ |
---|
9 | 5 | #include <linux/export.h> |
---|
10 | 6 | #include <linux/phy.h> |
---|
| 7 | +#include <linux/of.h> |
---|
11 | 8 | |
---|
| 9 | +/** |
---|
| 10 | + * phy_speed_to_str - Return a string representing the PHY link speed |
---|
| 11 | + * |
---|
| 12 | + * @speed: Speed of the link |
---|
| 13 | + */ |
---|
12 | 14 | const char *phy_speed_to_str(int speed) |
---|
13 | 15 | { |
---|
| 16 | + BUILD_BUG_ON_MSG(__ETHTOOL_LINK_MODE_MASK_NBITS != 92, |
---|
| 17 | + "Enum ethtool_link_mode_bit_indices and phylib are out of sync. " |
---|
| 18 | + "If a speed or mode has been added please update phy_speed_to_str " |
---|
| 19 | + "and the PHY settings array.\n"); |
---|
| 20 | + |
---|
14 | 21 | switch (speed) { |
---|
15 | 22 | case SPEED_10: |
---|
16 | 23 | return "10Mbps"; |
---|
.. | .. |
---|
38 | 45 | return "56Gbps"; |
---|
39 | 46 | case SPEED_100000: |
---|
40 | 47 | return "100Gbps"; |
---|
| 48 | + case SPEED_200000: |
---|
| 49 | + return "200Gbps"; |
---|
| 50 | + case SPEED_400000: |
---|
| 51 | + return "400Gbps"; |
---|
41 | 52 | case SPEED_UNKNOWN: |
---|
42 | 53 | return "Unknown"; |
---|
43 | 54 | default: |
---|
.. | .. |
---|
46 | 57 | } |
---|
47 | 58 | EXPORT_SYMBOL_GPL(phy_speed_to_str); |
---|
48 | 59 | |
---|
| 60 | +/** |
---|
| 61 | + * phy_duplex_to_str - Return string describing the duplex |
---|
| 62 | + * |
---|
| 63 | + * @duplex: Duplex setting to describe |
---|
| 64 | + */ |
---|
49 | 65 | const char *phy_duplex_to_str(unsigned int duplex) |
---|
50 | 66 | { |
---|
51 | 67 | if (duplex == DUPLEX_HALF) |
---|
.. | .. |
---|
61 | 77 | /* A mapping of all SUPPORTED settings to speed/duplex. This table |
---|
62 | 78 | * must be grouped by speed and sorted in descending match priority |
---|
63 | 79 | * - iow, descending speed. */ |
---|
| 80 | + |
---|
| 81 | +#define PHY_SETTING(s, d, b) { .speed = SPEED_ ## s, .duplex = DUPLEX_ ## d, \ |
---|
| 82 | + .bit = ETHTOOL_LINK_MODE_ ## b ## _BIT} |
---|
| 83 | + |
---|
64 | 84 | static const struct phy_setting settings[] = { |
---|
65 | | - { |
---|
66 | | - .speed = SPEED_10000, |
---|
67 | | - .duplex = DUPLEX_FULL, |
---|
68 | | - .bit = ETHTOOL_LINK_MODE_10000baseKR_Full_BIT, |
---|
69 | | - }, |
---|
70 | | - { |
---|
71 | | - .speed = SPEED_10000, |
---|
72 | | - .duplex = DUPLEX_FULL, |
---|
73 | | - .bit = ETHTOOL_LINK_MODE_10000baseKX4_Full_BIT, |
---|
74 | | - }, |
---|
75 | | - { |
---|
76 | | - .speed = SPEED_10000, |
---|
77 | | - .duplex = DUPLEX_FULL, |
---|
78 | | - .bit = ETHTOOL_LINK_MODE_10000baseT_Full_BIT, |
---|
79 | | - }, |
---|
80 | | - { |
---|
81 | | - .speed = SPEED_2500, |
---|
82 | | - .duplex = DUPLEX_FULL, |
---|
83 | | - .bit = ETHTOOL_LINK_MODE_2500baseX_Full_BIT, |
---|
84 | | - }, |
---|
85 | | - { |
---|
86 | | - .speed = SPEED_1000, |
---|
87 | | - .duplex = DUPLEX_FULL, |
---|
88 | | - .bit = ETHTOOL_LINK_MODE_1000baseKX_Full_BIT, |
---|
89 | | - }, |
---|
90 | | - { |
---|
91 | | - .speed = SPEED_1000, |
---|
92 | | - .duplex = DUPLEX_FULL, |
---|
93 | | - .bit = ETHTOOL_LINK_MODE_1000baseX_Full_BIT, |
---|
94 | | - }, |
---|
95 | | - { |
---|
96 | | - .speed = SPEED_1000, |
---|
97 | | - .duplex = DUPLEX_FULL, |
---|
98 | | - .bit = ETHTOOL_LINK_MODE_1000baseT_Full_BIT, |
---|
99 | | - }, |
---|
100 | | - { |
---|
101 | | - .speed = SPEED_1000, |
---|
102 | | - .duplex = DUPLEX_HALF, |
---|
103 | | - .bit = ETHTOOL_LINK_MODE_1000baseT_Half_BIT, |
---|
104 | | - }, |
---|
105 | | - { |
---|
106 | | - .speed = SPEED_100, |
---|
107 | | - .duplex = DUPLEX_FULL, |
---|
108 | | - .bit = ETHTOOL_LINK_MODE_100baseT_Full_BIT, |
---|
109 | | - }, |
---|
110 | | - { |
---|
111 | | - .speed = SPEED_100, |
---|
112 | | - .duplex = DUPLEX_HALF, |
---|
113 | | - .bit = ETHTOOL_LINK_MODE_100baseT_Half_BIT, |
---|
114 | | - }, |
---|
115 | | - { |
---|
116 | | - .speed = SPEED_10, |
---|
117 | | - .duplex = DUPLEX_FULL, |
---|
118 | | - .bit = ETHTOOL_LINK_MODE_10baseT_Full_BIT, |
---|
119 | | - }, |
---|
120 | | - { |
---|
121 | | - .speed = SPEED_10, |
---|
122 | | - .duplex = DUPLEX_HALF, |
---|
123 | | - .bit = ETHTOOL_LINK_MODE_10baseT_Half_BIT, |
---|
124 | | - }, |
---|
| 85 | + /* 400G */ |
---|
| 86 | + PHY_SETTING( 400000, FULL, 400000baseCR8_Full ), |
---|
| 87 | + PHY_SETTING( 400000, FULL, 400000baseKR8_Full ), |
---|
| 88 | + PHY_SETTING( 400000, FULL, 400000baseLR8_ER8_FR8_Full ), |
---|
| 89 | + PHY_SETTING( 400000, FULL, 400000baseDR8_Full ), |
---|
| 90 | + PHY_SETTING( 400000, FULL, 400000baseSR8_Full ), |
---|
| 91 | + PHY_SETTING( 400000, FULL, 400000baseCR4_Full ), |
---|
| 92 | + PHY_SETTING( 400000, FULL, 400000baseKR4_Full ), |
---|
| 93 | + PHY_SETTING( 400000, FULL, 400000baseLR4_ER4_FR4_Full ), |
---|
| 94 | + PHY_SETTING( 400000, FULL, 400000baseDR4_Full ), |
---|
| 95 | + PHY_SETTING( 400000, FULL, 400000baseSR4_Full ), |
---|
| 96 | + /* 200G */ |
---|
| 97 | + PHY_SETTING( 200000, FULL, 200000baseCR4_Full ), |
---|
| 98 | + PHY_SETTING( 200000, FULL, 200000baseKR4_Full ), |
---|
| 99 | + PHY_SETTING( 200000, FULL, 200000baseLR4_ER4_FR4_Full ), |
---|
| 100 | + PHY_SETTING( 200000, FULL, 200000baseDR4_Full ), |
---|
| 101 | + PHY_SETTING( 200000, FULL, 200000baseSR4_Full ), |
---|
| 102 | + PHY_SETTING( 200000, FULL, 200000baseCR2_Full ), |
---|
| 103 | + PHY_SETTING( 200000, FULL, 200000baseKR2_Full ), |
---|
| 104 | + PHY_SETTING( 200000, FULL, 200000baseLR2_ER2_FR2_Full ), |
---|
| 105 | + PHY_SETTING( 200000, FULL, 200000baseDR2_Full ), |
---|
| 106 | + PHY_SETTING( 200000, FULL, 200000baseSR2_Full ), |
---|
| 107 | + /* 100G */ |
---|
| 108 | + PHY_SETTING( 100000, FULL, 100000baseCR4_Full ), |
---|
| 109 | + PHY_SETTING( 100000, FULL, 100000baseKR4_Full ), |
---|
| 110 | + PHY_SETTING( 100000, FULL, 100000baseLR4_ER4_Full ), |
---|
| 111 | + PHY_SETTING( 100000, FULL, 100000baseSR4_Full ), |
---|
| 112 | + PHY_SETTING( 100000, FULL, 100000baseCR2_Full ), |
---|
| 113 | + PHY_SETTING( 100000, FULL, 100000baseKR2_Full ), |
---|
| 114 | + PHY_SETTING( 100000, FULL, 100000baseLR2_ER2_FR2_Full ), |
---|
| 115 | + PHY_SETTING( 100000, FULL, 100000baseDR2_Full ), |
---|
| 116 | + PHY_SETTING( 100000, FULL, 100000baseSR2_Full ), |
---|
| 117 | + PHY_SETTING( 100000, FULL, 100000baseCR_Full ), |
---|
| 118 | + PHY_SETTING( 100000, FULL, 100000baseKR_Full ), |
---|
| 119 | + PHY_SETTING( 100000, FULL, 100000baseLR_ER_FR_Full ), |
---|
| 120 | + PHY_SETTING( 100000, FULL, 100000baseDR_Full ), |
---|
| 121 | + PHY_SETTING( 100000, FULL, 100000baseSR_Full ), |
---|
| 122 | + /* 56G */ |
---|
| 123 | + PHY_SETTING( 56000, FULL, 56000baseCR4_Full ), |
---|
| 124 | + PHY_SETTING( 56000, FULL, 56000baseKR4_Full ), |
---|
| 125 | + PHY_SETTING( 56000, FULL, 56000baseLR4_Full ), |
---|
| 126 | + PHY_SETTING( 56000, FULL, 56000baseSR4_Full ), |
---|
| 127 | + /* 50G */ |
---|
| 128 | + PHY_SETTING( 50000, FULL, 50000baseCR2_Full ), |
---|
| 129 | + PHY_SETTING( 50000, FULL, 50000baseKR2_Full ), |
---|
| 130 | + PHY_SETTING( 50000, FULL, 50000baseSR2_Full ), |
---|
| 131 | + PHY_SETTING( 50000, FULL, 50000baseCR_Full ), |
---|
| 132 | + PHY_SETTING( 50000, FULL, 50000baseKR_Full ), |
---|
| 133 | + PHY_SETTING( 50000, FULL, 50000baseLR_ER_FR_Full ), |
---|
| 134 | + PHY_SETTING( 50000, FULL, 50000baseDR_Full ), |
---|
| 135 | + PHY_SETTING( 50000, FULL, 50000baseSR_Full ), |
---|
| 136 | + /* 40G */ |
---|
| 137 | + PHY_SETTING( 40000, FULL, 40000baseCR4_Full ), |
---|
| 138 | + PHY_SETTING( 40000, FULL, 40000baseKR4_Full ), |
---|
| 139 | + PHY_SETTING( 40000, FULL, 40000baseLR4_Full ), |
---|
| 140 | + PHY_SETTING( 40000, FULL, 40000baseSR4_Full ), |
---|
| 141 | + /* 25G */ |
---|
| 142 | + PHY_SETTING( 25000, FULL, 25000baseCR_Full ), |
---|
| 143 | + PHY_SETTING( 25000, FULL, 25000baseKR_Full ), |
---|
| 144 | + PHY_SETTING( 25000, FULL, 25000baseSR_Full ), |
---|
| 145 | + /* 20G */ |
---|
| 146 | + PHY_SETTING( 20000, FULL, 20000baseKR2_Full ), |
---|
| 147 | + PHY_SETTING( 20000, FULL, 20000baseMLD2_Full ), |
---|
| 148 | + /* 10G */ |
---|
| 149 | + PHY_SETTING( 10000, FULL, 10000baseCR_Full ), |
---|
| 150 | + PHY_SETTING( 10000, FULL, 10000baseER_Full ), |
---|
| 151 | + PHY_SETTING( 10000, FULL, 10000baseKR_Full ), |
---|
| 152 | + PHY_SETTING( 10000, FULL, 10000baseKX4_Full ), |
---|
| 153 | + PHY_SETTING( 10000, FULL, 10000baseLR_Full ), |
---|
| 154 | + PHY_SETTING( 10000, FULL, 10000baseLRM_Full ), |
---|
| 155 | + PHY_SETTING( 10000, FULL, 10000baseR_FEC ), |
---|
| 156 | + PHY_SETTING( 10000, FULL, 10000baseSR_Full ), |
---|
| 157 | + PHY_SETTING( 10000, FULL, 10000baseT_Full ), |
---|
| 158 | + /* 5G */ |
---|
| 159 | + PHY_SETTING( 5000, FULL, 5000baseT_Full ), |
---|
| 160 | + /* 2.5G */ |
---|
| 161 | + PHY_SETTING( 2500, FULL, 2500baseT_Full ), |
---|
| 162 | + PHY_SETTING( 2500, FULL, 2500baseX_Full ), |
---|
| 163 | + /* 1G */ |
---|
| 164 | + PHY_SETTING( 1000, FULL, 1000baseT_Full ), |
---|
| 165 | + PHY_SETTING( 1000, HALF, 1000baseT_Half ), |
---|
| 166 | + PHY_SETTING( 1000, FULL, 1000baseT1_Full ), |
---|
| 167 | + PHY_SETTING( 1000, FULL, 1000baseX_Full ), |
---|
| 168 | + PHY_SETTING( 1000, FULL, 1000baseKX_Full ), |
---|
| 169 | + /* 100M */ |
---|
| 170 | + PHY_SETTING( 100, FULL, 100baseT_Full ), |
---|
| 171 | + PHY_SETTING( 100, FULL, 100baseT1_Full ), |
---|
| 172 | + PHY_SETTING( 100, HALF, 100baseT_Half ), |
---|
| 173 | + PHY_SETTING( 100, HALF, 100baseFX_Half ), |
---|
| 174 | + PHY_SETTING( 100, FULL, 100baseFX_Full ), |
---|
| 175 | + /* 10M */ |
---|
| 176 | + PHY_SETTING( 10, FULL, 10baseT_Full ), |
---|
| 177 | + PHY_SETTING( 10, HALF, 10baseT_Half ), |
---|
125 | 178 | }; |
---|
| 179 | +#undef PHY_SETTING |
---|
126 | 180 | |
---|
127 | 181 | /** |
---|
128 | 182 | * phy_lookup_setting - lookup a PHY setting |
---|
129 | 183 | * @speed: speed to match |
---|
130 | 184 | * @duplex: duplex to match |
---|
131 | 185 | * @mask: allowed link modes |
---|
132 | | - * @maxbit: bit size of link modes |
---|
133 | 186 | * @exact: an exact match is required |
---|
134 | 187 | * |
---|
135 | 188 | * Search the settings array for a setting that matches the speed and |
---|
.. | .. |
---|
143 | 196 | * they all fail, %NULL will be returned. |
---|
144 | 197 | */ |
---|
145 | 198 | const struct phy_setting * |
---|
146 | | -phy_lookup_setting(int speed, int duplex, const unsigned long *mask, |
---|
147 | | - size_t maxbit, bool exact) |
---|
| 199 | +phy_lookup_setting(int speed, int duplex, const unsigned long *mask, bool exact) |
---|
148 | 200 | { |
---|
149 | 201 | const struct phy_setting *p, *match = NULL, *last = NULL; |
---|
150 | 202 | int i; |
---|
151 | 203 | |
---|
152 | 204 | for (i = 0, p = settings; i < ARRAY_SIZE(settings); i++, p++) { |
---|
153 | | - if (p->bit < maxbit && test_bit(p->bit, mask)) { |
---|
| 205 | + if (p->bit < __ETHTOOL_LINK_MODE_MASK_NBITS && |
---|
| 206 | + test_bit(p->bit, mask)) { |
---|
154 | 207 | last = p; |
---|
155 | 208 | if (p->speed == speed && p->duplex == duplex) { |
---|
156 | 209 | /* Exact match for speed and duplex */ |
---|
.. | .. |
---|
175 | 228 | EXPORT_SYMBOL_GPL(phy_lookup_setting); |
---|
176 | 229 | |
---|
177 | 230 | size_t phy_speeds(unsigned int *speeds, size_t size, |
---|
178 | | - unsigned long *mask, size_t maxbit) |
---|
| 231 | + unsigned long *mask) |
---|
179 | 232 | { |
---|
180 | 233 | size_t count; |
---|
181 | 234 | int i; |
---|
182 | 235 | |
---|
183 | 236 | for (i = 0, count = 0; i < ARRAY_SIZE(settings) && count < size; i++) |
---|
184 | | - if (settings[i].bit < maxbit && |
---|
| 237 | + if (settings[i].bit < __ETHTOOL_LINK_MODE_MASK_NBITS && |
---|
185 | 238 | test_bit(settings[i].bit, mask) && |
---|
186 | 239 | (count == 0 || speeds[count - 1] != settings[i].speed)) |
---|
187 | 240 | speeds[count++] = settings[i].speed; |
---|
.. | .. |
---|
189 | 242 | return count; |
---|
190 | 243 | } |
---|
191 | 244 | |
---|
| 245 | +static int __set_linkmode_max_speed(u32 max_speed, unsigned long *addr) |
---|
| 246 | +{ |
---|
| 247 | + const struct phy_setting *p; |
---|
| 248 | + int i; |
---|
| 249 | + |
---|
| 250 | + for (i = 0, p = settings; i < ARRAY_SIZE(settings); i++, p++) { |
---|
| 251 | + if (p->speed > max_speed) |
---|
| 252 | + linkmode_clear_bit(p->bit, addr); |
---|
| 253 | + else |
---|
| 254 | + break; |
---|
| 255 | + } |
---|
| 256 | + |
---|
| 257 | + return 0; |
---|
| 258 | +} |
---|
| 259 | + |
---|
| 260 | +static int __set_phy_supported(struct phy_device *phydev, u32 max_speed) |
---|
| 261 | +{ |
---|
| 262 | + return __set_linkmode_max_speed(max_speed, phydev->supported); |
---|
| 263 | +} |
---|
| 264 | + |
---|
192 | 265 | /** |
---|
193 | | - * phy_resolve_aneg_linkmode - resolve the advertisements into phy settings |
---|
| 266 | + * phy_set_max_speed - Set the maximum speed the PHY should support |
---|
| 267 | + * |
---|
| 268 | + * @phydev: The phy_device struct |
---|
| 269 | + * @max_speed: Maximum speed |
---|
| 270 | + * |
---|
| 271 | + * The PHY might be more capable than the MAC. For example a Fast Ethernet |
---|
| 272 | + * is connected to a 1G PHY. This function allows the MAC to indicate its |
---|
| 273 | + * maximum speed, and so limit what the PHY will advertise. |
---|
| 274 | + */ |
---|
| 275 | +int phy_set_max_speed(struct phy_device *phydev, u32 max_speed) |
---|
| 276 | +{ |
---|
| 277 | + int err; |
---|
| 278 | + |
---|
| 279 | + err = __set_phy_supported(phydev, max_speed); |
---|
| 280 | + if (err) |
---|
| 281 | + return err; |
---|
| 282 | + |
---|
| 283 | + phy_advertise_supported(phydev); |
---|
| 284 | + |
---|
| 285 | + return 0; |
---|
| 286 | +} |
---|
| 287 | +EXPORT_SYMBOL(phy_set_max_speed); |
---|
| 288 | + |
---|
| 289 | +void of_set_phy_supported(struct phy_device *phydev) |
---|
| 290 | +{ |
---|
| 291 | + struct device_node *node = phydev->mdio.dev.of_node; |
---|
| 292 | + u32 max_speed; |
---|
| 293 | + |
---|
| 294 | + if (!IS_ENABLED(CONFIG_OF_MDIO)) |
---|
| 295 | + return; |
---|
| 296 | + |
---|
| 297 | + if (!node) |
---|
| 298 | + return; |
---|
| 299 | + |
---|
| 300 | + if (!of_property_read_u32(node, "max-speed", &max_speed)) |
---|
| 301 | + __set_phy_supported(phydev, max_speed); |
---|
| 302 | +} |
---|
| 303 | + |
---|
| 304 | +void of_set_phy_eee_broken(struct phy_device *phydev) |
---|
| 305 | +{ |
---|
| 306 | + struct device_node *node = phydev->mdio.dev.of_node; |
---|
| 307 | + u32 broken = 0; |
---|
| 308 | + |
---|
| 309 | + if (!IS_ENABLED(CONFIG_OF_MDIO)) |
---|
| 310 | + return; |
---|
| 311 | + |
---|
| 312 | + if (!node) |
---|
| 313 | + return; |
---|
| 314 | + |
---|
| 315 | + if (of_property_read_bool(node, "eee-broken-100tx")) |
---|
| 316 | + broken |= MDIO_EEE_100TX; |
---|
| 317 | + if (of_property_read_bool(node, "eee-broken-1000t")) |
---|
| 318 | + broken |= MDIO_EEE_1000T; |
---|
| 319 | + if (of_property_read_bool(node, "eee-broken-10gt")) |
---|
| 320 | + broken |= MDIO_EEE_10GT; |
---|
| 321 | + if (of_property_read_bool(node, "eee-broken-1000kx")) |
---|
| 322 | + broken |= MDIO_EEE_1000KX; |
---|
| 323 | + if (of_property_read_bool(node, "eee-broken-10gkx4")) |
---|
| 324 | + broken |= MDIO_EEE_10GKX4; |
---|
| 325 | + if (of_property_read_bool(node, "eee-broken-10gkr")) |
---|
| 326 | + broken |= MDIO_EEE_10GKR; |
---|
| 327 | + |
---|
| 328 | + phydev->eee_broken_modes = broken; |
---|
| 329 | +} |
---|
| 330 | + |
---|
| 331 | +/** |
---|
| 332 | + * phy_resolve_aneg_pause - Determine pause autoneg results |
---|
| 333 | + * |
---|
| 334 | + * @phydev: The phy_device struct |
---|
| 335 | + * |
---|
| 336 | + * Once autoneg has completed the local pause settings can be |
---|
| 337 | + * resolved. Determine if pause and asymmetric pause should be used |
---|
| 338 | + * by the MAC. |
---|
| 339 | + */ |
---|
| 340 | + |
---|
| 341 | +void phy_resolve_aneg_pause(struct phy_device *phydev) |
---|
| 342 | +{ |
---|
| 343 | + if (phydev->duplex == DUPLEX_FULL) { |
---|
| 344 | + phydev->pause = linkmode_test_bit(ETHTOOL_LINK_MODE_Pause_BIT, |
---|
| 345 | + phydev->lp_advertising); |
---|
| 346 | + phydev->asym_pause = linkmode_test_bit( |
---|
| 347 | + ETHTOOL_LINK_MODE_Asym_Pause_BIT, |
---|
| 348 | + phydev->lp_advertising); |
---|
| 349 | + } |
---|
| 350 | +} |
---|
| 351 | +EXPORT_SYMBOL_GPL(phy_resolve_aneg_pause); |
---|
| 352 | + |
---|
| 353 | +/** |
---|
| 354 | + * phy_resolve_aneg_linkmode - resolve the advertisements into PHY settings |
---|
194 | 355 | * @phydev: The phy_device struct |
---|
195 | 356 | * |
---|
196 | 357 | * Resolve our and the link partner advertisements into their corresponding |
---|
.. | .. |
---|
199 | 360 | */ |
---|
200 | 361 | void phy_resolve_aneg_linkmode(struct phy_device *phydev) |
---|
201 | 362 | { |
---|
202 | | - u32 common = phydev->lp_advertising & phydev->advertising; |
---|
| 363 | + __ETHTOOL_DECLARE_LINK_MODE_MASK(common); |
---|
| 364 | + int i; |
---|
203 | 365 | |
---|
204 | | - if (common & ADVERTISED_10000baseT_Full) { |
---|
205 | | - phydev->speed = SPEED_10000; |
---|
206 | | - phydev->duplex = DUPLEX_FULL; |
---|
207 | | - } else if (common & ADVERTISED_1000baseT_Full) { |
---|
208 | | - phydev->speed = SPEED_1000; |
---|
209 | | - phydev->duplex = DUPLEX_FULL; |
---|
210 | | - } else if (common & ADVERTISED_1000baseT_Half) { |
---|
211 | | - phydev->speed = SPEED_1000; |
---|
212 | | - phydev->duplex = DUPLEX_HALF; |
---|
213 | | - } else if (common & ADVERTISED_100baseT_Full) { |
---|
214 | | - phydev->speed = SPEED_100; |
---|
215 | | - phydev->duplex = DUPLEX_FULL; |
---|
216 | | - } else if (common & ADVERTISED_100baseT_Half) { |
---|
217 | | - phydev->speed = SPEED_100; |
---|
218 | | - phydev->duplex = DUPLEX_HALF; |
---|
219 | | - } else if (common & ADVERTISED_10baseT_Full) { |
---|
220 | | - phydev->speed = SPEED_10; |
---|
221 | | - phydev->duplex = DUPLEX_FULL; |
---|
222 | | - } else if (common & ADVERTISED_10baseT_Half) { |
---|
223 | | - phydev->speed = SPEED_10; |
---|
224 | | - phydev->duplex = DUPLEX_HALF; |
---|
225 | | - } |
---|
| 366 | + linkmode_and(common, phydev->lp_advertising, phydev->advertising); |
---|
226 | 367 | |
---|
227 | | - if (phydev->duplex == DUPLEX_FULL) { |
---|
228 | | - phydev->pause = !!(phydev->lp_advertising & ADVERTISED_Pause); |
---|
229 | | - phydev->asym_pause = !!(phydev->lp_advertising & |
---|
230 | | - ADVERTISED_Asym_Pause); |
---|
231 | | - } |
---|
| 368 | + for (i = 0; i < ARRAY_SIZE(settings); i++) |
---|
| 369 | + if (test_bit(settings[i].bit, common)) { |
---|
| 370 | + phydev->speed = settings[i].speed; |
---|
| 371 | + phydev->duplex = settings[i].duplex; |
---|
| 372 | + break; |
---|
| 373 | + } |
---|
| 374 | + |
---|
| 375 | + phy_resolve_aneg_pause(phydev); |
---|
232 | 376 | } |
---|
233 | 377 | EXPORT_SYMBOL_GPL(phy_resolve_aneg_linkmode); |
---|
| 378 | + |
---|
| 379 | +/** |
---|
| 380 | + * phy_check_downshift - check whether downshift occurred |
---|
| 381 | + * @phydev: The phy_device struct |
---|
| 382 | + * |
---|
| 383 | + * Check whether a downshift to a lower speed occurred. If this should be the |
---|
| 384 | + * case warn the user. |
---|
| 385 | + * Prerequisite for detecting downshift is that PHY driver implements the |
---|
| 386 | + * read_status callback and sets phydev->speed to the actual link speed. |
---|
| 387 | + */ |
---|
| 388 | +void phy_check_downshift(struct phy_device *phydev) |
---|
| 389 | +{ |
---|
| 390 | + __ETHTOOL_DECLARE_LINK_MODE_MASK(common); |
---|
| 391 | + int i, speed = SPEED_UNKNOWN; |
---|
| 392 | + |
---|
| 393 | + phydev->downshifted_rate = 0; |
---|
| 394 | + |
---|
| 395 | + if (phydev->autoneg == AUTONEG_DISABLE || |
---|
| 396 | + phydev->speed == SPEED_UNKNOWN) |
---|
| 397 | + return; |
---|
| 398 | + |
---|
| 399 | + linkmode_and(common, phydev->lp_advertising, phydev->advertising); |
---|
| 400 | + |
---|
| 401 | + for (i = 0; i < ARRAY_SIZE(settings); i++) |
---|
| 402 | + if (test_bit(settings[i].bit, common)) { |
---|
| 403 | + speed = settings[i].speed; |
---|
| 404 | + break; |
---|
| 405 | + } |
---|
| 406 | + |
---|
| 407 | + if (speed == SPEED_UNKNOWN || phydev->speed >= speed) |
---|
| 408 | + return; |
---|
| 409 | + |
---|
| 410 | + phydev_warn(phydev, "Downshift occurred from negotiated speed %s to actual speed %s, check cabling!\n", |
---|
| 411 | + phy_speed_to_str(speed), phy_speed_to_str(phydev->speed)); |
---|
| 412 | + |
---|
| 413 | + phydev->downshifted_rate = 1; |
---|
| 414 | +} |
---|
| 415 | +EXPORT_SYMBOL_GPL(phy_check_downshift); |
---|
| 416 | + |
---|
| 417 | +static int phy_resolve_min_speed(struct phy_device *phydev, bool fdx_only) |
---|
| 418 | +{ |
---|
| 419 | + __ETHTOOL_DECLARE_LINK_MODE_MASK(common); |
---|
| 420 | + int i = ARRAY_SIZE(settings); |
---|
| 421 | + |
---|
| 422 | + linkmode_and(common, phydev->lp_advertising, phydev->advertising); |
---|
| 423 | + |
---|
| 424 | + while (--i >= 0) { |
---|
| 425 | + if (test_bit(settings[i].bit, common)) { |
---|
| 426 | + if (fdx_only && settings[i].duplex != DUPLEX_FULL) |
---|
| 427 | + continue; |
---|
| 428 | + return settings[i].speed; |
---|
| 429 | + } |
---|
| 430 | + } |
---|
| 431 | + |
---|
| 432 | + return SPEED_UNKNOWN; |
---|
| 433 | +} |
---|
| 434 | + |
---|
| 435 | +int phy_speed_down_core(struct phy_device *phydev) |
---|
| 436 | +{ |
---|
| 437 | + int min_common_speed = phy_resolve_min_speed(phydev, true); |
---|
| 438 | + |
---|
| 439 | + if (min_common_speed == SPEED_UNKNOWN) |
---|
| 440 | + return -EINVAL; |
---|
| 441 | + |
---|
| 442 | + return __set_linkmode_max_speed(min_common_speed, phydev->advertising); |
---|
| 443 | +} |
---|
234 | 444 | |
---|
235 | 445 | static void mmd_phy_indirect(struct mii_bus *bus, int phy_addr, int devad, |
---|
236 | 446 | u16 regnum) |
---|
.. | .. |
---|
247 | 457 | } |
---|
248 | 458 | |
---|
249 | 459 | /** |
---|
250 | | - * phy_read_mmd - Convenience function for reading a register |
---|
| 460 | + * __phy_read_mmd - Convenience function for reading a register |
---|
251 | 461 | * from an MMD on a given PHY. |
---|
252 | 462 | * @phydev: The phy_device struct |
---|
253 | 463 | * @devad: The MMD to read from (0..31) |
---|
254 | 464 | * @regnum: The register on the MMD to read (0..65535) |
---|
255 | 465 | * |
---|
256 | | - * Same rules as for phy_read(); |
---|
| 466 | + * Same rules as for __phy_read(); |
---|
257 | 467 | */ |
---|
258 | | -int phy_read_mmd(struct phy_device *phydev, int devad, u32 regnum) |
---|
| 468 | +int __phy_read_mmd(struct phy_device *phydev, int devad, u32 regnum) |
---|
259 | 469 | { |
---|
260 | 470 | int val; |
---|
261 | 471 | |
---|
262 | 472 | if (regnum > (u16)~0 || devad > 32) |
---|
263 | 473 | return -EINVAL; |
---|
264 | 474 | |
---|
265 | | - if (phydev->drv->read_mmd) { |
---|
| 475 | + if (phydev->drv && phydev->drv->read_mmd) { |
---|
266 | 476 | val = phydev->drv->read_mmd(phydev, devad, regnum); |
---|
267 | 477 | } else if (phydev->is_c45) { |
---|
268 | | - u32 addr = MII_ADDR_C45 | (devad << 16) | (regnum & 0xffff); |
---|
269 | | - |
---|
270 | | - val = mdiobus_read(phydev->mdio.bus, phydev->mdio.addr, addr); |
---|
| 478 | + val = __mdiobus_c45_read(phydev->mdio.bus, phydev->mdio.addr, |
---|
| 479 | + devad, regnum); |
---|
271 | 480 | } else { |
---|
272 | 481 | struct mii_bus *bus = phydev->mdio.bus; |
---|
273 | 482 | int phy_addr = phydev->mdio.addr; |
---|
274 | 483 | |
---|
275 | | - mutex_lock(&bus->mdio_lock); |
---|
276 | 484 | mmd_phy_indirect(bus, phy_addr, devad, regnum); |
---|
277 | 485 | |
---|
278 | 486 | /* Read the content of the MMD's selected register */ |
---|
279 | 487 | val = __mdiobus_read(bus, phy_addr, MII_MMD_DATA); |
---|
280 | | - mutex_unlock(&bus->mdio_lock); |
---|
281 | 488 | } |
---|
282 | 489 | return val; |
---|
283 | 490 | } |
---|
| 491 | +EXPORT_SYMBOL(__phy_read_mmd); |
---|
| 492 | + |
---|
| 493 | +/** |
---|
| 494 | + * phy_read_mmd - Convenience function for reading a register |
---|
| 495 | + * from an MMD on a given PHY. |
---|
| 496 | + * @phydev: The phy_device struct |
---|
| 497 | + * @devad: The MMD to read from |
---|
| 498 | + * @regnum: The register on the MMD to read |
---|
| 499 | + * |
---|
| 500 | + * Same rules as for phy_read(); |
---|
| 501 | + */ |
---|
| 502 | +int phy_read_mmd(struct phy_device *phydev, int devad, u32 regnum) |
---|
| 503 | +{ |
---|
| 504 | + int ret; |
---|
| 505 | + |
---|
| 506 | + phy_lock_mdio_bus(phydev); |
---|
| 507 | + ret = __phy_read_mmd(phydev, devad, regnum); |
---|
| 508 | + phy_unlock_mdio_bus(phydev); |
---|
| 509 | + |
---|
| 510 | + return ret; |
---|
| 511 | +} |
---|
284 | 512 | EXPORT_SYMBOL(phy_read_mmd); |
---|
| 513 | + |
---|
| 514 | +/** |
---|
| 515 | + * __phy_write_mmd - Convenience function for writing a register |
---|
| 516 | + * on an MMD on a given PHY. |
---|
| 517 | + * @phydev: The phy_device struct |
---|
| 518 | + * @devad: The MMD to read from |
---|
| 519 | + * @regnum: The register on the MMD to read |
---|
| 520 | + * @val: value to write to @regnum |
---|
| 521 | + * |
---|
| 522 | + * Same rules as for __phy_write(); |
---|
| 523 | + */ |
---|
| 524 | +int __phy_write_mmd(struct phy_device *phydev, int devad, u32 regnum, u16 val) |
---|
| 525 | +{ |
---|
| 526 | + int ret; |
---|
| 527 | + |
---|
| 528 | + if (regnum > (u16)~0 || devad > 32) |
---|
| 529 | + return -EINVAL; |
---|
| 530 | + |
---|
| 531 | + if (phydev->drv && phydev->drv->write_mmd) { |
---|
| 532 | + ret = phydev->drv->write_mmd(phydev, devad, regnum, val); |
---|
| 533 | + } else if (phydev->is_c45) { |
---|
| 534 | + ret = __mdiobus_c45_write(phydev->mdio.bus, phydev->mdio.addr, |
---|
| 535 | + devad, regnum, val); |
---|
| 536 | + } else { |
---|
| 537 | + struct mii_bus *bus = phydev->mdio.bus; |
---|
| 538 | + int phy_addr = phydev->mdio.addr; |
---|
| 539 | + |
---|
| 540 | + mmd_phy_indirect(bus, phy_addr, devad, regnum); |
---|
| 541 | + |
---|
| 542 | + /* Write the data into MMD's selected register */ |
---|
| 543 | + __mdiobus_write(bus, phy_addr, MII_MMD_DATA, val); |
---|
| 544 | + |
---|
| 545 | + ret = 0; |
---|
| 546 | + } |
---|
| 547 | + return ret; |
---|
| 548 | +} |
---|
| 549 | +EXPORT_SYMBOL(__phy_write_mmd); |
---|
285 | 550 | |
---|
286 | 551 | /** |
---|
287 | 552 | * phy_write_mmd - Convenience function for writing a register |
---|
.. | .. |
---|
297 | 562 | { |
---|
298 | 563 | int ret; |
---|
299 | 564 | |
---|
300 | | - if (regnum > (u16)~0 || devad > 32) |
---|
301 | | - return -EINVAL; |
---|
| 565 | + phy_lock_mdio_bus(phydev); |
---|
| 566 | + ret = __phy_write_mmd(phydev, devad, regnum, val); |
---|
| 567 | + phy_unlock_mdio_bus(phydev); |
---|
302 | 568 | |
---|
303 | | - if (phydev->drv->write_mmd) { |
---|
304 | | - ret = phydev->drv->write_mmd(phydev, devad, regnum, val); |
---|
305 | | - } else if (phydev->is_c45) { |
---|
306 | | - u32 addr = MII_ADDR_C45 | (devad << 16) | (regnum & 0xffff); |
---|
307 | | - |
---|
308 | | - ret = mdiobus_write(phydev->mdio.bus, phydev->mdio.addr, |
---|
309 | | - addr, val); |
---|
310 | | - } else { |
---|
311 | | - struct mii_bus *bus = phydev->mdio.bus; |
---|
312 | | - int phy_addr = phydev->mdio.addr; |
---|
313 | | - |
---|
314 | | - mutex_lock(&bus->mdio_lock); |
---|
315 | | - mmd_phy_indirect(bus, phy_addr, devad, regnum); |
---|
316 | | - |
---|
317 | | - /* Write the data into MMD's selected register */ |
---|
318 | | - __mdiobus_write(bus, phy_addr, MII_MMD_DATA, val); |
---|
319 | | - mutex_unlock(&bus->mdio_lock); |
---|
320 | | - |
---|
321 | | - ret = 0; |
---|
322 | | - } |
---|
323 | 569 | return ret; |
---|
324 | 570 | } |
---|
325 | 571 | EXPORT_SYMBOL(phy_write_mmd); |
---|
326 | 572 | |
---|
327 | 573 | /** |
---|
328 | | - * __phy_modify() - Convenience function for modifying a PHY register |
---|
329 | | - * @phydev: a pointer to a &struct phy_device |
---|
330 | | - * @regnum: register number |
---|
| 574 | + * phy_modify_changed - Function for modifying a PHY register |
---|
| 575 | + * @phydev: the phy_device struct |
---|
| 576 | + * @regnum: register number to modify |
---|
331 | 577 | * @mask: bit mask of bits to clear |
---|
332 | | - * @set: bit mask of bits to set |
---|
| 578 | + * @set: new value of bits set in mask to write to @regnum |
---|
333 | 579 | * |
---|
334 | | - * Unlocked helper function which allows a PHY register to be modified as |
---|
335 | | - * new register value = (old register value & ~mask) | set |
---|
| 580 | + * NOTE: MUST NOT be called from interrupt context, |
---|
| 581 | + * because the bus read/write functions may wait for an interrupt |
---|
| 582 | + * to conclude the operation. |
---|
| 583 | + * |
---|
| 584 | + * Returns negative errno, 0 if there was no change, and 1 in case of change |
---|
| 585 | + */ |
---|
| 586 | +int phy_modify_changed(struct phy_device *phydev, u32 regnum, u16 mask, u16 set) |
---|
| 587 | +{ |
---|
| 588 | + int ret; |
---|
| 589 | + |
---|
| 590 | + phy_lock_mdio_bus(phydev); |
---|
| 591 | + ret = __phy_modify_changed(phydev, regnum, mask, set); |
---|
| 592 | + phy_unlock_mdio_bus(phydev); |
---|
| 593 | + |
---|
| 594 | + return ret; |
---|
| 595 | +} |
---|
| 596 | +EXPORT_SYMBOL_GPL(phy_modify_changed); |
---|
| 597 | + |
---|
| 598 | +/** |
---|
| 599 | + * __phy_modify - Convenience function for modifying a PHY register |
---|
| 600 | + * @phydev: the phy_device struct |
---|
| 601 | + * @regnum: register number to modify |
---|
| 602 | + * @mask: bit mask of bits to clear |
---|
| 603 | + * @set: new value of bits set in mask to write to @regnum |
---|
| 604 | + * |
---|
| 605 | + * NOTE: MUST NOT be called from interrupt context, |
---|
| 606 | + * because the bus read/write functions may wait for an interrupt |
---|
| 607 | + * to conclude the operation. |
---|
336 | 608 | */ |
---|
337 | 609 | int __phy_modify(struct phy_device *phydev, u32 regnum, u16 mask, u16 set) |
---|
338 | 610 | { |
---|
339 | 611 | int ret; |
---|
340 | 612 | |
---|
341 | | - ret = __phy_read(phydev, regnum); |
---|
342 | | - if (ret < 0) |
---|
343 | | - return ret; |
---|
344 | | - |
---|
345 | | - ret = __phy_write(phydev, regnum, (ret & ~mask) | set); |
---|
| 613 | + ret = __phy_modify_changed(phydev, regnum, mask, set); |
---|
346 | 614 | |
---|
347 | 615 | return ret < 0 ? ret : 0; |
---|
348 | 616 | } |
---|
.. | .. |
---|
363 | 631 | { |
---|
364 | 632 | int ret; |
---|
365 | 633 | |
---|
366 | | - mutex_lock(&phydev->mdio.bus->mdio_lock); |
---|
| 634 | + phy_lock_mdio_bus(phydev); |
---|
367 | 635 | ret = __phy_modify(phydev, regnum, mask, set); |
---|
368 | | - mutex_unlock(&phydev->mdio.bus->mdio_lock); |
---|
| 636 | + phy_unlock_mdio_bus(phydev); |
---|
369 | 637 | |
---|
370 | 638 | return ret; |
---|
371 | 639 | } |
---|
372 | 640 | EXPORT_SYMBOL_GPL(phy_modify); |
---|
373 | 641 | |
---|
| 642 | +/** |
---|
| 643 | + * __phy_modify_mmd_changed - Function for modifying a register on MMD |
---|
| 644 | + * @phydev: the phy_device struct |
---|
| 645 | + * @devad: the MMD containing register to modify |
---|
| 646 | + * @regnum: register number to modify |
---|
| 647 | + * @mask: bit mask of bits to clear |
---|
| 648 | + * @set: new value of bits set in mask to write to @regnum |
---|
| 649 | + * |
---|
| 650 | + * Unlocked helper function which allows a MMD register to be modified as |
---|
| 651 | + * new register value = (old register value & ~mask) | set |
---|
| 652 | + * |
---|
| 653 | + * Returns negative errno, 0 if there was no change, and 1 in case of change |
---|
| 654 | + */ |
---|
| 655 | +int __phy_modify_mmd_changed(struct phy_device *phydev, int devad, u32 regnum, |
---|
| 656 | + u16 mask, u16 set) |
---|
| 657 | +{ |
---|
| 658 | + int new, ret; |
---|
| 659 | + |
---|
| 660 | + ret = __phy_read_mmd(phydev, devad, regnum); |
---|
| 661 | + if (ret < 0) |
---|
| 662 | + return ret; |
---|
| 663 | + |
---|
| 664 | + new = (ret & ~mask) | set; |
---|
| 665 | + if (new == ret) |
---|
| 666 | + return 0; |
---|
| 667 | + |
---|
| 668 | + ret = __phy_write_mmd(phydev, devad, regnum, new); |
---|
| 669 | + |
---|
| 670 | + return ret < 0 ? ret : 1; |
---|
| 671 | +} |
---|
| 672 | +EXPORT_SYMBOL_GPL(__phy_modify_mmd_changed); |
---|
| 673 | + |
---|
| 674 | +/** |
---|
| 675 | + * phy_modify_mmd_changed - Function for modifying a register on MMD |
---|
| 676 | + * @phydev: the phy_device struct |
---|
| 677 | + * @devad: the MMD containing register to modify |
---|
| 678 | + * @regnum: register number to modify |
---|
| 679 | + * @mask: bit mask of bits to clear |
---|
| 680 | + * @set: new value of bits set in mask to write to @regnum |
---|
| 681 | + * |
---|
| 682 | + * NOTE: MUST NOT be called from interrupt context, |
---|
| 683 | + * because the bus read/write functions may wait for an interrupt |
---|
| 684 | + * to conclude the operation. |
---|
| 685 | + * |
---|
| 686 | + * Returns negative errno, 0 if there was no change, and 1 in case of change |
---|
| 687 | + */ |
---|
| 688 | +int phy_modify_mmd_changed(struct phy_device *phydev, int devad, u32 regnum, |
---|
| 689 | + u16 mask, u16 set) |
---|
| 690 | +{ |
---|
| 691 | + int ret; |
---|
| 692 | + |
---|
| 693 | + phy_lock_mdio_bus(phydev); |
---|
| 694 | + ret = __phy_modify_mmd_changed(phydev, devad, regnum, mask, set); |
---|
| 695 | + phy_unlock_mdio_bus(phydev); |
---|
| 696 | + |
---|
| 697 | + return ret; |
---|
| 698 | +} |
---|
| 699 | +EXPORT_SYMBOL_GPL(phy_modify_mmd_changed); |
---|
| 700 | + |
---|
| 701 | +/** |
---|
| 702 | + * __phy_modify_mmd - Convenience function for modifying a register on MMD |
---|
| 703 | + * @phydev: the phy_device struct |
---|
| 704 | + * @devad: the MMD containing register to modify |
---|
| 705 | + * @regnum: register number to modify |
---|
| 706 | + * @mask: bit mask of bits to clear |
---|
| 707 | + * @set: new value of bits set in mask to write to @regnum |
---|
| 708 | + * |
---|
| 709 | + * NOTE: MUST NOT be called from interrupt context, |
---|
| 710 | + * because the bus read/write functions may wait for an interrupt |
---|
| 711 | + * to conclude the operation. |
---|
| 712 | + */ |
---|
| 713 | +int __phy_modify_mmd(struct phy_device *phydev, int devad, u32 regnum, |
---|
| 714 | + u16 mask, u16 set) |
---|
| 715 | +{ |
---|
| 716 | + int ret; |
---|
| 717 | + |
---|
| 718 | + ret = __phy_modify_mmd_changed(phydev, devad, regnum, mask, set); |
---|
| 719 | + |
---|
| 720 | + return ret < 0 ? ret : 0; |
---|
| 721 | +} |
---|
| 722 | +EXPORT_SYMBOL_GPL(__phy_modify_mmd); |
---|
| 723 | + |
---|
| 724 | +/** |
---|
| 725 | + * phy_modify_mmd - Convenience function for modifying a register on MMD |
---|
| 726 | + * @phydev: the phy_device struct |
---|
| 727 | + * @devad: the MMD containing register to modify |
---|
| 728 | + * @regnum: register number to modify |
---|
| 729 | + * @mask: bit mask of bits to clear |
---|
| 730 | + * @set: new value of bits set in mask to write to @regnum |
---|
| 731 | + * |
---|
| 732 | + * NOTE: MUST NOT be called from interrupt context, |
---|
| 733 | + * because the bus read/write functions may wait for an interrupt |
---|
| 734 | + * to conclude the operation. |
---|
| 735 | + */ |
---|
| 736 | +int phy_modify_mmd(struct phy_device *phydev, int devad, u32 regnum, |
---|
| 737 | + u16 mask, u16 set) |
---|
| 738 | +{ |
---|
| 739 | + int ret; |
---|
| 740 | + |
---|
| 741 | + phy_lock_mdio_bus(phydev); |
---|
| 742 | + ret = __phy_modify_mmd(phydev, devad, regnum, mask, set); |
---|
| 743 | + phy_unlock_mdio_bus(phydev); |
---|
| 744 | + |
---|
| 745 | + return ret; |
---|
| 746 | +} |
---|
| 747 | +EXPORT_SYMBOL_GPL(phy_modify_mmd); |
---|
| 748 | + |
---|
374 | 749 | static int __phy_read_page(struct phy_device *phydev) |
---|
375 | 750 | { |
---|
| 751 | + if (WARN_ONCE(!phydev->drv->read_page, "read_page callback not available, PHY driver not loaded?\n")) |
---|
| 752 | + return -EOPNOTSUPP; |
---|
| 753 | + |
---|
376 | 754 | return phydev->drv->read_page(phydev); |
---|
377 | 755 | } |
---|
378 | 756 | |
---|
379 | 757 | static int __phy_write_page(struct phy_device *phydev, int page) |
---|
380 | 758 | { |
---|
| 759 | + if (WARN_ONCE(!phydev->drv->write_page, "write_page callback not available, PHY driver not loaded?\n")) |
---|
| 760 | + return -EOPNOTSUPP; |
---|
| 761 | + |
---|
381 | 762 | return phydev->drv->write_page(phydev, page); |
---|
382 | 763 | } |
---|
383 | 764 | |
---|
.. | .. |
---|
391 | 772 | */ |
---|
392 | 773 | int phy_save_page(struct phy_device *phydev) |
---|
393 | 774 | { |
---|
394 | | - mutex_lock(&phydev->mdio.bus->mdio_lock); |
---|
| 775 | + phy_lock_mdio_bus(phydev); |
---|
395 | 776 | return __phy_read_page(phydev); |
---|
396 | 777 | } |
---|
397 | 778 | EXPORT_SYMBOL_GPL(phy_save_page); |
---|
.. | .. |
---|
458 | 839 | ret = oldpage; |
---|
459 | 840 | } |
---|
460 | 841 | |
---|
461 | | - mutex_unlock(&phydev->mdio.bus->mdio_lock); |
---|
| 842 | + phy_unlock_mdio_bus(phydev); |
---|
462 | 843 | |
---|
463 | 844 | return ret; |
---|
464 | 845 | } |
---|
.. | .. |
---|
506 | 887 | EXPORT_SYMBOL(phy_write_paged); |
---|
507 | 888 | |
---|
508 | 889 | /** |
---|
| 890 | + * phy_modify_paged_changed() - Function for modifying a paged register |
---|
| 891 | + * @phydev: a pointer to a &struct phy_device |
---|
| 892 | + * @page: the page for the phy |
---|
| 893 | + * @regnum: register number |
---|
| 894 | + * @mask: bit mask of bits to clear |
---|
| 895 | + * @set: bit mask of bits to set |
---|
| 896 | + * |
---|
| 897 | + * Returns negative errno, 0 if there was no change, and 1 in case of change |
---|
| 898 | + */ |
---|
| 899 | +int phy_modify_paged_changed(struct phy_device *phydev, int page, u32 regnum, |
---|
| 900 | + u16 mask, u16 set) |
---|
| 901 | +{ |
---|
| 902 | + int ret = 0, oldpage; |
---|
| 903 | + |
---|
| 904 | + oldpage = phy_select_page(phydev, page); |
---|
| 905 | + if (oldpage >= 0) |
---|
| 906 | + ret = __phy_modify_changed(phydev, regnum, mask, set); |
---|
| 907 | + |
---|
| 908 | + return phy_restore_page(phydev, oldpage, ret); |
---|
| 909 | +} |
---|
| 910 | +EXPORT_SYMBOL(phy_modify_paged_changed); |
---|
| 911 | + |
---|
| 912 | +/** |
---|
509 | 913 | * phy_modify_paged() - Convenience function for modifying a paged register |
---|
510 | 914 | * @phydev: a pointer to a &struct phy_device |
---|
511 | 915 | * @page: the page for the phy |
---|
.. | .. |
---|
518 | 922 | int phy_modify_paged(struct phy_device *phydev, int page, u32 regnum, |
---|
519 | 923 | u16 mask, u16 set) |
---|
520 | 924 | { |
---|
521 | | - int ret = 0, oldpage; |
---|
| 925 | + int ret = phy_modify_paged_changed(phydev, page, regnum, mask, set); |
---|
522 | 926 | |
---|
523 | | - oldpage = phy_select_page(phydev, page); |
---|
524 | | - if (oldpage >= 0) |
---|
525 | | - ret = __phy_modify(phydev, regnum, mask, set); |
---|
526 | | - |
---|
527 | | - return phy_restore_page(phydev, oldpage, ret); |
---|
| 927 | + return ret < 0 ? ret : 0; |
---|
528 | 928 | } |
---|
529 | 929 | EXPORT_SYMBOL(phy_modify_paged); |
---|