12874c5fdSThomas Gleixner // SPDX-License-Identifier: GPL-2.0-or-later 2f2f23566SVivien Didelot /* 3f2f23566SVivien Didelot * Handling of a master device, switching frames via its switch fabric CPU port 4f2f23566SVivien Didelot * 5f2f23566SVivien Didelot * Copyright (c) 2017 Savoir-faire Linux Inc. 6f2f23566SVivien Didelot * Vivien Didelot <vivien.didelot@savoirfairelinux.com> 7f2f23566SVivien Didelot */ 8f2f23566SVivien Didelot 9f2f23566SVivien Didelot #include "dsa_priv.h" 10f2f23566SVivien Didelot 1148e23311SVivien Didelot static int dsa_master_get_regs_len(struct net_device *dev) 1248e23311SVivien Didelot { 1348e23311SVivien Didelot struct dsa_port *cpu_dp = dev->dsa_ptr; 1448e23311SVivien Didelot const struct ethtool_ops *ops = cpu_dp->orig_ethtool_ops; 1548e23311SVivien Didelot struct dsa_switch *ds = cpu_dp->ds; 1648e23311SVivien Didelot int port = cpu_dp->index; 1748e23311SVivien Didelot int ret = 0; 1848e23311SVivien Didelot int len; 1948e23311SVivien Didelot 2048e23311SVivien Didelot if (ops->get_regs_len) { 2148e23311SVivien Didelot len = ops->get_regs_len(dev); 2248e23311SVivien Didelot if (len < 0) 2348e23311SVivien Didelot return len; 2448e23311SVivien Didelot ret += len; 2548e23311SVivien Didelot } 2648e23311SVivien Didelot 2748e23311SVivien Didelot ret += sizeof(struct ethtool_drvinfo); 2848e23311SVivien Didelot ret += sizeof(struct ethtool_regs); 2948e23311SVivien Didelot 3048e23311SVivien Didelot if (ds->ops->get_regs_len) { 3148e23311SVivien Didelot len = ds->ops->get_regs_len(ds, port); 3248e23311SVivien Didelot if (len < 0) 3348e23311SVivien Didelot return len; 3448e23311SVivien Didelot ret += len; 3548e23311SVivien Didelot } 3648e23311SVivien Didelot 3748e23311SVivien Didelot return ret; 3848e23311SVivien Didelot } 3948e23311SVivien Didelot 4048e23311SVivien Didelot static void dsa_master_get_regs(struct net_device *dev, 4148e23311SVivien Didelot struct ethtool_regs *regs, void *data) 4248e23311SVivien Didelot { 4348e23311SVivien Didelot struct dsa_port *cpu_dp = dev->dsa_ptr; 4448e23311SVivien Didelot const struct ethtool_ops *ops = cpu_dp->orig_ethtool_ops; 4548e23311SVivien Didelot struct dsa_switch *ds = cpu_dp->ds; 4648e23311SVivien Didelot struct ethtool_drvinfo *cpu_info; 4748e23311SVivien Didelot struct ethtool_regs *cpu_regs; 4848e23311SVivien Didelot int port = cpu_dp->index; 4948e23311SVivien Didelot int len; 5048e23311SVivien Didelot 5148e23311SVivien Didelot if (ops->get_regs_len && ops->get_regs) { 5248e23311SVivien Didelot len = ops->get_regs_len(dev); 5348e23311SVivien Didelot if (len < 0) 5448e23311SVivien Didelot return; 5548e23311SVivien Didelot regs->len = len; 5648e23311SVivien Didelot ops->get_regs(dev, regs, data); 5748e23311SVivien Didelot data += regs->len; 5848e23311SVivien Didelot } 5948e23311SVivien Didelot 6048e23311SVivien Didelot cpu_info = (struct ethtool_drvinfo *)data; 6148e23311SVivien Didelot strlcpy(cpu_info->driver, "dsa", sizeof(cpu_info->driver)); 6248e23311SVivien Didelot data += sizeof(*cpu_info); 6348e23311SVivien Didelot cpu_regs = (struct ethtool_regs *)data; 6448e23311SVivien Didelot data += sizeof(*cpu_regs); 6548e23311SVivien Didelot 6648e23311SVivien Didelot if (ds->ops->get_regs_len && ds->ops->get_regs) { 6748e23311SVivien Didelot len = ds->ops->get_regs_len(ds, port); 6848e23311SVivien Didelot if (len < 0) 6948e23311SVivien Didelot return; 7048e23311SVivien Didelot cpu_regs->len = len; 7148e23311SVivien Didelot ds->ops->get_regs(ds, port, cpu_regs, data); 7248e23311SVivien Didelot } 7348e23311SVivien Didelot } 7448e23311SVivien Didelot 75f2f23566SVivien Didelot static void dsa_master_get_ethtool_stats(struct net_device *dev, 76f2f23566SVivien Didelot struct ethtool_stats *stats, 77f2f23566SVivien Didelot uint64_t *data) 78f2f23566SVivien Didelot { 792f657a60SVivien Didelot struct dsa_port *cpu_dp = dev->dsa_ptr; 807ec764eeSVivien Didelot const struct ethtool_ops *ops = cpu_dp->orig_ethtool_ops; 817ec764eeSVivien Didelot struct dsa_switch *ds = cpu_dp->ds; 827ec764eeSVivien Didelot int port = cpu_dp->index; 83f2f23566SVivien Didelot int count = 0; 84f2f23566SVivien Didelot 851d1e79f1SFlorian Fainelli if (ops->get_sset_count && ops->get_ethtool_stats) { 86f2f23566SVivien Didelot count = ops->get_sset_count(dev, ETH_SS_STATS); 87f2f23566SVivien Didelot ops->get_ethtool_stats(dev, stats, data); 88f2f23566SVivien Didelot } 89f2f23566SVivien Didelot 90f2f23566SVivien Didelot if (ds->ops->get_ethtool_stats) 917ec764eeSVivien Didelot ds->ops->get_ethtool_stats(ds, port, data + count); 92f2f23566SVivien Didelot } 93f2f23566SVivien Didelot 94cf963573SFlorian Fainelli static void dsa_master_get_ethtool_phy_stats(struct net_device *dev, 95cf963573SFlorian Fainelli struct ethtool_stats *stats, 96cf963573SFlorian Fainelli uint64_t *data) 97cf963573SFlorian Fainelli { 98cf963573SFlorian Fainelli struct dsa_port *cpu_dp = dev->dsa_ptr; 99cf963573SFlorian Fainelli const struct ethtool_ops *ops = cpu_dp->orig_ethtool_ops; 100cf963573SFlorian Fainelli struct dsa_switch *ds = cpu_dp->ds; 101cf963573SFlorian Fainelli int port = cpu_dp->index; 102cf963573SFlorian Fainelli int count = 0; 103cf963573SFlorian Fainelli 104cf963573SFlorian Fainelli if (dev->phydev && !ops->get_ethtool_phy_stats) { 105cf963573SFlorian Fainelli count = phy_ethtool_get_sset_count(dev->phydev); 106cf963573SFlorian Fainelli if (count >= 0) 107cf963573SFlorian Fainelli phy_ethtool_get_stats(dev->phydev, stats, data); 108cf963573SFlorian Fainelli } else if (ops->get_sset_count && ops->get_ethtool_phy_stats) { 109cf963573SFlorian Fainelli count = ops->get_sset_count(dev, ETH_SS_PHY_STATS); 110cf963573SFlorian Fainelli ops->get_ethtool_phy_stats(dev, stats, data); 111cf963573SFlorian Fainelli } 112cf963573SFlorian Fainelli 113cf963573SFlorian Fainelli if (count < 0) 114cf963573SFlorian Fainelli count = 0; 115cf963573SFlorian Fainelli 116cf963573SFlorian Fainelli if (ds->ops->get_ethtool_phy_stats) 117cf963573SFlorian Fainelli ds->ops->get_ethtool_phy_stats(ds, port, data + count); 118cf963573SFlorian Fainelli } 119cf963573SFlorian Fainelli 120f2f23566SVivien Didelot static int dsa_master_get_sset_count(struct net_device *dev, int sset) 121f2f23566SVivien Didelot { 1222f657a60SVivien Didelot struct dsa_port *cpu_dp = dev->dsa_ptr; 1237ec764eeSVivien Didelot const struct ethtool_ops *ops = cpu_dp->orig_ethtool_ops; 1247ec764eeSVivien Didelot struct dsa_switch *ds = cpu_dp->ds; 125f2f23566SVivien Didelot int count = 0; 126f2f23566SVivien Didelot 127cf963573SFlorian Fainelli if (sset == ETH_SS_PHY_STATS && dev->phydev && 128cf963573SFlorian Fainelli !ops->get_ethtool_phy_stats) 129cf963573SFlorian Fainelli count = phy_ethtool_get_sset_count(dev->phydev); 130cf963573SFlorian Fainelli else if (ops->get_sset_count) 13189f09048SFlorian Fainelli count = ops->get_sset_count(dev, sset); 132cf963573SFlorian Fainelli 13389f09048SFlorian Fainelli if (count < 0) 13489f09048SFlorian Fainelli count = 0; 135f2f23566SVivien Didelot 13689f09048SFlorian Fainelli if (ds->ops->get_sset_count) 13789f09048SFlorian Fainelli count += ds->ops->get_sset_count(ds, cpu_dp->index, sset); 138f2f23566SVivien Didelot 139f2f23566SVivien Didelot return count; 140f2f23566SVivien Didelot } 141f2f23566SVivien Didelot 142f2f23566SVivien Didelot static void dsa_master_get_strings(struct net_device *dev, uint32_t stringset, 143f2f23566SVivien Didelot uint8_t *data) 144f2f23566SVivien Didelot { 1452f657a60SVivien Didelot struct dsa_port *cpu_dp = dev->dsa_ptr; 1467ec764eeSVivien Didelot const struct ethtool_ops *ops = cpu_dp->orig_ethtool_ops; 1477ec764eeSVivien Didelot struct dsa_switch *ds = cpu_dp->ds; 1487ec764eeSVivien Didelot int port = cpu_dp->index; 149f2f23566SVivien Didelot int len = ETH_GSTRING_LEN; 150a269333fSDan Carpenter int mcount = 0, count, i; 151f2f23566SVivien Didelot uint8_t pfx[4]; 152f2f23566SVivien Didelot uint8_t *ndata; 153f2f23566SVivien Didelot 1547ec764eeSVivien Didelot snprintf(pfx, sizeof(pfx), "p%.2d", port); 155f2f23566SVivien Didelot /* We do not want to be NULL-terminated, since this is a prefix */ 156f2f23566SVivien Didelot pfx[sizeof(pfx) - 1] = '_'; 157f2f23566SVivien Didelot 158cf963573SFlorian Fainelli if (stringset == ETH_SS_PHY_STATS && dev->phydev && 159cf963573SFlorian Fainelli !ops->get_ethtool_phy_stats) { 160cf963573SFlorian Fainelli mcount = phy_ethtool_get_sset_count(dev->phydev); 161cf963573SFlorian Fainelli if (mcount < 0) 162cf963573SFlorian Fainelli mcount = 0; 163cf963573SFlorian Fainelli else 164cf963573SFlorian Fainelli phy_ethtool_get_strings(dev->phydev, data); 165cf963573SFlorian Fainelli } else if (ops->get_sset_count && ops->get_strings) { 16689f09048SFlorian Fainelli mcount = ops->get_sset_count(dev, stringset); 16789f09048SFlorian Fainelli if (mcount < 0) 16889f09048SFlorian Fainelli mcount = 0; 169f2f23566SVivien Didelot ops->get_strings(dev, stringset, data); 170f2f23566SVivien Didelot } 171f2f23566SVivien Didelot 17289f09048SFlorian Fainelli if (ds->ops->get_strings) { 173f2f23566SVivien Didelot ndata = data + mcount * len; 174f2f23566SVivien Didelot /* This function copies ETH_GSTRINGS_LEN bytes, we will mangle 175f2f23566SVivien Didelot * the output after to prepend our CPU port prefix we 176f2f23566SVivien Didelot * constructed earlier 177f2f23566SVivien Didelot */ 17889f09048SFlorian Fainelli ds->ops->get_strings(ds, port, stringset, ndata); 17989f09048SFlorian Fainelli count = ds->ops->get_sset_count(ds, port, stringset); 180a269333fSDan Carpenter if (count < 0) 181a269333fSDan Carpenter return; 182f2f23566SVivien Didelot for (i = 0; i < count; i++) { 183f2f23566SVivien Didelot memmove(ndata + (i * len + sizeof(pfx)), 184f2f23566SVivien Didelot ndata + i * len, len - sizeof(pfx)); 185f2f23566SVivien Didelot memcpy(ndata + i * len, pfx, sizeof(pfx)); 186f2f23566SVivien Didelot } 187f2f23566SVivien Didelot } 188f2f23566SVivien Didelot } 189f2f23566SVivien Didelot 190f685e609SVladimir Oltean static int dsa_master_ioctl(struct net_device *dev, struct ifreq *ifr, int cmd) 191f685e609SVladimir Oltean { 192f685e609SVladimir Oltean struct dsa_port *cpu_dp = dev->dsa_ptr; 193f685e609SVladimir Oltean struct dsa_switch *ds = cpu_dp->ds; 194f685e609SVladimir Oltean struct dsa_switch_tree *dst; 195f685e609SVladimir Oltean int err = -EOPNOTSUPP; 196f685e609SVladimir Oltean struct dsa_port *dp; 197f685e609SVladimir Oltean 198f685e609SVladimir Oltean dst = ds->dst; 199f685e609SVladimir Oltean 200f685e609SVladimir Oltean switch (cmd) { 201f685e609SVladimir Oltean case SIOCGHWTSTAMP: 202f685e609SVladimir Oltean case SIOCSHWTSTAMP: 203f685e609SVladimir Oltean /* Deny PTP operations on master if there is at least one 204f685e609SVladimir Oltean * switch in the tree that is PTP capable. 205f685e609SVladimir Oltean */ 206f685e609SVladimir Oltean list_for_each_entry(dp, &dst->ports, list) 207f685e609SVladimir Oltean if (dp->ds->ops->port_hwtstamp_get || 208f685e609SVladimir Oltean dp->ds->ops->port_hwtstamp_set) 209f685e609SVladimir Oltean return -EBUSY; 210f685e609SVladimir Oltean break; 211f685e609SVladimir Oltean } 212f685e609SVladimir Oltean 213a7605370SArnd Bergmann if (dev->netdev_ops->ndo_eth_ioctl) 214a7605370SArnd Bergmann err = dev->netdev_ops->ndo_eth_ioctl(dev, ifr, cmd); 215f685e609SVladimir Oltean 216f685e609SVladimir Oltean return err; 217f685e609SVladimir Oltean } 218f685e609SVladimir Oltean 2199c0c7014SFlorian Fainelli static const struct dsa_netdevice_ops dsa_netdev_ops = { 220a7605370SArnd Bergmann .ndo_eth_ioctl = dsa_master_ioctl, 2219c0c7014SFlorian Fainelli }; 2229c0c7014SFlorian Fainelli 22317a22fcfSVivien Didelot static int dsa_master_ethtool_setup(struct net_device *dev) 224f2f23566SVivien Didelot { 2252f657a60SVivien Didelot struct dsa_port *cpu_dp = dev->dsa_ptr; 2267ec764eeSVivien Didelot struct dsa_switch *ds = cpu_dp->ds; 227f2f23566SVivien Didelot struct ethtool_ops *ops; 228f2f23566SVivien Didelot 229f2f23566SVivien Didelot ops = devm_kzalloc(ds->dev, sizeof(*ops), GFP_KERNEL); 230f2f23566SVivien Didelot if (!ops) 231f2f23566SVivien Didelot return -ENOMEM; 232f2f23566SVivien Didelot 2337ec764eeSVivien Didelot cpu_dp->orig_ethtool_ops = dev->ethtool_ops; 2347ec764eeSVivien Didelot if (cpu_dp->orig_ethtool_ops) 2357ec764eeSVivien Didelot memcpy(ops, cpu_dp->orig_ethtool_ops, sizeof(*ops)); 236f2f23566SVivien Didelot 23748e23311SVivien Didelot ops->get_regs_len = dsa_master_get_regs_len; 23848e23311SVivien Didelot ops->get_regs = dsa_master_get_regs; 239f2f23566SVivien Didelot ops->get_sset_count = dsa_master_get_sset_count; 240f2f23566SVivien Didelot ops->get_ethtool_stats = dsa_master_get_ethtool_stats; 241f2f23566SVivien Didelot ops->get_strings = dsa_master_get_strings; 242cf963573SFlorian Fainelli ops->get_ethtool_phy_stats = dsa_master_get_ethtool_phy_stats; 243f2f23566SVivien Didelot 244f2f23566SVivien Didelot dev->ethtool_ops = ops; 245f2f23566SVivien Didelot 246f2f23566SVivien Didelot return 0; 247f2f23566SVivien Didelot } 248f2f23566SVivien Didelot 24917a22fcfSVivien Didelot static void dsa_master_ethtool_teardown(struct net_device *dev) 250f2f23566SVivien Didelot { 2512f657a60SVivien Didelot struct dsa_port *cpu_dp = dev->dsa_ptr; 252f2f23566SVivien Didelot 2537ec764eeSVivien Didelot dev->ethtool_ops = cpu_dp->orig_ethtool_ops; 2547ec764eeSVivien Didelot cpu_dp->orig_ethtool_ops = NULL; 255f2f23566SVivien Didelot } 25617a22fcfSVivien Didelot 2579c0c7014SFlorian Fainelli static void dsa_netdev_ops_set(struct net_device *dev, 2589c0c7014SFlorian Fainelli const struct dsa_netdevice_ops *ops) 259da7b9e9bSFlorian Fainelli { 2609c0c7014SFlorian Fainelli dev->dsa_ptr->netdev_ops = ops; 261da7b9e9bSFlorian Fainelli } 262da7b9e9bSFlorian Fainelli 2638940e6b6SVladimir Oltean /* Keep the master always promiscuous if the tagging protocol requires that 2648940e6b6SVladimir Oltean * (garbles MAC DA) or if it doesn't support unicast filtering, case in which 2658940e6b6SVladimir Oltean * it would revert to promiscuous mode as soon as we call dev_uc_add() on it 2668940e6b6SVladimir Oltean * anyway. 2678940e6b6SVladimir Oltean */ 268c3975400SVladimir Oltean static void dsa_master_set_promiscuity(struct net_device *dev, int inc) 269c3975400SVladimir Oltean { 270c3975400SVladimir Oltean const struct dsa_device_ops *ops = dev->dsa_ptr->tag_ops; 271c3975400SVladimir Oltean 2728940e6b6SVladimir Oltean if ((dev->priv_flags & IFF_UNICAST_FLT) && !ops->promisc_on_master) 273c3975400SVladimir Oltean return; 274c3975400SVladimir Oltean 275c146f9bcSVladimir Oltean ASSERT_RTNL(); 276c146f9bcSVladimir Oltean 277c3975400SVladimir Oltean dev_set_promiscuity(dev, inc); 278c3975400SVladimir Oltean } 279c3975400SVladimir Oltean 280a3d7e01dSFlorian Fainelli static ssize_t tagging_show(struct device *d, struct device_attribute *attr, 281a3d7e01dSFlorian Fainelli char *buf) 282a3d7e01dSFlorian Fainelli { 283a3d7e01dSFlorian Fainelli struct net_device *dev = to_net_dev(d); 284a3d7e01dSFlorian Fainelli struct dsa_port *cpu_dp = dev->dsa_ptr; 285a3d7e01dSFlorian Fainelli 286a3d7e01dSFlorian Fainelli return sprintf(buf, "%s\n", 287a3d7e01dSFlorian Fainelli dsa_tag_protocol_to_str(cpu_dp->tag_ops)); 288a3d7e01dSFlorian Fainelli } 28953da0ebaSVladimir Oltean 29053da0ebaSVladimir Oltean static ssize_t tagging_store(struct device *d, struct device_attribute *attr, 29153da0ebaSVladimir Oltean const char *buf, size_t count) 29253da0ebaSVladimir Oltean { 29353da0ebaSVladimir Oltean const struct dsa_device_ops *new_tag_ops, *old_tag_ops; 29453da0ebaSVladimir Oltean struct net_device *dev = to_net_dev(d); 29553da0ebaSVladimir Oltean struct dsa_port *cpu_dp = dev->dsa_ptr; 29653da0ebaSVladimir Oltean int err; 29753da0ebaSVladimir Oltean 29853da0ebaSVladimir Oltean old_tag_ops = cpu_dp->tag_ops; 29953da0ebaSVladimir Oltean new_tag_ops = dsa_find_tagger_by_name(buf); 30053da0ebaSVladimir Oltean /* Bad tagger name, or module is not loaded? */ 30153da0ebaSVladimir Oltean if (IS_ERR(new_tag_ops)) 30253da0ebaSVladimir Oltean return PTR_ERR(new_tag_ops); 30353da0ebaSVladimir Oltean 30453da0ebaSVladimir Oltean if (new_tag_ops == old_tag_ops) 30553da0ebaSVladimir Oltean /* Drop the temporarily held duplicate reference, since 30653da0ebaSVladimir Oltean * the DSA switch tree uses this tagger. 30753da0ebaSVladimir Oltean */ 30853da0ebaSVladimir Oltean goto out; 30953da0ebaSVladimir Oltean 31053da0ebaSVladimir Oltean err = dsa_tree_change_tag_proto(cpu_dp->ds->dst, dev, new_tag_ops, 31153da0ebaSVladimir Oltean old_tag_ops); 31253da0ebaSVladimir Oltean if (err) { 31353da0ebaSVladimir Oltean /* On failure the old tagger is restored, so we don't need the 31453da0ebaSVladimir Oltean * driver for the new one. 31553da0ebaSVladimir Oltean */ 31653da0ebaSVladimir Oltean dsa_tag_driver_put(new_tag_ops); 31753da0ebaSVladimir Oltean return err; 31853da0ebaSVladimir Oltean } 31953da0ebaSVladimir Oltean 32053da0ebaSVladimir Oltean /* On success we no longer need the module for the old tagging protocol 32153da0ebaSVladimir Oltean */ 32253da0ebaSVladimir Oltean out: 32353da0ebaSVladimir Oltean dsa_tag_driver_put(old_tag_ops); 32453da0ebaSVladimir Oltean return count; 32553da0ebaSVladimir Oltean } 32653da0ebaSVladimir Oltean static DEVICE_ATTR_RW(tagging); 327a3d7e01dSFlorian Fainelli 328a3d7e01dSFlorian Fainelli static struct attribute *dsa_slave_attrs[] = { 329a3d7e01dSFlorian Fainelli &dev_attr_tagging.attr, 330a3d7e01dSFlorian Fainelli NULL 331a3d7e01dSFlorian Fainelli }; 332a3d7e01dSFlorian Fainelli 333a3d7e01dSFlorian Fainelli static const struct attribute_group dsa_group = { 334a3d7e01dSFlorian Fainelli .name = "dsa", 335a3d7e01dSFlorian Fainelli .attrs = dsa_slave_attrs, 336a3d7e01dSFlorian Fainelli }; 337a3d7e01dSFlorian Fainelli 338*066dfc42SVladimir Oltean static void dsa_master_reset_mtu(struct net_device *dev) 339*066dfc42SVladimir Oltean { 340*066dfc42SVladimir Oltean int err; 341*066dfc42SVladimir Oltean 342*066dfc42SVladimir Oltean err = dev_set_mtu(dev, ETH_DATA_LEN); 343*066dfc42SVladimir Oltean if (err) 344*066dfc42SVladimir Oltean netdev_dbg(dev, 345*066dfc42SVladimir Oltean "Unable to reset MTU to exclude DSA overheads\n"); 346*066dfc42SVladimir Oltean } 347*066dfc42SVladimir Oltean 34817a22fcfSVivien Didelot int dsa_master_setup(struct net_device *dev, struct dsa_port *cpu_dp) 34917a22fcfSVivien Didelot { 350*066dfc42SVladimir Oltean const struct dsa_device_ops *tag_ops = cpu_dp->tag_ops; 35107b90056SVladimir Oltean struct dsa_switch *ds = cpu_dp->ds; 35207b90056SVladimir Oltean struct device_link *consumer_link; 353*066dfc42SVladimir Oltean int mtu, ret; 354*066dfc42SVladimir Oltean 355*066dfc42SVladimir Oltean mtu = ETH_DATA_LEN + dsa_tag_protocol_overhead(tag_ops); 356a3d7e01dSFlorian Fainelli 35707b90056SVladimir Oltean /* The DSA master must use SET_NETDEV_DEV for this to work. */ 35807b90056SVladimir Oltean consumer_link = device_link_add(ds->dev, dev->dev.parent, 35907b90056SVladimir Oltean DL_FLAG_AUTOREMOVE_CONSUMER); 36007b90056SVladimir Oltean if (!consumer_link) 36107b90056SVladimir Oltean netdev_err(dev, 36207b90056SVladimir Oltean "Failed to create a device link to DSA switch %s\n", 36307b90056SVladimir Oltean dev_name(ds->dev)); 36407b90056SVladimir Oltean 365*066dfc42SVladimir Oltean /* The switch driver may not implement ->port_change_mtu(), case in 366*066dfc42SVladimir Oltean * which dsa_slave_change_mtu() will not update the master MTU either, 367*066dfc42SVladimir Oltean * so we need to do that here. 368*066dfc42SVladimir Oltean */ 369*066dfc42SVladimir Oltean ret = dev_set_mtu(dev, mtu); 370*066dfc42SVladimir Oltean if (ret) 371*066dfc42SVladimir Oltean netdev_warn(dev, "error %d setting MTU to %d to include DSA overhead\n", 372*066dfc42SVladimir Oltean ret, mtu); 373*066dfc42SVladimir Oltean 37417a22fcfSVivien Didelot /* If we use a tagging format that doesn't have an ethertype 37517a22fcfSVivien Didelot * field, make sure that all packets from this point on get 37617a22fcfSVivien Didelot * sent to the tag format's receive function. 37717a22fcfSVivien Didelot */ 37817a22fcfSVivien Didelot wmb(); 37917a22fcfSVivien Didelot 38017a22fcfSVivien Didelot dev->dsa_ptr = cpu_dp; 381c3975400SVladimir Oltean 382c3975400SVladimir Oltean dsa_master_set_promiscuity(dev, 1); 383c3975400SVladimir Oltean 384a3d7e01dSFlorian Fainelli ret = dsa_master_ethtool_setup(dev); 385a3d7e01dSFlorian Fainelli if (ret) 386c3975400SVladimir Oltean goto out_err_reset_promisc; 387a3d7e01dSFlorian Fainelli 3889c0c7014SFlorian Fainelli dsa_netdev_ops_set(dev, &dsa_netdev_ops); 389da7b9e9bSFlorian Fainelli 390a3d7e01dSFlorian Fainelli ret = sysfs_create_group(&dev->dev.kobj, &dsa_group); 391a3d7e01dSFlorian Fainelli if (ret) 392da7b9e9bSFlorian Fainelli goto out_err_ndo_teardown; 393a3d7e01dSFlorian Fainelli 394a3d7e01dSFlorian Fainelli return ret; 395da7b9e9bSFlorian Fainelli 396da7b9e9bSFlorian Fainelli out_err_ndo_teardown: 3979c0c7014SFlorian Fainelli dsa_netdev_ops_set(dev, NULL); 398da7b9e9bSFlorian Fainelli dsa_master_ethtool_teardown(dev); 399c3975400SVladimir Oltean out_err_reset_promisc: 400c3975400SVladimir Oltean dsa_master_set_promiscuity(dev, -1); 401da7b9e9bSFlorian Fainelli return ret; 40217a22fcfSVivien Didelot } 40317a22fcfSVivien Didelot 40417a22fcfSVivien Didelot void dsa_master_teardown(struct net_device *dev) 40517a22fcfSVivien Didelot { 406a3d7e01dSFlorian Fainelli sysfs_remove_group(&dev->dev.kobj, &dsa_group); 4079c0c7014SFlorian Fainelli dsa_netdev_ops_set(dev, NULL); 40817a22fcfSVivien Didelot dsa_master_ethtool_teardown(dev); 409*066dfc42SVladimir Oltean dsa_master_reset_mtu(dev); 410c3975400SVladimir Oltean dsa_master_set_promiscuity(dev, -1); 41117a22fcfSVivien Didelot 41217a22fcfSVivien Didelot dev->dsa_ptr = NULL; 41317a22fcfSVivien Didelot 41417a22fcfSVivien Didelot /* If we used a tagging format that doesn't have an ethertype 41517a22fcfSVivien Didelot * field, make sure that all packets from this point get sent 41617a22fcfSVivien Didelot * without the tag and go through the regular receive path. 41717a22fcfSVivien Didelot */ 41817a22fcfSVivien Didelot wmb(); 41917a22fcfSVivien Didelot } 420