123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272 |
- From cce5dd6034ed1651ee25c910edee708e6b84a44a Mon Sep 17 00:00:00 2001
- From: John Crispin <[email protected]>
- Date: Thu, 10 Aug 2017 14:45:08 +0200
- Subject: [PATCH 33/57] net: dsa: add multi gmac support
- Signed-off-by: John Crispin <[email protected]>
- ---
- drivers/net/dsa/mt7530.c | 10 +---------
- include/net/dsa.h | 21 ++++++++++++++++++++-
- net/dsa/dsa2.c | 40 +++++++++++++++++++++++++++++++++-------
- net/dsa/dsa_priv.h | 1 +
- net/dsa/slave.c | 26 ++++++++++++++++----------
- 5 files changed, 71 insertions(+), 27 deletions(-)
- --- a/drivers/net/dsa/mt7530.c
- +++ b/drivers/net/dsa/mt7530.c
- @@ -996,15 +996,7 @@ err:
- static enum dsa_tag_protocol
- mtk_get_tag_protocol(struct dsa_switch *ds)
- {
- - struct mt7530_priv *priv = ds->priv;
- -
- - if (!dsa_is_cpu_port(ds, MT7530_CPU_PORT)) {
- - dev_warn(priv->dev,
- - "port not matched with tagging CPU port\n");
- - return DSA_TAG_PROTO_NONE;
- - } else {
- - return DSA_TAG_PROTO_MTK;
- - }
- + return DSA_TAG_PROTO_MTK;
- }
-
- static struct dsa_switch_ops mt7530_switch_ops = {
- --- a/include/net/dsa.h
- +++ b/include/net/dsa.h
- @@ -145,6 +145,8 @@ struct dsa_port {
- struct device_node *dn;
- unsigned int ageing_time;
- u8 stp_state;
- + struct net_device *ethernet;
- + int upstream;
- };
-
- struct dsa_switch {
- @@ -205,7 +207,7 @@ struct dsa_switch {
-
- static inline bool dsa_is_cpu_port(struct dsa_switch *ds, int p)
- {
- - return !!(ds->index == ds->dst->cpu_switch && p == ds->dst->cpu_port);
- + return !!(ds->cpu_port_mask & (1 << p));
- }
-
- static inline bool dsa_is_dsa_port(struct dsa_switch *ds, int p)
- @@ -218,6 +220,11 @@ static inline bool dsa_is_port_initializ
- return ds->enabled_port_mask & (1 << p) && ds->ports[p].netdev;
- }
-
- +static inline bool dsa_is_upstream_port(struct dsa_switch *ds, int p)
- +{
- + return dsa_is_cpu_port(ds, p) || dsa_is_dsa_port(ds, p);
- +}
- +
- static inline u8 dsa_upstream_port(struct dsa_switch *ds)
- {
- struct dsa_switch_tree *dst = ds->dst;
- @@ -234,6 +241,18 @@ static inline u8 dsa_upstream_port(struc
- return ds->rtable[dst->cpu_switch];
- }
-
- +static inline u8 dsa_port_upstream_port(struct dsa_switch *ds, int port)
- +{
- + /*
- + * If this port has a specific upstream cpu port, use it,
- + * otherwise use the switch default.
- + */
- + if (ds->ports[port].upstream)
- + return ds->ports[port].upstream;
- + else
- + return dsa_upstream_port(ds);
- +}
- +
- struct switchdev_trans;
- struct switchdev_obj;
- struct switchdev_obj_port_fdb;
- --- a/net/dsa/dsa2.c
- +++ b/net/dsa/dsa2.c
- @@ -248,8 +248,6 @@ static int dsa_cpu_port_apply(struct dev
- return err;
- }
-
- - ds->cpu_port_mask |= BIT(index);
- -
- return 0;
- }
-
- @@ -259,6 +257,10 @@ static void dsa_cpu_port_unapply(struct
- dsa_cpu_dsa_destroy(port);
- ds->cpu_port_mask &= ~BIT(index);
-
- + if (ds->ports[index].ethernet) {
- + dev_put(ds->ports[index].ethernet);
- + ds->ports[index].ethernet = NULL;
- + }
- }
-
- static int dsa_user_port_apply(struct device_node *port, u32 index,
- @@ -479,6 +481,29 @@ static int dsa_cpu_parse(struct device_n
-
- dst->rcv = dst->tag_ops->rcv;
-
- + dev_hold(ethernet_dev);
- + ds->ports[index].ethernet = ethernet_dev;
- + ds->cpu_port_mask |= BIT(index);
- +
- + return 0;
- +}
- +
- +static int dsa_user_parse(struct device_node *port, u32 index,
- + struct dsa_switch *ds)
- +{
- + struct device_node *cpu_port;
- + const unsigned int *cpu_port_reg;
- + int cpu_port_index;
- +
- + cpu_port = of_parse_phandle(port, "cpu", 0);
- + if (cpu_port) {
- + cpu_port_reg = of_get_property(cpu_port, "reg", NULL);
- + if (!cpu_port_reg)
- + return -EINVAL;
- + cpu_port_index = be32_to_cpup(cpu_port_reg);
- + ds->ports[index].upstream = cpu_port_index;
- + }
- +
- return 0;
- }
-
- @@ -486,18 +511,19 @@ static int dsa_ds_parse(struct dsa_switc
- {
- struct device_node *port;
- u32 index;
- - int err;
- + int err = 0;
-
- for (index = 0; index < DSA_MAX_PORTS; index++) {
- port = ds->ports[index].dn;
- if (!port)
- continue;
-
- - if (dsa_port_is_cpu(port)) {
- + if (dsa_port_is_cpu(port))
- err = dsa_cpu_parse(port, index, dst, ds);
- - if (err)
- - return err;
- - }
- + else if (!dsa_port_is_dsa(port))
- + err = dsa_user_parse(port, index, ds);
- + if (err)
- + return err;
- }
-
- pr_info("DSA: switch %d %d parsed\n", dst->tree, ds->index);
- --- a/net/dsa/dsa_priv.h
- +++ b/net/dsa/dsa_priv.h
- @@ -43,6 +43,7 @@ struct dsa_slave_priv {
- int old_duplex;
-
- struct net_device *bridge_dev;
- + struct net_device *master;
- #ifdef CONFIG_NET_POLL_CONTROLLER
- struct netpoll *netpoll;
- #endif
- --- a/net/dsa/slave.c
- +++ b/net/dsa/slave.c
- @@ -61,7 +61,7 @@ static int dsa_slave_get_iflink(const st
- {
- struct dsa_slave_priv *p = netdev_priv(dev);
-
- - return p->parent->dst->master_netdev->ifindex;
- + return p->master->ifindex;
- }
-
- static inline bool dsa_port_is_bridged(struct dsa_slave_priv *p)
- @@ -96,7 +96,7 @@ static void dsa_port_set_stp_state(struc
- static int dsa_slave_open(struct net_device *dev)
- {
- struct dsa_slave_priv *p = netdev_priv(dev);
- - struct net_device *master = p->parent->dst->master_netdev;
- + struct net_device *master = p->master;
- struct dsa_switch *ds = p->parent;
- u8 stp_state = dsa_port_is_bridged(p) ?
- BR_STATE_BLOCKING : BR_STATE_FORWARDING;
- @@ -151,7 +151,7 @@ out:
- static int dsa_slave_close(struct net_device *dev)
- {
- struct dsa_slave_priv *p = netdev_priv(dev);
- - struct net_device *master = p->parent->dst->master_netdev;
- + struct net_device *master = p->master;
- struct dsa_switch *ds = p->parent;
-
- if (p->phy)
- @@ -178,7 +178,7 @@ static int dsa_slave_close(struct net_de
- static void dsa_slave_change_rx_flags(struct net_device *dev, int change)
- {
- struct dsa_slave_priv *p = netdev_priv(dev);
- - struct net_device *master = p->parent->dst->master_netdev;
- + struct net_device *master = p->master;
-
- if (change & IFF_ALLMULTI)
- dev_set_allmulti(master, dev->flags & IFF_ALLMULTI ? 1 : -1);
- @@ -189,7 +189,7 @@ static void dsa_slave_change_rx_flags(st
- static void dsa_slave_set_rx_mode(struct net_device *dev)
- {
- struct dsa_slave_priv *p = netdev_priv(dev);
- - struct net_device *master = p->parent->dst->master_netdev;
- + struct net_device *master = p->master;
-
- dev_mc_sync(master, dev);
- dev_uc_sync(master, dev);
- @@ -198,7 +198,7 @@ static void dsa_slave_set_rx_mode(struct
- static int dsa_slave_set_mac_address(struct net_device *dev, void *a)
- {
- struct dsa_slave_priv *p = netdev_priv(dev);
- - struct net_device *master = p->parent->dst->master_netdev;
- + struct net_device *master = p->master;
- struct sockaddr *addr = a;
- int err;
-
- @@ -633,7 +633,7 @@ static netdev_tx_t dsa_slave_xmit(struct
- /* Queue the SKB for transmission on the parent interface, but
- * do not modify its EtherType
- */
- - nskb->dev = p->parent->dst->master_netdev;
- + nskb->dev = p->master;
- dev_queue_xmit(nskb);
-
- return NETDEV_TX_OK;
- @@ -945,7 +945,7 @@ static int dsa_slave_netpoll_setup(struc
- {
- struct dsa_slave_priv *p = netdev_priv(dev);
- struct dsa_switch *ds = p->parent;
- - struct net_device *master = ds->dst->master_netdev;
- + struct net_device *master = p->master;
- struct netpoll *netpoll;
- int err = 0;
-
- @@ -1233,11 +1233,16 @@ int dsa_slave_create(struct dsa_switch *
- struct net_device *master;
- struct net_device *slave_dev;
- struct dsa_slave_priv *p;
- + int port_cpu = ds->ports[port].upstream;
- int ret;
-
- - master = ds->dst->master_netdev;
- - if (ds->master_netdev)
- + if (port_cpu && ds->ports[port_cpu].ethernet)
- + master = ds->ports[port_cpu].ethernet;
- + else if (ds->master_netdev)
- master = ds->master_netdev;
- + else
- + master = ds->dst->master_netdev;
- + master->dsa_ptr = (void *)ds->dst;
-
- slave_dev = alloc_netdev(sizeof(struct dsa_slave_priv), name,
- NET_NAME_UNKNOWN, ether_setup);
- @@ -1263,6 +1268,7 @@ int dsa_slave_create(struct dsa_switch *
- p->parent = ds;
- p->port = port;
- p->xmit = dst->tag_ops->xmit;
- + p->master = master;
-
- p->old_pause = -1;
- p->old_link = -1;
|