| .. | .. |
|---|
| 1 | +// SPDX-License-Identifier: GPL-2.0-or-later |
|---|
| 1 | 2 | /* |
|---|
| 2 | 3 | * Handling of a single switch port |
|---|
| 3 | 4 | * |
|---|
| 4 | 5 | * Copyright (c) 2017 Savoir-faire Linux Inc. |
|---|
| 5 | 6 | * Vivien Didelot <vivien.didelot@savoirfairelinux.com> |
|---|
| 6 | | - * |
|---|
| 7 | | - * This program is free software; you can redistribute it and/or modify |
|---|
| 8 | | - * it under the terms of the GNU General Public License as published by |
|---|
| 9 | | - * the Free Software Foundation; either version 2 of the License, or |
|---|
| 10 | | - * (at your option) any later version. |
|---|
| 11 | 7 | */ |
|---|
| 12 | 8 | |
|---|
| 13 | 9 | #include <linux/if_bridge.h> |
|---|
| .. | .. |
|---|
| 16 | 12 | #include <linux/of_net.h> |
|---|
| 17 | 13 | |
|---|
| 18 | 14 | #include "dsa_priv.h" |
|---|
| 15 | + |
|---|
| 16 | +static int dsa_broadcast(unsigned long e, void *v) |
|---|
| 17 | +{ |
|---|
| 18 | + struct dsa_switch_tree *dst; |
|---|
| 19 | + int err = 0; |
|---|
| 20 | + |
|---|
| 21 | + list_for_each_entry(dst, &dsa_tree_list, list) { |
|---|
| 22 | + struct raw_notifier_head *nh = &dst->nh; |
|---|
| 23 | + |
|---|
| 24 | + err = raw_notifier_call_chain(nh, e, v); |
|---|
| 25 | + err = notifier_to_errno(err); |
|---|
| 26 | + if (err) |
|---|
| 27 | + break; |
|---|
| 28 | + } |
|---|
| 29 | + |
|---|
| 30 | + return err; |
|---|
| 31 | +} |
|---|
| 19 | 32 | |
|---|
| 20 | 33 | static int dsa_port_notify(const struct dsa_port *dp, unsigned long e, void *v) |
|---|
| 21 | 34 | { |
|---|
| .. | .. |
|---|
| 67 | 80 | pr_err("DSA: failed to set STP state %u (%d)\n", state, err); |
|---|
| 68 | 81 | } |
|---|
| 69 | 82 | |
|---|
| 70 | | -int dsa_port_enable(struct dsa_port *dp, struct phy_device *phy) |
|---|
| 83 | +int dsa_port_enable_rt(struct dsa_port *dp, struct phy_device *phy) |
|---|
| 71 | 84 | { |
|---|
| 72 | 85 | struct dsa_switch *ds = dp->ds; |
|---|
| 73 | 86 | int port = dp->index; |
|---|
| .. | .. |
|---|
| 82 | 95 | if (!dp->bridge_dev) |
|---|
| 83 | 96 | dsa_port_set_state_now(dp, BR_STATE_FORWARDING); |
|---|
| 84 | 97 | |
|---|
| 98 | + if (dp->pl) |
|---|
| 99 | + phylink_start(dp->pl); |
|---|
| 100 | + |
|---|
| 85 | 101 | return 0; |
|---|
| 86 | 102 | } |
|---|
| 87 | 103 | |
|---|
| 88 | | -void dsa_port_disable(struct dsa_port *dp, struct phy_device *phy) |
|---|
| 104 | +int dsa_port_enable(struct dsa_port *dp, struct phy_device *phy) |
|---|
| 105 | +{ |
|---|
| 106 | + int err; |
|---|
| 107 | + |
|---|
| 108 | + rtnl_lock(); |
|---|
| 109 | + err = dsa_port_enable_rt(dp, phy); |
|---|
| 110 | + rtnl_unlock(); |
|---|
| 111 | + |
|---|
| 112 | + return err; |
|---|
| 113 | +} |
|---|
| 114 | + |
|---|
| 115 | +void dsa_port_disable_rt(struct dsa_port *dp) |
|---|
| 89 | 116 | { |
|---|
| 90 | 117 | struct dsa_switch *ds = dp->ds; |
|---|
| 91 | 118 | int port = dp->index; |
|---|
| 119 | + |
|---|
| 120 | + if (dp->pl) |
|---|
| 121 | + phylink_stop(dp->pl); |
|---|
| 92 | 122 | |
|---|
| 93 | 123 | if (!dp->bridge_dev) |
|---|
| 94 | 124 | dsa_port_set_state_now(dp, BR_STATE_DISABLED); |
|---|
| 95 | 125 | |
|---|
| 96 | 126 | if (ds->ops->port_disable) |
|---|
| 97 | | - ds->ops->port_disable(ds, port, phy); |
|---|
| 127 | + ds->ops->port_disable(ds, port); |
|---|
| 128 | +} |
|---|
| 129 | + |
|---|
| 130 | +void dsa_port_disable(struct dsa_port *dp) |
|---|
| 131 | +{ |
|---|
| 132 | + rtnl_lock(); |
|---|
| 133 | + dsa_port_disable_rt(dp); |
|---|
| 134 | + rtnl_unlock(); |
|---|
| 98 | 135 | } |
|---|
| 99 | 136 | |
|---|
| 100 | 137 | int dsa_port_bridge_join(struct dsa_port *dp, struct net_device *br) |
|---|
| 101 | 138 | { |
|---|
| 102 | 139 | struct dsa_notifier_bridge_info info = { |
|---|
| 140 | + .tree_index = dp->ds->dst->index, |
|---|
| 103 | 141 | .sw_index = dp->ds->index, |
|---|
| 104 | 142 | .port = dp->index, |
|---|
| 105 | 143 | .br = br, |
|---|
| 106 | 144 | }; |
|---|
| 107 | 145 | int err; |
|---|
| 108 | 146 | |
|---|
| 109 | | - /* Here the port is already bridged. Reflect the current configuration |
|---|
| 110 | | - * so that drivers can program their chips accordingly. |
|---|
| 147 | + /* Set the flooding mode before joining the port in the switch */ |
|---|
| 148 | + err = dsa_port_bridge_flags(dp, BR_FLOOD | BR_MCAST_FLOOD, NULL); |
|---|
| 149 | + if (err) |
|---|
| 150 | + return err; |
|---|
| 151 | + |
|---|
| 152 | + /* Here the interface is already bridged. Reflect the current |
|---|
| 153 | + * configuration so that drivers can program their chips accordingly. |
|---|
| 111 | 154 | */ |
|---|
| 112 | 155 | dp->bridge_dev = br; |
|---|
| 113 | 156 | |
|---|
| 114 | | - err = dsa_port_notify(dp, DSA_NOTIFIER_BRIDGE_JOIN, &info); |
|---|
| 157 | + err = dsa_broadcast(DSA_NOTIFIER_BRIDGE_JOIN, &info); |
|---|
| 115 | 158 | |
|---|
| 116 | 159 | /* The bridging is rolled back on error */ |
|---|
| 117 | | - if (err) |
|---|
| 160 | + if (err) { |
|---|
| 161 | + dsa_port_bridge_flags(dp, 0, NULL); |
|---|
| 118 | 162 | dp->bridge_dev = NULL; |
|---|
| 163 | + } |
|---|
| 119 | 164 | |
|---|
| 120 | 165 | return err; |
|---|
| 121 | 166 | } |
|---|
| .. | .. |
|---|
| 123 | 168 | void dsa_port_bridge_leave(struct dsa_port *dp, struct net_device *br) |
|---|
| 124 | 169 | { |
|---|
| 125 | 170 | struct dsa_notifier_bridge_info info = { |
|---|
| 171 | + .tree_index = dp->ds->dst->index, |
|---|
| 126 | 172 | .sw_index = dp->ds->index, |
|---|
| 127 | 173 | .port = dp->index, |
|---|
| 128 | 174 | .br = br, |
|---|
| .. | .. |
|---|
| 134 | 180 | */ |
|---|
| 135 | 181 | dp->bridge_dev = NULL; |
|---|
| 136 | 182 | |
|---|
| 137 | | - err = dsa_port_notify(dp, DSA_NOTIFIER_BRIDGE_LEAVE, &info); |
|---|
| 183 | + err = dsa_broadcast(DSA_NOTIFIER_BRIDGE_LEAVE, &info); |
|---|
| 138 | 184 | if (err) |
|---|
| 139 | 185 | pr_err("DSA: failed to notify DSA_NOTIFIER_BRIDGE_LEAVE\n"); |
|---|
| 186 | + |
|---|
| 187 | + /* Port is leaving the bridge, disable flooding */ |
|---|
| 188 | + dsa_port_bridge_flags(dp, 0, NULL); |
|---|
| 140 | 189 | |
|---|
| 141 | 190 | /* Port left the bridge, put in BR_STATE_DISABLED by the bridge layer, |
|---|
| 142 | 191 | * so allow it to be in BR_STATE_FORWARDING to be kept functional |
|---|
| .. | .. |
|---|
| 144 | 193 | dsa_port_set_state_now(dp, BR_STATE_FORWARDING); |
|---|
| 145 | 194 | } |
|---|
| 146 | 195 | |
|---|
| 196 | +/* Must be called under rcu_read_lock() */ |
|---|
| 197 | +static bool dsa_port_can_apply_vlan_filtering(struct dsa_port *dp, |
|---|
| 198 | + bool vlan_filtering) |
|---|
| 199 | +{ |
|---|
| 200 | + struct dsa_switch *ds = dp->ds; |
|---|
| 201 | + int err, i; |
|---|
| 202 | + |
|---|
| 203 | + /* VLAN awareness was off, so the question is "can we turn it on". |
|---|
| 204 | + * We may have had 8021q uppers, those need to go. Make sure we don't |
|---|
| 205 | + * enter an inconsistent state: deny changing the VLAN awareness state |
|---|
| 206 | + * as long as we have 8021q uppers. |
|---|
| 207 | + */ |
|---|
| 208 | + if (vlan_filtering && dsa_is_user_port(ds, dp->index)) { |
|---|
| 209 | + struct net_device *upper_dev, *slave = dp->slave; |
|---|
| 210 | + struct net_device *br = dp->bridge_dev; |
|---|
| 211 | + struct list_head *iter; |
|---|
| 212 | + |
|---|
| 213 | + netdev_for_each_upper_dev_rcu(slave, upper_dev, iter) { |
|---|
| 214 | + struct bridge_vlan_info br_info; |
|---|
| 215 | + u16 vid; |
|---|
| 216 | + |
|---|
| 217 | + if (!is_vlan_dev(upper_dev)) |
|---|
| 218 | + continue; |
|---|
| 219 | + |
|---|
| 220 | + vid = vlan_dev_vlan_id(upper_dev); |
|---|
| 221 | + |
|---|
| 222 | + /* br_vlan_get_info() returns -EINVAL or -ENOENT if the |
|---|
| 223 | + * device, respectively the VID is not found, returning |
|---|
| 224 | + * 0 means success, which is a failure for us here. |
|---|
| 225 | + */ |
|---|
| 226 | + err = br_vlan_get_info(br, vid, &br_info); |
|---|
| 227 | + if (err == 0) { |
|---|
| 228 | + dev_err(ds->dev, "Must remove upper %s first\n", |
|---|
| 229 | + upper_dev->name); |
|---|
| 230 | + return false; |
|---|
| 231 | + } |
|---|
| 232 | + } |
|---|
| 233 | + } |
|---|
| 234 | + |
|---|
| 235 | + if (!ds->vlan_filtering_is_global) |
|---|
| 236 | + return true; |
|---|
| 237 | + |
|---|
| 238 | + /* For cases where enabling/disabling VLAN awareness is global to the |
|---|
| 239 | + * switch, we need to handle the case where multiple bridges span |
|---|
| 240 | + * different ports of the same switch device and one of them has a |
|---|
| 241 | + * different setting than what is being requested. |
|---|
| 242 | + */ |
|---|
| 243 | + for (i = 0; i < ds->num_ports; i++) { |
|---|
| 244 | + struct net_device *other_bridge; |
|---|
| 245 | + |
|---|
| 246 | + other_bridge = dsa_to_port(ds, i)->bridge_dev; |
|---|
| 247 | + if (!other_bridge) |
|---|
| 248 | + continue; |
|---|
| 249 | + /* If it's the same bridge, it also has same |
|---|
| 250 | + * vlan_filtering setting => no need to check |
|---|
| 251 | + */ |
|---|
| 252 | + if (other_bridge == dp->bridge_dev) |
|---|
| 253 | + continue; |
|---|
| 254 | + if (br_vlan_enabled(other_bridge) != vlan_filtering) { |
|---|
| 255 | + dev_err(ds->dev, "VLAN filtering is a global setting\n"); |
|---|
| 256 | + return false; |
|---|
| 257 | + } |
|---|
| 258 | + } |
|---|
| 259 | + return true; |
|---|
| 260 | +} |
|---|
| 261 | + |
|---|
| 147 | 262 | int dsa_port_vlan_filtering(struct dsa_port *dp, bool vlan_filtering, |
|---|
| 148 | 263 | struct switchdev_trans *trans) |
|---|
| 149 | 264 | { |
|---|
| 150 | 265 | struct dsa_switch *ds = dp->ds; |
|---|
| 266 | + int err; |
|---|
| 151 | 267 | |
|---|
| 152 | | - /* bridge skips -EOPNOTSUPP, so skip the prepare phase */ |
|---|
| 153 | | - if (switchdev_trans_ph_prepare(trans)) |
|---|
| 268 | + if (switchdev_trans_ph_prepare(trans)) { |
|---|
| 269 | + bool apply; |
|---|
| 270 | + |
|---|
| 271 | + if (!ds->ops->port_vlan_filtering) |
|---|
| 272 | + return -EOPNOTSUPP; |
|---|
| 273 | + |
|---|
| 274 | + /* We are called from dsa_slave_switchdev_blocking_event(), |
|---|
| 275 | + * which is not under rcu_read_lock(), unlike |
|---|
| 276 | + * dsa_slave_switchdev_event(). |
|---|
| 277 | + */ |
|---|
| 278 | + rcu_read_lock(); |
|---|
| 279 | + apply = dsa_port_can_apply_vlan_filtering(dp, vlan_filtering); |
|---|
| 280 | + rcu_read_unlock(); |
|---|
| 281 | + if (!apply) |
|---|
| 282 | + return -EINVAL; |
|---|
| 283 | + } |
|---|
| 284 | + |
|---|
| 285 | + if (dsa_port_is_vlan_filtering(dp) == vlan_filtering) |
|---|
| 154 | 286 | return 0; |
|---|
| 155 | 287 | |
|---|
| 156 | | - if (ds->ops->port_vlan_filtering) |
|---|
| 157 | | - return ds->ops->port_vlan_filtering(ds, dp->index, |
|---|
| 158 | | - vlan_filtering); |
|---|
| 288 | + err = ds->ops->port_vlan_filtering(ds, dp->index, vlan_filtering, |
|---|
| 289 | + trans); |
|---|
| 290 | + if (err) |
|---|
| 291 | + return err; |
|---|
| 292 | + |
|---|
| 293 | + if (switchdev_trans_ph_commit(trans)) { |
|---|
| 294 | + if (ds->vlan_filtering_is_global) |
|---|
| 295 | + ds->vlan_filtering = vlan_filtering; |
|---|
| 296 | + else |
|---|
| 297 | + dp->vlan_filtering = vlan_filtering; |
|---|
| 298 | + } |
|---|
| 159 | 299 | |
|---|
| 160 | 300 | return 0; |
|---|
| 301 | +} |
|---|
| 302 | + |
|---|
| 303 | +/* This enforces legacy behavior for switch drivers which assume they can't |
|---|
| 304 | + * receive VLAN configuration when enslaved to a bridge with vlan_filtering=0 |
|---|
| 305 | + */ |
|---|
| 306 | +bool dsa_port_skip_vlan_configuration(struct dsa_port *dp) |
|---|
| 307 | +{ |
|---|
| 308 | + struct dsa_switch *ds = dp->ds; |
|---|
| 309 | + |
|---|
| 310 | + if (!dp->bridge_dev) |
|---|
| 311 | + return false; |
|---|
| 312 | + |
|---|
| 313 | + return (!ds->configure_vlan_while_not_filtering && |
|---|
| 314 | + !br_vlan_enabled(dp->bridge_dev)); |
|---|
| 161 | 315 | } |
|---|
| 162 | 316 | |
|---|
| 163 | 317 | int dsa_port_ageing_time(struct dsa_port *dp, clock_t ageing_clock, |
|---|
| .. | .. |
|---|
| 176 | 330 | dp->ageing_time = ageing_time; |
|---|
| 177 | 331 | |
|---|
| 178 | 332 | return dsa_port_notify(dp, DSA_NOTIFIER_AGEING_TIME, &info); |
|---|
| 333 | +} |
|---|
| 334 | + |
|---|
| 335 | +int dsa_port_pre_bridge_flags(const struct dsa_port *dp, unsigned long flags, |
|---|
| 336 | + struct switchdev_trans *trans) |
|---|
| 337 | +{ |
|---|
| 338 | + struct dsa_switch *ds = dp->ds; |
|---|
| 339 | + |
|---|
| 340 | + if (!ds->ops->port_egress_floods || |
|---|
| 341 | + (flags & ~(BR_FLOOD | BR_MCAST_FLOOD))) |
|---|
| 342 | + return -EINVAL; |
|---|
| 343 | + |
|---|
| 344 | + return 0; |
|---|
| 345 | +} |
|---|
| 346 | + |
|---|
| 347 | +int dsa_port_bridge_flags(const struct dsa_port *dp, unsigned long flags, |
|---|
| 348 | + struct switchdev_trans *trans) |
|---|
| 349 | +{ |
|---|
| 350 | + struct dsa_switch *ds = dp->ds; |
|---|
| 351 | + int port = dp->index; |
|---|
| 352 | + int err = 0; |
|---|
| 353 | + |
|---|
| 354 | + if (switchdev_trans_ph_prepare(trans)) |
|---|
| 355 | + return 0; |
|---|
| 356 | + |
|---|
| 357 | + if (ds->ops->port_egress_floods) |
|---|
| 358 | + err = ds->ops->port_egress_floods(ds, port, flags & BR_FLOOD, |
|---|
| 359 | + flags & BR_MCAST_FLOOD); |
|---|
| 360 | + |
|---|
| 361 | + return err; |
|---|
| 362 | +} |
|---|
| 363 | + |
|---|
| 364 | +int dsa_port_mrouter(struct dsa_port *dp, bool mrouter, |
|---|
| 365 | + struct switchdev_trans *trans) |
|---|
| 366 | +{ |
|---|
| 367 | + struct dsa_switch *ds = dp->ds; |
|---|
| 368 | + int port = dp->index; |
|---|
| 369 | + |
|---|
| 370 | + if (switchdev_trans_ph_prepare(trans)) |
|---|
| 371 | + return ds->ops->port_egress_floods ? 0 : -EOPNOTSUPP; |
|---|
| 372 | + |
|---|
| 373 | + return ds->ops->port_egress_floods(ds, port, true, mrouter); |
|---|
| 374 | +} |
|---|
| 375 | + |
|---|
| 376 | +int dsa_port_mtu_change(struct dsa_port *dp, int new_mtu, |
|---|
| 377 | + bool propagate_upstream) |
|---|
| 378 | +{ |
|---|
| 379 | + struct dsa_notifier_mtu_info info = { |
|---|
| 380 | + .sw_index = dp->ds->index, |
|---|
| 381 | + .propagate_upstream = propagate_upstream, |
|---|
| 382 | + .port = dp->index, |
|---|
| 383 | + .mtu = new_mtu, |
|---|
| 384 | + }; |
|---|
| 385 | + |
|---|
| 386 | + return dsa_port_notify(dp, DSA_NOTIFIER_MTU, &info); |
|---|
| 179 | 387 | } |
|---|
| 180 | 388 | |
|---|
| 181 | 389 | int dsa_port_fdb_add(struct dsa_port *dp, const unsigned char *addr, |
|---|
| .. | .. |
|---|
| 253 | 461 | .vlan = vlan, |
|---|
| 254 | 462 | }; |
|---|
| 255 | 463 | |
|---|
| 256 | | - if (netif_is_bridge_master(vlan->obj.orig_dev)) |
|---|
| 257 | | - return -EOPNOTSUPP; |
|---|
| 258 | | - |
|---|
| 259 | | - if (br_vlan_enabled(dp->bridge_dev)) |
|---|
| 260 | | - return dsa_port_notify(dp, DSA_NOTIFIER_VLAN_ADD, &info); |
|---|
| 261 | | - |
|---|
| 262 | | - return 0; |
|---|
| 464 | + return dsa_port_notify(dp, DSA_NOTIFIER_VLAN_ADD, &info); |
|---|
| 263 | 465 | } |
|---|
| 264 | 466 | |
|---|
| 265 | 467 | int dsa_port_vlan_del(struct dsa_port *dp, |
|---|
| .. | .. |
|---|
| 271 | 473 | .vlan = vlan, |
|---|
| 272 | 474 | }; |
|---|
| 273 | 475 | |
|---|
| 274 | | - if (netif_is_bridge_master(vlan->obj.orig_dev)) |
|---|
| 275 | | - return -EOPNOTSUPP; |
|---|
| 276 | | - |
|---|
| 277 | | - if (br_vlan_enabled(dp->bridge_dev)) |
|---|
| 278 | | - return dsa_port_notify(dp, DSA_NOTIFIER_VLAN_DEL, &info); |
|---|
| 279 | | - |
|---|
| 280 | | - return 0; |
|---|
| 476 | + return dsa_port_notify(dp, DSA_NOTIFIER_VLAN_DEL, &info); |
|---|
| 281 | 477 | } |
|---|
| 282 | 478 | |
|---|
| 283 | 479 | static struct phy_device *dsa_port_get_phy_device(struct dsa_port *dp) |
|---|
| .. | .. |
|---|
| 295 | 491 | return ERR_PTR(-EPROBE_DEFER); |
|---|
| 296 | 492 | } |
|---|
| 297 | 493 | |
|---|
| 494 | + of_node_put(phy_dn); |
|---|
| 298 | 495 | return phydev; |
|---|
| 299 | 496 | } |
|---|
| 497 | + |
|---|
| 498 | +static void dsa_port_phylink_validate(struct phylink_config *config, |
|---|
| 499 | + unsigned long *supported, |
|---|
| 500 | + struct phylink_link_state *state) |
|---|
| 501 | +{ |
|---|
| 502 | + struct dsa_port *dp = container_of(config, struct dsa_port, pl_config); |
|---|
| 503 | + struct dsa_switch *ds = dp->ds; |
|---|
| 504 | + |
|---|
| 505 | + if (!ds->ops->phylink_validate) |
|---|
| 506 | + return; |
|---|
| 507 | + |
|---|
| 508 | + ds->ops->phylink_validate(ds, dp->index, supported, state); |
|---|
| 509 | +} |
|---|
| 510 | + |
|---|
| 511 | +static void dsa_port_phylink_mac_pcs_get_state(struct phylink_config *config, |
|---|
| 512 | + struct phylink_link_state *state) |
|---|
| 513 | +{ |
|---|
| 514 | + struct dsa_port *dp = container_of(config, struct dsa_port, pl_config); |
|---|
| 515 | + struct dsa_switch *ds = dp->ds; |
|---|
| 516 | + int err; |
|---|
| 517 | + |
|---|
| 518 | + /* Only called for inband modes */ |
|---|
| 519 | + if (!ds->ops->phylink_mac_link_state) { |
|---|
| 520 | + state->link = 0; |
|---|
| 521 | + return; |
|---|
| 522 | + } |
|---|
| 523 | + |
|---|
| 524 | + err = ds->ops->phylink_mac_link_state(ds, dp->index, state); |
|---|
| 525 | + if (err < 0) { |
|---|
| 526 | + dev_err(ds->dev, "p%d: phylink_mac_link_state() failed: %d\n", |
|---|
| 527 | + dp->index, err); |
|---|
| 528 | + state->link = 0; |
|---|
| 529 | + } |
|---|
| 530 | +} |
|---|
| 531 | + |
|---|
| 532 | +static void dsa_port_phylink_mac_config(struct phylink_config *config, |
|---|
| 533 | + unsigned int mode, |
|---|
| 534 | + const struct phylink_link_state *state) |
|---|
| 535 | +{ |
|---|
| 536 | + struct dsa_port *dp = container_of(config, struct dsa_port, pl_config); |
|---|
| 537 | + struct dsa_switch *ds = dp->ds; |
|---|
| 538 | + |
|---|
| 539 | + if (!ds->ops->phylink_mac_config) |
|---|
| 540 | + return; |
|---|
| 541 | + |
|---|
| 542 | + ds->ops->phylink_mac_config(ds, dp->index, mode, state); |
|---|
| 543 | +} |
|---|
| 544 | + |
|---|
| 545 | +static void dsa_port_phylink_mac_an_restart(struct phylink_config *config) |
|---|
| 546 | +{ |
|---|
| 547 | + struct dsa_port *dp = container_of(config, struct dsa_port, pl_config); |
|---|
| 548 | + struct dsa_switch *ds = dp->ds; |
|---|
| 549 | + |
|---|
| 550 | + if (!ds->ops->phylink_mac_an_restart) |
|---|
| 551 | + return; |
|---|
| 552 | + |
|---|
| 553 | + ds->ops->phylink_mac_an_restart(ds, dp->index); |
|---|
| 554 | +} |
|---|
| 555 | + |
|---|
| 556 | +static void dsa_port_phylink_mac_link_down(struct phylink_config *config, |
|---|
| 557 | + unsigned int mode, |
|---|
| 558 | + phy_interface_t interface) |
|---|
| 559 | +{ |
|---|
| 560 | + struct dsa_port *dp = container_of(config, struct dsa_port, pl_config); |
|---|
| 561 | + struct phy_device *phydev = NULL; |
|---|
| 562 | + struct dsa_switch *ds = dp->ds; |
|---|
| 563 | + |
|---|
| 564 | + if (dsa_is_user_port(ds, dp->index)) |
|---|
| 565 | + phydev = dp->slave->phydev; |
|---|
| 566 | + |
|---|
| 567 | + if (!ds->ops->phylink_mac_link_down) { |
|---|
| 568 | + if (ds->ops->adjust_link && phydev) |
|---|
| 569 | + ds->ops->adjust_link(ds, dp->index, phydev); |
|---|
| 570 | + return; |
|---|
| 571 | + } |
|---|
| 572 | + |
|---|
| 573 | + ds->ops->phylink_mac_link_down(ds, dp->index, mode, interface); |
|---|
| 574 | +} |
|---|
| 575 | + |
|---|
| 576 | +static void dsa_port_phylink_mac_link_up(struct phylink_config *config, |
|---|
| 577 | + struct phy_device *phydev, |
|---|
| 578 | + unsigned int mode, |
|---|
| 579 | + phy_interface_t interface, |
|---|
| 580 | + int speed, int duplex, |
|---|
| 581 | + bool tx_pause, bool rx_pause) |
|---|
| 582 | +{ |
|---|
| 583 | + struct dsa_port *dp = container_of(config, struct dsa_port, pl_config); |
|---|
| 584 | + struct dsa_switch *ds = dp->ds; |
|---|
| 585 | + |
|---|
| 586 | + if (!ds->ops->phylink_mac_link_up) { |
|---|
| 587 | + if (ds->ops->adjust_link && phydev) |
|---|
| 588 | + ds->ops->adjust_link(ds, dp->index, phydev); |
|---|
| 589 | + return; |
|---|
| 590 | + } |
|---|
| 591 | + |
|---|
| 592 | + ds->ops->phylink_mac_link_up(ds, dp->index, mode, interface, phydev, |
|---|
| 593 | + speed, duplex, tx_pause, rx_pause); |
|---|
| 594 | +} |
|---|
| 595 | + |
|---|
| 596 | +const struct phylink_mac_ops dsa_port_phylink_mac_ops = { |
|---|
| 597 | + .validate = dsa_port_phylink_validate, |
|---|
| 598 | + .mac_pcs_get_state = dsa_port_phylink_mac_pcs_get_state, |
|---|
| 599 | + .mac_config = dsa_port_phylink_mac_config, |
|---|
| 600 | + .mac_an_restart = dsa_port_phylink_mac_an_restart, |
|---|
| 601 | + .mac_link_down = dsa_port_phylink_mac_link_down, |
|---|
| 602 | + .mac_link_up = dsa_port_phylink_mac_link_up, |
|---|
| 603 | +}; |
|---|
| 300 | 604 | |
|---|
| 301 | 605 | static int dsa_port_setup_phy_of(struct dsa_port *dp, bool enable) |
|---|
| 302 | 606 | { |
|---|
| .. | .. |
|---|
| 313 | 617 | return PTR_ERR(phydev); |
|---|
| 314 | 618 | |
|---|
| 315 | 619 | if (enable) { |
|---|
| 316 | | - err = genphy_config_init(phydev); |
|---|
| 317 | | - if (err < 0) |
|---|
| 318 | | - goto err_put_dev; |
|---|
| 319 | | - |
|---|
| 320 | 620 | err = genphy_resume(phydev); |
|---|
| 321 | 621 | if (err < 0) |
|---|
| 322 | 622 | goto err_put_dev; |
|---|
| .. | .. |
|---|
| 346 | 646 | struct dsa_switch *ds = dp->ds; |
|---|
| 347 | 647 | struct phy_device *phydev; |
|---|
| 348 | 648 | int port = dp->index; |
|---|
| 349 | | - int mode; |
|---|
| 649 | + phy_interface_t mode; |
|---|
| 350 | 650 | int err; |
|---|
| 351 | 651 | |
|---|
| 352 | 652 | err = of_phy_register_fixed_link(dn); |
|---|
| .. | .. |
|---|
| 359 | 659 | |
|---|
| 360 | 660 | phydev = of_phy_find_device(dn); |
|---|
| 361 | 661 | |
|---|
| 362 | | - mode = of_get_phy_mode(dn); |
|---|
| 363 | | - if (mode < 0) |
|---|
| 662 | + err = of_get_phy_mode(dn, &mode); |
|---|
| 663 | + if (err) |
|---|
| 364 | 664 | mode = PHY_INTERFACE_MODE_NA; |
|---|
| 365 | 665 | phydev->interface = mode; |
|---|
| 366 | 666 | |
|---|
| 367 | | - genphy_config_init(phydev); |
|---|
| 368 | 667 | genphy_read_status(phydev); |
|---|
| 369 | 668 | |
|---|
| 370 | 669 | if (ds->ops->adjust_link) |
|---|
| .. | .. |
|---|
| 375 | 674 | return 0; |
|---|
| 376 | 675 | } |
|---|
| 377 | 676 | |
|---|
| 677 | +static int dsa_port_phylink_register(struct dsa_port *dp) |
|---|
| 678 | +{ |
|---|
| 679 | + struct dsa_switch *ds = dp->ds; |
|---|
| 680 | + struct device_node *port_dn = dp->dn; |
|---|
| 681 | + phy_interface_t mode; |
|---|
| 682 | + int err; |
|---|
| 683 | + |
|---|
| 684 | + err = of_get_phy_mode(port_dn, &mode); |
|---|
| 685 | + if (err) |
|---|
| 686 | + mode = PHY_INTERFACE_MODE_NA; |
|---|
| 687 | + |
|---|
| 688 | + dp->pl_config.dev = ds->dev; |
|---|
| 689 | + dp->pl_config.type = PHYLINK_DEV; |
|---|
| 690 | + dp->pl_config.pcs_poll = ds->pcs_poll; |
|---|
| 691 | + |
|---|
| 692 | + dp->pl = phylink_create(&dp->pl_config, of_fwnode_handle(port_dn), |
|---|
| 693 | + mode, &dsa_port_phylink_mac_ops); |
|---|
| 694 | + if (IS_ERR(dp->pl)) { |
|---|
| 695 | + pr_err("error creating PHYLINK: %ld\n", PTR_ERR(dp->pl)); |
|---|
| 696 | + return PTR_ERR(dp->pl); |
|---|
| 697 | + } |
|---|
| 698 | + |
|---|
| 699 | + err = phylink_of_phy_connect(dp->pl, port_dn, 0); |
|---|
| 700 | + if (err && err != -ENODEV) { |
|---|
| 701 | + pr_err("could not attach to PHY: %d\n", err); |
|---|
| 702 | + goto err_phy_connect; |
|---|
| 703 | + } |
|---|
| 704 | + |
|---|
| 705 | + return 0; |
|---|
| 706 | + |
|---|
| 707 | +err_phy_connect: |
|---|
| 708 | + phylink_destroy(dp->pl); |
|---|
| 709 | + return err; |
|---|
| 710 | +} |
|---|
| 711 | + |
|---|
| 378 | 712 | int dsa_port_link_register_of(struct dsa_port *dp) |
|---|
| 379 | 713 | { |
|---|
| 714 | + struct dsa_switch *ds = dp->ds; |
|---|
| 715 | + struct device_node *phy_np; |
|---|
| 716 | + int port = dp->index; |
|---|
| 717 | + |
|---|
| 718 | + if (!ds->ops->adjust_link) { |
|---|
| 719 | + phy_np = of_parse_phandle(dp->dn, "phy-handle", 0); |
|---|
| 720 | + if (of_phy_is_fixed_link(dp->dn) || phy_np) { |
|---|
| 721 | + if (ds->ops->phylink_mac_link_down) |
|---|
| 722 | + ds->ops->phylink_mac_link_down(ds, port, |
|---|
| 723 | + MLO_AN_FIXED, PHY_INTERFACE_MODE_NA); |
|---|
| 724 | + of_node_put(phy_np); |
|---|
| 725 | + return dsa_port_phylink_register(dp); |
|---|
| 726 | + } |
|---|
| 727 | + of_node_put(phy_np); |
|---|
| 728 | + return 0; |
|---|
| 729 | + } |
|---|
| 730 | + |
|---|
| 731 | + dev_warn(ds->dev, |
|---|
| 732 | + "Using legacy PHYLIB callbacks. Please migrate to PHYLINK!\n"); |
|---|
| 733 | + |
|---|
| 380 | 734 | if (of_phy_is_fixed_link(dp->dn)) |
|---|
| 381 | 735 | return dsa_port_fixed_link_register_of(dp); |
|---|
| 382 | 736 | else |
|---|
| .. | .. |
|---|
| 385 | 739 | |
|---|
| 386 | 740 | void dsa_port_link_unregister_of(struct dsa_port *dp) |
|---|
| 387 | 741 | { |
|---|
| 742 | + struct dsa_switch *ds = dp->ds; |
|---|
| 743 | + |
|---|
| 744 | + if (!ds->ops->adjust_link && dp->pl) { |
|---|
| 745 | + rtnl_lock(); |
|---|
| 746 | + phylink_disconnect_phy(dp->pl); |
|---|
| 747 | + rtnl_unlock(); |
|---|
| 748 | + phylink_destroy(dp->pl); |
|---|
| 749 | + dp->pl = NULL; |
|---|
| 750 | + return; |
|---|
| 751 | + } |
|---|
| 752 | + |
|---|
| 388 | 753 | if (of_phy_is_fixed_link(dp->dn)) |
|---|
| 389 | 754 | of_phy_deregister_fixed_link(dp->dn); |
|---|
| 390 | 755 | else |
|---|