net: dsa: change scope of notifier call chain
authorVivien Didelot <vivien.didelot@savoirfairelinux.com>
Fri, 19 May 2017 21:00:37 +0000 (17:00 -0400)
committerDavid S. Miller <davem@davemloft.net>
Mon, 22 May 2017 23:37:32 +0000 (19:37 -0400)
Change the scope of the fabric notification helper from the DSA slave to
the DSA port, since this is a DSA layer specific notion, that can be
used by non-slave ports (CPU and DSA).

Signed-off-by: Vivien Didelot <vivien.didelot@savoirfairelinux.com>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/dsa/slave.c

index 403d1dfe7f5011268ad1915b37c8c47b4e4c5ae4..371f6d267917d45a855cb0cf3a8789af2d0283b4 100644 (file)
 
 static bool dsa_slave_dev_check(struct net_device *dev);
 
-static int dsa_slave_notify(struct net_device *dev, unsigned long e, void *v)
+static int dsa_port_notify(struct dsa_port *dp, unsigned long e, void *v)
 {
-       struct dsa_slave_priv *p = netdev_priv(dev);
-       struct raw_notifier_head *nh = &p->dp->ds->dst->nh;
+       struct raw_notifier_head *nh = &dp->ds->dst->nh;
        int err;
 
        err = raw_notifier_call_chain(nh, e, v);
@@ -589,7 +588,7 @@ static int dsa_slave_bridge_port_join(struct net_device *dev,
         */
        p->dp->bridge_dev = br;
 
-       err = dsa_slave_notify(dev, DSA_NOTIFIER_BRIDGE_JOIN, &info);
+       err = dsa_port_notify(p->dp, DSA_NOTIFIER_BRIDGE_JOIN, &info);
 
        /* The bridging is rolled back on error */
        if (err)
@@ -614,7 +613,7 @@ static void dsa_slave_bridge_port_leave(struct net_device *dev,
         */
        p->dp->bridge_dev = NULL;
 
-       err = dsa_slave_notify(dev, DSA_NOTIFIER_BRIDGE_LEAVE, &info);
+       err = dsa_port_notify(p->dp, DSA_NOTIFIER_BRIDGE_LEAVE, &info);
        if (err)
                netdev_err(dev, "failed to notify DSA_NOTIFIER_BRIDGE_LEAVE\n");