Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 1 | /* |
| 2 | * net/dsa/dsa2.c - Hardware switch handling, binding version 2 |
| 3 | * Copyright (c) 2008-2009 Marvell Semiconductor |
| 4 | * Copyright (c) 2013 Florian Fainelli <florian@openwrt.org> |
| 5 | * Copyright (c) 2016 Andrew Lunn <andrew@lunn.ch> |
| 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 | */ |
| 12 | |
| 13 | #include <linux/device.h> |
| 14 | #include <linux/err.h> |
| 15 | #include <linux/list.h> |
Andrew Lunn | c6e970a | 2017-03-28 23:45:06 +0200 | [diff] [blame] | 16 | #include <linux/netdevice.h> |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 17 | #include <linux/slab.h> |
| 18 | #include <linux/rtnetlink.h> |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 19 | #include <linux/of.h> |
| 20 | #include <linux/of_net.h> |
Andrew Lunn | c6e970a | 2017-03-28 23:45:06 +0200 | [diff] [blame] | 21 | #include <net/dsa.h> |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 22 | #include "dsa_priv.h" |
| 23 | |
| 24 | static LIST_HEAD(dsa_switch_trees); |
| 25 | static DEFINE_MUTEX(dsa2_mutex); |
| 26 | |
Andrew Lunn | 96567d5 | 2017-03-28 23:45:07 +0200 | [diff] [blame] | 27 | static const struct devlink_ops dsa_devlink_ops = { |
| 28 | }; |
| 29 | |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 30 | static struct dsa_switch_tree *dsa_get_dst(u32 tree) |
| 31 | { |
| 32 | struct dsa_switch_tree *dst; |
| 33 | |
| 34 | list_for_each_entry(dst, &dsa_switch_trees, list) |
Nikita Yushchenko | 7a99cd6 | 2016-11-28 09:48:48 +0300 | [diff] [blame] | 35 | if (dst->tree == tree) { |
| 36 | kref_get(&dst->refcount); |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 37 | return dst; |
Nikita Yushchenko | 7a99cd6 | 2016-11-28 09:48:48 +0300 | [diff] [blame] | 38 | } |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 39 | return NULL; |
| 40 | } |
| 41 | |
| 42 | static void dsa_free_dst(struct kref *ref) |
| 43 | { |
| 44 | struct dsa_switch_tree *dst = container_of(ref, struct dsa_switch_tree, |
| 45 | refcount); |
| 46 | |
| 47 | list_del(&dst->list); |
| 48 | kfree(dst); |
| 49 | } |
| 50 | |
| 51 | static void dsa_put_dst(struct dsa_switch_tree *dst) |
| 52 | { |
| 53 | kref_put(&dst->refcount, dsa_free_dst); |
| 54 | } |
| 55 | |
| 56 | static struct dsa_switch_tree *dsa_add_dst(u32 tree) |
| 57 | { |
| 58 | struct dsa_switch_tree *dst; |
| 59 | |
| 60 | dst = kzalloc(sizeof(*dst), GFP_KERNEL); |
| 61 | if (!dst) |
| 62 | return NULL; |
| 63 | dst->tree = tree; |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 64 | INIT_LIST_HEAD(&dst->list); |
| 65 | list_add_tail(&dsa_switch_trees, &dst->list); |
| 66 | kref_init(&dst->refcount); |
| 67 | |
| 68 | return dst; |
| 69 | } |
| 70 | |
| 71 | static void dsa_dst_add_ds(struct dsa_switch_tree *dst, |
| 72 | struct dsa_switch *ds, u32 index) |
| 73 | { |
| 74 | kref_get(&dst->refcount); |
| 75 | dst->ds[index] = ds; |
| 76 | } |
| 77 | |
| 78 | static void dsa_dst_del_ds(struct dsa_switch_tree *dst, |
| 79 | struct dsa_switch *ds, u32 index) |
| 80 | { |
| 81 | dst->ds[index] = NULL; |
| 82 | kref_put(&dst->refcount, dsa_free_dst); |
| 83 | } |
| 84 | |
Florian Fainelli | 71e0bbd | 2017-02-04 13:02:43 -0800 | [diff] [blame] | 85 | /* For platform data configurations, we need to have a valid name argument to |
| 86 | * differentiate a disabled port from an enabled one |
| 87 | */ |
Florian Fainelli | 293784a | 2017-01-26 10:45:52 -0800 | [diff] [blame] | 88 | static bool dsa_port_is_valid(struct dsa_port *port) |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 89 | { |
Florian Fainelli | 71e0bbd | 2017-02-04 13:02:43 -0800 | [diff] [blame] | 90 | return !!(port->dn || port->name); |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 91 | } |
| 92 | |
Florian Fainelli | 293784a | 2017-01-26 10:45:52 -0800 | [diff] [blame] | 93 | static bool dsa_port_is_dsa(struct dsa_port *port) |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 94 | { |
Florian Fainelli | 71e0bbd | 2017-02-04 13:02:43 -0800 | [diff] [blame] | 95 | if (port->name && !strcmp(port->name, "dsa")) |
| 96 | return true; |
| 97 | else |
| 98 | return !!of_parse_phandle(port->dn, "link", 0); |
Florian Fainelli | 293784a | 2017-01-26 10:45:52 -0800 | [diff] [blame] | 99 | } |
| 100 | |
| 101 | static bool dsa_port_is_cpu(struct dsa_port *port) |
| 102 | { |
Florian Fainelli | 71e0bbd | 2017-02-04 13:02:43 -0800 | [diff] [blame] | 103 | if (port->name && !strcmp(port->name, "cpu")) |
| 104 | return true; |
| 105 | else |
| 106 | return !!of_parse_phandle(port->dn, "ethernet", 0); |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 107 | } |
| 108 | |
Florian Fainelli | 3512a8e | 2017-01-26 10:45:53 -0800 | [diff] [blame] | 109 | static bool dsa_ds_find_port_dn(struct dsa_switch *ds, |
| 110 | struct device_node *port) |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 111 | { |
| 112 | u32 index; |
| 113 | |
Vivien Didelot | 26895e2 | 2017-01-27 15:29:37 -0500 | [diff] [blame] | 114 | for (index = 0; index < ds->num_ports; index++) |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 115 | if (ds->ports[index].dn == port) |
| 116 | return true; |
| 117 | return false; |
| 118 | } |
| 119 | |
Florian Fainelli | 3512a8e | 2017-01-26 10:45:53 -0800 | [diff] [blame] | 120 | static struct dsa_switch *dsa_dst_find_port_dn(struct dsa_switch_tree *dst, |
| 121 | struct device_node *port) |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 122 | { |
| 123 | struct dsa_switch *ds; |
| 124 | u32 index; |
| 125 | |
| 126 | for (index = 0; index < DSA_MAX_SWITCHES; index++) { |
| 127 | ds = dst->ds[index]; |
| 128 | if (!ds) |
| 129 | continue; |
| 130 | |
Florian Fainelli | 3512a8e | 2017-01-26 10:45:53 -0800 | [diff] [blame] | 131 | if (dsa_ds_find_port_dn(ds, port)) |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 132 | return ds; |
| 133 | } |
| 134 | |
| 135 | return NULL; |
| 136 | } |
| 137 | |
| 138 | static int dsa_port_complete(struct dsa_switch_tree *dst, |
| 139 | struct dsa_switch *src_ds, |
Florian Fainelli | 293784a | 2017-01-26 10:45:52 -0800 | [diff] [blame] | 140 | struct dsa_port *port, |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 141 | u32 src_port) |
| 142 | { |
| 143 | struct device_node *link; |
| 144 | int index; |
| 145 | struct dsa_switch *dst_ds; |
| 146 | |
| 147 | for (index = 0;; index++) { |
Florian Fainelli | 293784a | 2017-01-26 10:45:52 -0800 | [diff] [blame] | 148 | link = of_parse_phandle(port->dn, "link", index); |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 149 | if (!link) |
| 150 | break; |
| 151 | |
Florian Fainelli | 3512a8e | 2017-01-26 10:45:53 -0800 | [diff] [blame] | 152 | dst_ds = dsa_dst_find_port_dn(dst, link); |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 153 | of_node_put(link); |
| 154 | |
| 155 | if (!dst_ds) |
| 156 | return 1; |
| 157 | |
| 158 | src_ds->rtable[dst_ds->index] = src_port; |
| 159 | } |
| 160 | |
| 161 | return 0; |
| 162 | } |
| 163 | |
| 164 | /* A switch is complete if all the DSA ports phandles point to ports |
| 165 | * known in the tree. A return value of 1 means the tree is not |
| 166 | * complete. This is not an error condition. A value of 0 is |
| 167 | * success. |
| 168 | */ |
| 169 | static int dsa_ds_complete(struct dsa_switch_tree *dst, struct dsa_switch *ds) |
| 170 | { |
Florian Fainelli | 293784a | 2017-01-26 10:45:52 -0800 | [diff] [blame] | 171 | struct dsa_port *port; |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 172 | u32 index; |
| 173 | int err; |
| 174 | |
Vivien Didelot | 26895e2 | 2017-01-27 15:29:37 -0500 | [diff] [blame] | 175 | for (index = 0; index < ds->num_ports; index++) { |
Florian Fainelli | 293784a | 2017-01-26 10:45:52 -0800 | [diff] [blame] | 176 | port = &ds->ports[index]; |
| 177 | if (!dsa_port_is_valid(port)) |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 178 | continue; |
| 179 | |
| 180 | if (!dsa_port_is_dsa(port)) |
| 181 | continue; |
| 182 | |
| 183 | err = dsa_port_complete(dst, ds, port, index); |
| 184 | if (err != 0) |
| 185 | return err; |
| 186 | |
| 187 | ds->dsa_port_mask |= BIT(index); |
| 188 | } |
| 189 | |
| 190 | return 0; |
| 191 | } |
| 192 | |
| 193 | /* A tree is complete if all the DSA ports phandles point to ports |
| 194 | * known in the tree. A return value of 1 means the tree is not |
| 195 | * complete. This is not an error condition. A value of 0 is |
| 196 | * success. |
| 197 | */ |
| 198 | static int dsa_dst_complete(struct dsa_switch_tree *dst) |
| 199 | { |
| 200 | struct dsa_switch *ds; |
| 201 | u32 index; |
| 202 | int err; |
| 203 | |
| 204 | for (index = 0; index < DSA_MAX_SWITCHES; index++) { |
| 205 | ds = dst->ds[index]; |
| 206 | if (!ds) |
| 207 | continue; |
| 208 | |
| 209 | err = dsa_ds_complete(dst, ds); |
| 210 | if (err != 0) |
| 211 | return err; |
| 212 | } |
| 213 | |
| 214 | return 0; |
| 215 | } |
| 216 | |
Florian Fainelli | 293784a | 2017-01-26 10:45:52 -0800 | [diff] [blame] | 217 | static int dsa_dsa_port_apply(struct dsa_port *port, u32 index, |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 218 | struct dsa_switch *ds) |
| 219 | { |
| 220 | int err; |
| 221 | |
| 222 | err = dsa_cpu_dsa_setup(ds, ds->dev, port, index); |
| 223 | if (err) { |
| 224 | dev_warn(ds->dev, "Failed to setup dsa port %d: %d\n", |
| 225 | index, err); |
| 226 | return err; |
| 227 | } |
| 228 | |
Andrew Lunn | 96567d5 | 2017-03-28 23:45:07 +0200 | [diff] [blame] | 229 | memset(&ds->ports[index].devlink_port, 0, |
| 230 | sizeof(ds->ports[index].devlink_port)); |
| 231 | |
| 232 | return devlink_port_register(ds->devlink, |
| 233 | &ds->ports[index].devlink_port, |
| 234 | index); |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 235 | } |
| 236 | |
Florian Fainelli | 293784a | 2017-01-26 10:45:52 -0800 | [diff] [blame] | 237 | static void dsa_dsa_port_unapply(struct dsa_port *port, u32 index, |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 238 | struct dsa_switch *ds) |
| 239 | { |
Andrew Lunn | 96567d5 | 2017-03-28 23:45:07 +0200 | [diff] [blame] | 240 | devlink_port_unregister(&ds->ports[index].devlink_port); |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 241 | dsa_cpu_dsa_destroy(port); |
| 242 | } |
| 243 | |
Florian Fainelli | 293784a | 2017-01-26 10:45:52 -0800 | [diff] [blame] | 244 | static int dsa_cpu_port_apply(struct dsa_port *port, u32 index, |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 245 | struct dsa_switch *ds) |
| 246 | { |
| 247 | int err; |
| 248 | |
| 249 | err = dsa_cpu_dsa_setup(ds, ds->dev, port, index); |
| 250 | if (err) { |
| 251 | dev_warn(ds->dev, "Failed to setup cpu port %d: %d\n", |
| 252 | index, err); |
| 253 | return err; |
| 254 | } |
| 255 | |
| 256 | ds->cpu_port_mask |= BIT(index); |
| 257 | |
Andrew Lunn | 96567d5 | 2017-03-28 23:45:07 +0200 | [diff] [blame] | 258 | memset(&ds->ports[index].devlink_port, 0, |
| 259 | sizeof(ds->ports[index].devlink_port)); |
| 260 | err = devlink_port_register(ds->devlink, &ds->ports[index].devlink_port, |
| 261 | index); |
| 262 | return err; |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 263 | } |
| 264 | |
Florian Fainelli | 293784a | 2017-01-26 10:45:52 -0800 | [diff] [blame] | 265 | static void dsa_cpu_port_unapply(struct dsa_port *port, u32 index, |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 266 | struct dsa_switch *ds) |
| 267 | { |
Andrew Lunn | 96567d5 | 2017-03-28 23:45:07 +0200 | [diff] [blame] | 268 | devlink_port_unregister(&ds->ports[index].devlink_port); |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 269 | dsa_cpu_dsa_destroy(port); |
| 270 | ds->cpu_port_mask &= ~BIT(index); |
| 271 | |
| 272 | } |
| 273 | |
Florian Fainelli | 293784a | 2017-01-26 10:45:52 -0800 | [diff] [blame] | 274 | static int dsa_user_port_apply(struct dsa_port *port, u32 index, |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 275 | struct dsa_switch *ds) |
| 276 | { |
Florian Fainelli | 71e0bbd | 2017-02-04 13:02:43 -0800 | [diff] [blame] | 277 | const char *name = port->name; |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 278 | int err; |
| 279 | |
Florian Fainelli | 71e0bbd | 2017-02-04 13:02:43 -0800 | [diff] [blame] | 280 | if (port->dn) |
| 281 | name = of_get_property(port->dn, "label", NULL); |
Vivien Didelot | 9f91484 | 2017-01-09 18:13:51 -0500 | [diff] [blame] | 282 | if (!name) |
| 283 | name = "eth%d"; |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 284 | |
| 285 | err = dsa_slave_create(ds, ds->dev, index, name); |
| 286 | if (err) { |
| 287 | dev_warn(ds->dev, "Failed to create slave %d: %d\n", |
| 288 | index, err); |
Florian Fainelli | 382e1ee | 2017-02-07 23:10:13 -0800 | [diff] [blame] | 289 | ds->ports[index].netdev = NULL; |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 290 | return err; |
| 291 | } |
| 292 | |
Andrew Lunn | 96567d5 | 2017-03-28 23:45:07 +0200 | [diff] [blame] | 293 | memset(&ds->ports[index].devlink_port, 0, |
| 294 | sizeof(ds->ports[index].devlink_port)); |
| 295 | err = devlink_port_register(ds->devlink, &ds->ports[index].devlink_port, |
| 296 | index); |
| 297 | if (err) |
| 298 | return err; |
| 299 | |
| 300 | devlink_port_type_eth_set(&ds->ports[index].devlink_port, |
| 301 | ds->ports[index].netdev); |
| 302 | |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 303 | return 0; |
| 304 | } |
| 305 | |
Florian Fainelli | 293784a | 2017-01-26 10:45:52 -0800 | [diff] [blame] | 306 | static void dsa_user_port_unapply(struct dsa_port *port, u32 index, |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 307 | struct dsa_switch *ds) |
| 308 | { |
Andrew Lunn | 96567d5 | 2017-03-28 23:45:07 +0200 | [diff] [blame] | 309 | devlink_port_unregister(&ds->ports[index].devlink_port); |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 310 | if (ds->ports[index].netdev) { |
| 311 | dsa_slave_destroy(ds->ports[index].netdev); |
| 312 | ds->ports[index].netdev = NULL; |
Florian Fainelli | 6e830d8 | 2016-06-07 16:32:39 -0700 | [diff] [blame] | 313 | ds->enabled_port_mask &= ~(1 << index); |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 314 | } |
| 315 | } |
| 316 | |
| 317 | static int dsa_ds_apply(struct dsa_switch_tree *dst, struct dsa_switch *ds) |
| 318 | { |
Florian Fainelli | 293784a | 2017-01-26 10:45:52 -0800 | [diff] [blame] | 319 | struct dsa_port *port; |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 320 | u32 index; |
| 321 | int err; |
| 322 | |
Florian Fainelli | 6e830d8 | 2016-06-07 16:32:39 -0700 | [diff] [blame] | 323 | /* Initialize ds->phys_mii_mask before registering the slave MDIO bus |
Vivien Didelot | 9d490b4 | 2016-08-23 12:38:56 -0400 | [diff] [blame] | 324 | * driver and before ops->setup() has run, since the switch drivers and |
Florian Fainelli | 6e830d8 | 2016-06-07 16:32:39 -0700 | [diff] [blame] | 325 | * the slave MDIO bus driver rely on these values for probing PHY |
| 326 | * devices or not |
| 327 | */ |
| 328 | ds->phys_mii_mask = ds->enabled_port_mask; |
| 329 | |
Andrew Lunn | 96567d5 | 2017-03-28 23:45:07 +0200 | [diff] [blame] | 330 | /* Add the switch to devlink before calling setup, so that setup can |
| 331 | * add dpipe tables |
| 332 | */ |
| 333 | ds->devlink = devlink_alloc(&dsa_devlink_ops, 0); |
| 334 | if (!ds->devlink) |
| 335 | return -ENOMEM; |
| 336 | |
| 337 | err = devlink_register(ds->devlink, ds->dev); |
| 338 | if (err) |
| 339 | return err; |
| 340 | |
Vivien Didelot | 9d490b4 | 2016-08-23 12:38:56 -0400 | [diff] [blame] | 341 | err = ds->ops->setup(ds); |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 342 | if (err < 0) |
| 343 | return err; |
| 344 | |
Vivien Didelot | f515f19 | 2017-02-03 13:20:20 -0500 | [diff] [blame] | 345 | err = dsa_switch_register_notifier(ds); |
| 346 | if (err) |
| 347 | return err; |
| 348 | |
John Crispin | 092183d | 2016-09-19 15:28:01 +0200 | [diff] [blame] | 349 | if (ds->ops->set_addr) { |
| 350 | err = ds->ops->set_addr(ds, dst->master_netdev->dev_addr); |
| 351 | if (err < 0) |
| 352 | return err; |
| 353 | } |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 354 | |
Vivien Didelot | 9d490b4 | 2016-08-23 12:38:56 -0400 | [diff] [blame] | 355 | if (!ds->slave_mii_bus && ds->ops->phy_read) { |
Florian Fainelli | 1eb5944 | 2016-06-07 16:32:40 -0700 | [diff] [blame] | 356 | ds->slave_mii_bus = devm_mdiobus_alloc(ds->dev); |
| 357 | if (!ds->slave_mii_bus) |
| 358 | return -ENOMEM; |
| 359 | |
| 360 | dsa_slave_mii_bus_init(ds); |
| 361 | |
| 362 | err = mdiobus_register(ds->slave_mii_bus); |
| 363 | if (err < 0) |
| 364 | return err; |
| 365 | } |
| 366 | |
Vivien Didelot | 26895e2 | 2017-01-27 15:29:37 -0500 | [diff] [blame] | 367 | for (index = 0; index < ds->num_ports; index++) { |
Florian Fainelli | 293784a | 2017-01-26 10:45:52 -0800 | [diff] [blame] | 368 | port = &ds->ports[index]; |
| 369 | if (!dsa_port_is_valid(port)) |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 370 | continue; |
| 371 | |
| 372 | if (dsa_port_is_dsa(port)) { |
| 373 | err = dsa_dsa_port_apply(port, index, ds); |
| 374 | if (err) |
| 375 | return err; |
| 376 | continue; |
| 377 | } |
| 378 | |
| 379 | if (dsa_port_is_cpu(port)) { |
| 380 | err = dsa_cpu_port_apply(port, index, ds); |
| 381 | if (err) |
| 382 | return err; |
| 383 | continue; |
| 384 | } |
| 385 | |
| 386 | err = dsa_user_port_apply(port, index, ds); |
| 387 | if (err) |
| 388 | continue; |
| 389 | } |
| 390 | |
| 391 | return 0; |
| 392 | } |
| 393 | |
| 394 | static void dsa_ds_unapply(struct dsa_switch_tree *dst, struct dsa_switch *ds) |
| 395 | { |
Florian Fainelli | 293784a | 2017-01-26 10:45:52 -0800 | [diff] [blame] | 396 | struct dsa_port *port; |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 397 | u32 index; |
| 398 | |
Vivien Didelot | 26895e2 | 2017-01-27 15:29:37 -0500 | [diff] [blame] | 399 | for (index = 0; index < ds->num_ports; index++) { |
Florian Fainelli | 293784a | 2017-01-26 10:45:52 -0800 | [diff] [blame] | 400 | port = &ds->ports[index]; |
| 401 | if (!dsa_port_is_valid(port)) |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 402 | continue; |
| 403 | |
| 404 | if (dsa_port_is_dsa(port)) { |
| 405 | dsa_dsa_port_unapply(port, index, ds); |
| 406 | continue; |
| 407 | } |
| 408 | |
| 409 | if (dsa_port_is_cpu(port)) { |
| 410 | dsa_cpu_port_unapply(port, index, ds); |
| 411 | continue; |
| 412 | } |
| 413 | |
| 414 | dsa_user_port_unapply(port, index, ds); |
| 415 | } |
Florian Fainelli | 1eb5944 | 2016-06-07 16:32:40 -0700 | [diff] [blame] | 416 | |
Vivien Didelot | 9d490b4 | 2016-08-23 12:38:56 -0400 | [diff] [blame] | 417 | if (ds->slave_mii_bus && ds->ops->phy_read) |
Florian Fainelli | 1eb5944 | 2016-06-07 16:32:40 -0700 | [diff] [blame] | 418 | mdiobus_unregister(ds->slave_mii_bus); |
Vivien Didelot | f515f19 | 2017-02-03 13:20:20 -0500 | [diff] [blame] | 419 | |
| 420 | dsa_switch_unregister_notifier(ds); |
Andrew Lunn | 96567d5 | 2017-03-28 23:45:07 +0200 | [diff] [blame] | 421 | |
| 422 | if (ds->devlink) { |
| 423 | devlink_unregister(ds->devlink); |
| 424 | devlink_free(ds->devlink); |
| 425 | ds->devlink = NULL; |
| 426 | } |
| 427 | |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 428 | } |
| 429 | |
| 430 | static int dsa_dst_apply(struct dsa_switch_tree *dst) |
| 431 | { |
| 432 | struct dsa_switch *ds; |
| 433 | u32 index; |
| 434 | int err; |
| 435 | |
| 436 | for (index = 0; index < DSA_MAX_SWITCHES; index++) { |
| 437 | ds = dst->ds[index]; |
| 438 | if (!ds) |
| 439 | continue; |
| 440 | |
| 441 | err = dsa_ds_apply(dst, ds); |
| 442 | if (err) |
| 443 | return err; |
| 444 | } |
| 445 | |
Vivien Didelot | 9520ed8 | 2017-01-17 20:41:39 -0500 | [diff] [blame] | 446 | if (dst->cpu_switch) { |
| 447 | err = dsa_cpu_port_ethtool_setup(dst->cpu_switch); |
Florian Fainelli | faf3a93 | 2017-01-09 11:58:34 -0800 | [diff] [blame] | 448 | if (err) |
| 449 | return err; |
| 450 | } |
Florian Fainelli | 0c73c52 | 2016-06-07 16:32:42 -0700 | [diff] [blame] | 451 | |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 452 | /* If we use a tagging format that doesn't have an ethertype |
| 453 | * field, make sure that all packets from this point on get |
| 454 | * sent to the tag format's receive function. |
| 455 | */ |
| 456 | wmb(); |
| 457 | dst->master_netdev->dsa_ptr = (void *)dst; |
| 458 | dst->applied = true; |
| 459 | |
| 460 | return 0; |
| 461 | } |
| 462 | |
| 463 | static void dsa_dst_unapply(struct dsa_switch_tree *dst) |
| 464 | { |
| 465 | struct dsa_switch *ds; |
| 466 | u32 index; |
| 467 | |
| 468 | if (!dst->applied) |
| 469 | return; |
| 470 | |
| 471 | dst->master_netdev->dsa_ptr = NULL; |
| 472 | |
| 473 | /* If we used a tagging format that doesn't have an ethertype |
| 474 | * field, make sure that all packets from this point get sent |
| 475 | * without the tag and go through the regular receive path. |
| 476 | */ |
| 477 | wmb(); |
| 478 | |
| 479 | for (index = 0; index < DSA_MAX_SWITCHES; index++) { |
| 480 | ds = dst->ds[index]; |
| 481 | if (!ds) |
| 482 | continue; |
| 483 | |
| 484 | dsa_ds_unapply(dst, ds); |
| 485 | } |
| 486 | |
Vivien Didelot | 9520ed8 | 2017-01-17 20:41:39 -0500 | [diff] [blame] | 487 | if (dst->cpu_switch) |
| 488 | dsa_cpu_port_ethtool_restore(dst->cpu_switch); |
Florian Fainelli | 0c73c52 | 2016-06-07 16:32:42 -0700 | [diff] [blame] | 489 | |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 490 | pr_info("DSA: tree %d unapplied\n", dst->tree); |
| 491 | dst->applied = false; |
| 492 | } |
| 493 | |
Florian Fainelli | 293784a | 2017-01-26 10:45:52 -0800 | [diff] [blame] | 494 | static int dsa_cpu_parse(struct dsa_port *port, u32 index, |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 495 | struct dsa_switch_tree *dst, |
| 496 | struct dsa_switch *ds) |
| 497 | { |
Andrew Lunn | 7b31436 | 2016-08-22 16:01:01 +0200 | [diff] [blame] | 498 | enum dsa_tag_protocol tag_protocol; |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 499 | struct net_device *ethernet_dev; |
| 500 | struct device_node *ethernet; |
| 501 | |
Florian Fainelli | 71e0bbd | 2017-02-04 13:02:43 -0800 | [diff] [blame] | 502 | if (port->dn) { |
| 503 | ethernet = of_parse_phandle(port->dn, "ethernet", 0); |
| 504 | if (!ethernet) |
| 505 | return -EINVAL; |
| 506 | ethernet_dev = of_find_net_device_by_node(ethernet); |
| 507 | } else { |
| 508 | ethernet_dev = dsa_dev_to_net_device(ds->cd->netdev[index]); |
| 509 | dev_put(ethernet_dev); |
| 510 | } |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 511 | |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 512 | if (!ethernet_dev) |
| 513 | return -EPROBE_DEFER; |
| 514 | |
| 515 | if (!ds->master_netdev) |
| 516 | ds->master_netdev = ethernet_dev; |
| 517 | |
| 518 | if (!dst->master_netdev) |
| 519 | dst->master_netdev = ethernet_dev; |
| 520 | |
Vivien Didelot | b22de49 | 2017-01-17 20:41:38 -0500 | [diff] [blame] | 521 | if (!dst->cpu_switch) { |
| 522 | dst->cpu_switch = ds; |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 523 | dst->cpu_port = index; |
| 524 | } |
| 525 | |
Vivien Didelot | 9d490b4 | 2016-08-23 12:38:56 -0400 | [diff] [blame] | 526 | tag_protocol = ds->ops->get_tag_protocol(ds); |
Andrew Lunn | 7b31436 | 2016-08-22 16:01:01 +0200 | [diff] [blame] | 527 | dst->tag_ops = dsa_resolve_tag_protocol(tag_protocol); |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 528 | if (IS_ERR(dst->tag_ops)) { |
| 529 | dev_warn(ds->dev, "No tagger for this switch\n"); |
| 530 | return PTR_ERR(dst->tag_ops); |
| 531 | } |
| 532 | |
| 533 | dst->rcv = dst->tag_ops->rcv; |
| 534 | |
| 535 | return 0; |
| 536 | } |
| 537 | |
| 538 | static int dsa_ds_parse(struct dsa_switch_tree *dst, struct dsa_switch *ds) |
| 539 | { |
Florian Fainelli | 293784a | 2017-01-26 10:45:52 -0800 | [diff] [blame] | 540 | struct dsa_port *port; |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 541 | u32 index; |
| 542 | int err; |
| 543 | |
Vivien Didelot | 26895e2 | 2017-01-27 15:29:37 -0500 | [diff] [blame] | 544 | for (index = 0; index < ds->num_ports; index++) { |
Florian Fainelli | 293784a | 2017-01-26 10:45:52 -0800 | [diff] [blame] | 545 | port = &ds->ports[index]; |
| 546 | if (!dsa_port_is_valid(port)) |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 547 | continue; |
| 548 | |
| 549 | if (dsa_port_is_cpu(port)) { |
| 550 | err = dsa_cpu_parse(port, index, dst, ds); |
| 551 | if (err) |
| 552 | return err; |
| 553 | } |
| 554 | } |
| 555 | |
| 556 | pr_info("DSA: switch %d %d parsed\n", dst->tree, ds->index); |
| 557 | |
| 558 | return 0; |
| 559 | } |
| 560 | |
| 561 | static int dsa_dst_parse(struct dsa_switch_tree *dst) |
| 562 | { |
| 563 | struct dsa_switch *ds; |
| 564 | u32 index; |
| 565 | int err; |
| 566 | |
| 567 | for (index = 0; index < DSA_MAX_SWITCHES; index++) { |
| 568 | ds = dst->ds[index]; |
| 569 | if (!ds) |
| 570 | continue; |
| 571 | |
| 572 | err = dsa_ds_parse(dst, ds); |
| 573 | if (err) |
| 574 | return err; |
| 575 | } |
| 576 | |
| 577 | if (!dst->master_netdev) { |
| 578 | pr_warn("Tree has no master device\n"); |
| 579 | return -EINVAL; |
| 580 | } |
| 581 | |
| 582 | pr_info("DSA: tree %d parsed\n", dst->tree); |
| 583 | |
| 584 | return 0; |
| 585 | } |
| 586 | |
| 587 | static int dsa_parse_ports_dn(struct device_node *ports, struct dsa_switch *ds) |
| 588 | { |
| 589 | struct device_node *port; |
| 590 | int err; |
| 591 | u32 reg; |
| 592 | |
| 593 | for_each_available_child_of_node(ports, port) { |
| 594 | err = of_property_read_u32(port, "reg", ®); |
| 595 | if (err) |
| 596 | return err; |
| 597 | |
Vivien Didelot | 26895e2 | 2017-01-27 15:29:37 -0500 | [diff] [blame] | 598 | if (reg >= ds->num_ports) |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 599 | return -EINVAL; |
| 600 | |
| 601 | ds->ports[reg].dn = port; |
Florian Fainelli | 6e830d8 | 2016-06-07 16:32:39 -0700 | [diff] [blame] | 602 | |
Vivien Didelot | 9d490b4 | 2016-08-23 12:38:56 -0400 | [diff] [blame] | 603 | /* Initialize enabled_port_mask now for ops->setup() |
Florian Fainelli | 6e830d8 | 2016-06-07 16:32:39 -0700 | [diff] [blame] | 604 | * to have access to a correct value, just like what |
| 605 | * net/dsa/dsa.c::dsa_switch_setup_one does. |
| 606 | */ |
Florian Fainelli | 293784a | 2017-01-26 10:45:52 -0800 | [diff] [blame] | 607 | if (!dsa_port_is_cpu(&ds->ports[reg])) |
Florian Fainelli | 6e830d8 | 2016-06-07 16:32:39 -0700 | [diff] [blame] | 608 | ds->enabled_port_mask |= 1 << reg; |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 609 | } |
| 610 | |
| 611 | return 0; |
| 612 | } |
| 613 | |
Florian Fainelli | 71e0bbd | 2017-02-04 13:02:43 -0800 | [diff] [blame] | 614 | static int dsa_parse_ports(struct dsa_chip_data *cd, struct dsa_switch *ds) |
| 615 | { |
| 616 | bool valid_name_found = false; |
| 617 | unsigned int i; |
| 618 | |
| 619 | for (i = 0; i < DSA_MAX_PORTS; i++) { |
| 620 | if (!cd->port_names[i]) |
| 621 | continue; |
| 622 | |
| 623 | ds->ports[i].name = cd->port_names[i]; |
| 624 | |
| 625 | /* Initialize enabled_port_mask now for drv->setup() |
| 626 | * to have access to a correct value, just like what |
| 627 | * net/dsa/dsa.c::dsa_switch_setup_one does. |
| 628 | */ |
| 629 | if (!dsa_port_is_cpu(&ds->ports[i])) |
| 630 | ds->enabled_port_mask |= 1 << i; |
| 631 | |
| 632 | valid_name_found = true; |
| 633 | } |
| 634 | |
| 635 | if (!valid_name_found && i == DSA_MAX_PORTS) |
| 636 | return -EINVAL; |
| 637 | |
| 638 | return 0; |
| 639 | } |
| 640 | |
Florian Fainelli | 3512a8e | 2017-01-26 10:45:53 -0800 | [diff] [blame] | 641 | static int dsa_parse_member_dn(struct device_node *np, u32 *tree, u32 *index) |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 642 | { |
| 643 | int err; |
| 644 | |
| 645 | *tree = *index = 0; |
| 646 | |
| 647 | err = of_property_read_u32_index(np, "dsa,member", 0, tree); |
| 648 | if (err) { |
| 649 | /* Does not exist, but it is optional */ |
| 650 | if (err == -EINVAL) |
| 651 | return 0; |
| 652 | return err; |
| 653 | } |
| 654 | |
| 655 | err = of_property_read_u32_index(np, "dsa,member", 1, index); |
| 656 | if (err) |
| 657 | return err; |
| 658 | |
| 659 | if (*index >= DSA_MAX_SWITCHES) |
| 660 | return -EINVAL; |
| 661 | |
| 662 | return 0; |
| 663 | } |
| 664 | |
Florian Fainelli | 71e0bbd | 2017-02-04 13:02:43 -0800 | [diff] [blame] | 665 | static int dsa_parse_member(struct dsa_chip_data *pd, u32 *tree, u32 *index) |
| 666 | { |
| 667 | if (!pd) |
| 668 | return -ENODEV; |
| 669 | |
| 670 | /* We do not support complex trees with dsa_chip_data */ |
| 671 | *tree = 0; |
| 672 | *index = 0; |
| 673 | |
| 674 | return 0; |
| 675 | } |
| 676 | |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 677 | static struct device_node *dsa_get_ports(struct dsa_switch *ds, |
| 678 | struct device_node *np) |
| 679 | { |
| 680 | struct device_node *ports; |
| 681 | |
| 682 | ports = of_get_child_by_name(np, "ports"); |
| 683 | if (!ports) { |
| 684 | dev_err(ds->dev, "no ports child node found\n"); |
| 685 | return ERR_PTR(-EINVAL); |
| 686 | } |
| 687 | |
| 688 | return ports; |
| 689 | } |
| 690 | |
Florian Fainelli | 55ed0ce | 2017-01-26 10:45:51 -0800 | [diff] [blame] | 691 | static int _dsa_register_switch(struct dsa_switch *ds, struct device *dev) |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 692 | { |
Florian Fainelli | 71e0bbd | 2017-02-04 13:02:43 -0800 | [diff] [blame] | 693 | struct dsa_chip_data *pdata = dev->platform_data; |
Florian Fainelli | 55ed0ce | 2017-01-26 10:45:51 -0800 | [diff] [blame] | 694 | struct device_node *np = dev->of_node; |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 695 | struct dsa_switch_tree *dst; |
Florian Fainelli | bc1727d | 2017-01-26 10:45:54 -0800 | [diff] [blame] | 696 | struct device_node *ports; |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 697 | u32 tree, index; |
Vivien Didelot | d390238 | 2016-07-06 20:03:54 -0400 | [diff] [blame] | 698 | int i, err; |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 699 | |
Florian Fainelli | 71e0bbd | 2017-02-04 13:02:43 -0800 | [diff] [blame] | 700 | if (np) { |
| 701 | err = dsa_parse_member_dn(np, &tree, &index); |
| 702 | if (err) |
| 703 | return err; |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 704 | |
Florian Fainelli | 71e0bbd | 2017-02-04 13:02:43 -0800 | [diff] [blame] | 705 | ports = dsa_get_ports(ds, np); |
| 706 | if (IS_ERR(ports)) |
| 707 | return PTR_ERR(ports); |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 708 | |
Florian Fainelli | 71e0bbd | 2017-02-04 13:02:43 -0800 | [diff] [blame] | 709 | err = dsa_parse_ports_dn(ports, ds); |
| 710 | if (err) |
| 711 | return err; |
| 712 | } else { |
| 713 | err = dsa_parse_member(pdata, &tree, &index); |
| 714 | if (err) |
| 715 | return err; |
| 716 | |
| 717 | err = dsa_parse_ports(pdata, ds); |
| 718 | if (err) |
| 719 | return err; |
| 720 | } |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 721 | |
| 722 | dst = dsa_get_dst(tree); |
| 723 | if (!dst) { |
| 724 | dst = dsa_add_dst(tree); |
| 725 | if (!dst) |
| 726 | return -ENOMEM; |
| 727 | } |
| 728 | |
| 729 | if (dst->ds[index]) { |
| 730 | err = -EBUSY; |
| 731 | goto out; |
| 732 | } |
| 733 | |
| 734 | ds->dst = dst; |
| 735 | ds->index = index; |
Florian Fainelli | 71e0bbd | 2017-02-04 13:02:43 -0800 | [diff] [blame] | 736 | ds->cd = pdata; |
Vivien Didelot | d390238 | 2016-07-06 20:03:54 -0400 | [diff] [blame] | 737 | |
| 738 | /* Initialize the routing table */ |
| 739 | for (i = 0; i < DSA_MAX_SWITCHES; ++i) |
| 740 | ds->rtable[i] = DSA_RTABLE_NONE; |
| 741 | |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 742 | dsa_dst_add_ds(dst, ds, index); |
| 743 | |
| 744 | err = dsa_dst_complete(dst); |
| 745 | if (err < 0) |
| 746 | goto out_del_dst; |
| 747 | |
| 748 | if (err == 1) { |
| 749 | /* Not all switches registered yet */ |
| 750 | err = 0; |
| 751 | goto out; |
| 752 | } |
| 753 | |
| 754 | if (dst->applied) { |
| 755 | pr_info("DSA: Disjoint trees?\n"); |
| 756 | return -EINVAL; |
| 757 | } |
| 758 | |
| 759 | err = dsa_dst_parse(dst); |
Volodymyr Bendiuga | 5e6eb45 | 2017-01-05 11:10:13 +0100 | [diff] [blame] | 760 | if (err) { |
| 761 | if (err == -EPROBE_DEFER) { |
| 762 | dsa_dst_del_ds(dst, ds, ds->index); |
| 763 | return err; |
| 764 | } |
| 765 | |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 766 | goto out_del_dst; |
Volodymyr Bendiuga | 5e6eb45 | 2017-01-05 11:10:13 +0100 | [diff] [blame] | 767 | } |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 768 | |
| 769 | err = dsa_dst_apply(dst); |
| 770 | if (err) { |
| 771 | dsa_dst_unapply(dst); |
| 772 | goto out_del_dst; |
| 773 | } |
| 774 | |
| 775 | dsa_put_dst(dst); |
| 776 | return 0; |
| 777 | |
| 778 | out_del_dst: |
| 779 | dsa_dst_del_ds(dst, ds, ds->index); |
| 780 | out: |
| 781 | dsa_put_dst(dst); |
| 782 | |
| 783 | return err; |
| 784 | } |
| 785 | |
Vivien Didelot | a0c0216 | 2017-01-27 15:29:36 -0500 | [diff] [blame] | 786 | struct dsa_switch *dsa_switch_alloc(struct device *dev, size_t n) |
| 787 | { |
| 788 | size_t size = sizeof(struct dsa_switch) + n * sizeof(struct dsa_port); |
| 789 | struct dsa_switch *ds; |
Vivien Didelot | 818be84 | 2017-01-27 15:29:38 -0500 | [diff] [blame] | 790 | int i; |
Vivien Didelot | a0c0216 | 2017-01-27 15:29:36 -0500 | [diff] [blame] | 791 | |
| 792 | ds = devm_kzalloc(dev, size, GFP_KERNEL); |
| 793 | if (!ds) |
| 794 | return NULL; |
| 795 | |
| 796 | ds->dev = dev; |
| 797 | ds->num_ports = n; |
| 798 | |
Vivien Didelot | 818be84 | 2017-01-27 15:29:38 -0500 | [diff] [blame] | 799 | for (i = 0; i < ds->num_ports; ++i) { |
| 800 | ds->ports[i].index = i; |
| 801 | ds->ports[i].ds = ds; |
| 802 | } |
| 803 | |
Vivien Didelot | a0c0216 | 2017-01-27 15:29:36 -0500 | [diff] [blame] | 804 | return ds; |
| 805 | } |
| 806 | EXPORT_SYMBOL_GPL(dsa_switch_alloc); |
| 807 | |
Florian Fainelli | 55ed0ce | 2017-01-26 10:45:51 -0800 | [diff] [blame] | 808 | int dsa_register_switch(struct dsa_switch *ds, struct device *dev) |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 809 | { |
| 810 | int err; |
| 811 | |
| 812 | mutex_lock(&dsa2_mutex); |
Florian Fainelli | 55ed0ce | 2017-01-26 10:45:51 -0800 | [diff] [blame] | 813 | err = _dsa_register_switch(ds, dev); |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 814 | mutex_unlock(&dsa2_mutex); |
| 815 | |
| 816 | return err; |
| 817 | } |
| 818 | EXPORT_SYMBOL_GPL(dsa_register_switch); |
| 819 | |
Wei Yongjun | 85c22ba | 2016-07-12 15:24:10 +0000 | [diff] [blame] | 820 | static void _dsa_unregister_switch(struct dsa_switch *ds) |
Andrew Lunn | 83c0afa | 2016-06-04 21:17:07 +0200 | [diff] [blame] | 821 | { |
| 822 | struct dsa_switch_tree *dst = ds->dst; |
| 823 | |
| 824 | dsa_dst_unapply(dst); |
| 825 | |
| 826 | dsa_dst_del_ds(dst, ds, ds->index); |
| 827 | } |
| 828 | |
| 829 | void dsa_unregister_switch(struct dsa_switch *ds) |
| 830 | { |
| 831 | mutex_lock(&dsa2_mutex); |
| 832 | _dsa_unregister_switch(ds); |
| 833 | mutex_unlock(&dsa2_mutex); |
| 834 | } |
| 835 | EXPORT_SYMBOL_GPL(dsa_unregister_switch); |