diff options
author | Neil Armstrong <narmstrong@baylibre.com> | 2015-12-07 07:57:35 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-12-07 16:35:50 -0500 |
commit | cda5c15b23fb9d683a491e8bd137d11d8552ac02 (patch) | |
tree | 795c5b3cb40f66415c0237c7602026b08ff78b18 /net/dsa/slave.c | |
parent | 679fb46c57859b59a70257477bfbdfc7edfac4f5 (diff) |
net: dsa: move dsa slave destroy code to slave.c
Move dsa slave dedicated code from dsa_switch_destroy to a new
dsa_slave_destroy function in slave.c.
Add the netif_carrier_off and phy_disconnect calls in order to
correctly cleanup the netdev state and PHY state machine.
Signed-off-by: Frode Isaksen <fisaksen@baylibre.com>
Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/dsa/slave.c')
-rw-r--r-- | net/dsa/slave.c | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/net/dsa/slave.c b/net/dsa/slave.c index 7bc787b095c8..1e9e9424a33d 100644 --- a/net/dsa/slave.c +++ b/net/dsa/slave.c | |||
@@ -1212,6 +1212,17 @@ int dsa_slave_create(struct dsa_switch *ds, struct device *parent, | |||
1212 | return 0; | 1212 | return 0; |
1213 | } | 1213 | } |
1214 | 1214 | ||
1215 | void dsa_slave_destroy(struct net_device *slave_dev) | ||
1216 | { | ||
1217 | struct dsa_slave_priv *p = netdev_priv(slave_dev); | ||
1218 | |||
1219 | netif_carrier_off(slave_dev); | ||
1220 | if (p->phy) | ||
1221 | phy_disconnect(p->phy); | ||
1222 | unregister_netdev(slave_dev); | ||
1223 | free_netdev(slave_dev); | ||
1224 | } | ||
1225 | |||
1215 | static bool dsa_slave_dev_check(struct net_device *dev) | 1226 | static bool dsa_slave_dev_check(struct net_device *dev) |
1216 | { | 1227 | { |
1217 | return dev->netdev_ops == &dsa_slave_netdev_ops; | 1228 | return dev->netdev_ops == &dsa_slave_netdev_ops; |