net: dsa: move bridging routines
authorVivien Didelot <vivien.didelot@savoirfairelinux.com>
Fri, 19 May 2017 21:00:45 +0000 (17:00 -0400)
committerDavid S. Miller <davem@davemloft.net>
Mon, 22 May 2017 23:37:32 +0000 (19:37 -0400)
Move the DSA port code which bridges a port in port.c, where it belongs.

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/dsa_priv.h
net/dsa/port.c
net/dsa/slave.c

index cda218cd9b0550362a41ed5e61761a0b1b35033f..f0b6cd3c8a655813c5df04d0ca5a50ccdcbec7b0 100644 (file)
@@ -64,6 +64,8 @@ void dsa_legacy_unregister(void);
 int dsa_port_set_state(struct dsa_port *dp, u8 state,
                       struct switchdev_trans *trans);
 void dsa_port_set_state_now(struct dsa_port *dp, u8 state);
+int dsa_port_bridge_join(struct dsa_port *dp, struct net_device *br);
+void dsa_port_bridge_leave(struct dsa_port *dp, struct net_device *br);
 
 /* slave.c */
 extern const struct dsa_device_ops notag_netdev_ops;
index 6cc4704190fdeafc930ae23f9b0031a9d6cf722b..da8577fb3d0732a4ba81353f6eee9554431655cc 100644 (file)
  */
 
 #include <linux/if_bridge.h>
+#include <linux/notifier.h>
 
 #include "dsa_priv.h"
 
+static int dsa_port_notify(struct dsa_port *dp, unsigned long e, void *v)
+{
+       struct raw_notifier_head *nh = &dp->ds->dst->nh;
+       int err;
+
+       err = raw_notifier_call_chain(nh, e, v);
+
+       return notifier_to_errno(err);
+}
+
 int dsa_port_set_state(struct dsa_port *dp, u8 state,
                       struct switchdev_trans *trans)
 {
@@ -53,3 +64,50 @@ void dsa_port_set_state_now(struct dsa_port *dp, u8 state)
        if (err)
                pr_err("DSA: failed to set STP state %u (%d)\n", state, err);
 }
+
+int dsa_port_bridge_join(struct dsa_port *dp, struct net_device *br)
+{
+       struct dsa_notifier_bridge_info info = {
+               .sw_index = dp->ds->index,
+               .port = dp->index,
+               .br = br,
+       };
+       int err;
+
+       /* Here the port is already bridged. Reflect the current configuration
+        * so that drivers can program their chips accordingly.
+        */
+       dp->bridge_dev = br;
+
+       err = dsa_port_notify(dp, DSA_NOTIFIER_BRIDGE_JOIN, &info);
+
+       /* The bridging is rolled back on error */
+       if (err)
+               dp->bridge_dev = NULL;
+
+       return err;
+}
+
+void dsa_port_bridge_leave(struct dsa_port *dp, struct net_device *br)
+{
+       struct dsa_notifier_bridge_info info = {
+               .sw_index = dp->ds->index,
+               .port = dp->index,
+               .br = br,
+       };
+       int err;
+
+       /* Here the port is already unbridged. Reflect the current configuration
+        * so that drivers can program their chips accordingly.
+        */
+       dp->bridge_dev = NULL;
+
+       err = dsa_port_notify(dp, DSA_NOTIFIER_BRIDGE_LEAVE, &info);
+       if (err)
+               pr_err("DSA: failed to notify DSA_NOTIFIER_BRIDGE_LEAVE\n");
+
+       /* Port left the bridge, put in BR_STATE_DISABLED by the bridge layer,
+        * so allow it to be in BR_STATE_FORWARDING to be kept functional
+        */
+       dsa_port_set_state_now(dp, BR_STATE_FORWARDING);
+}
index 2c57c7205aa3dc6f4f52f5a24a66f981f1815ba7..ab298c41b8e7cc699b2f0189d5fd8a99f1294393 100644 (file)
 
 static bool dsa_slave_dev_check(struct net_device *dev);
 
-static int dsa_port_notify(struct dsa_port *dp, unsigned long e, void *v)
-{
-       struct raw_notifier_head *nh = &dp->ds->dst->nh;
-       int err;
-
-       err = raw_notifier_call_chain(nh, e, v);
-
-       return notifier_to_errno(err);
-}
-
 /* slave mii_bus handling ***************************************************/
 static int dsa_slave_phy_read(struct mii_bus *bus, int addr, int reg)
 {
@@ -514,53 +504,6 @@ static int dsa_slave_port_obj_dump(struct net_device *dev,
        return err;
 }
 
-static int dsa_port_bridge_join(struct dsa_port *dp, struct net_device *br)
-{
-       struct dsa_notifier_bridge_info info = {
-               .sw_index = dp->ds->index,
-               .port = dp->index,
-               .br = br,
-       };
-       int err;
-
-       /* Here the port is already bridged. Reflect the current configuration
-        * so that drivers can program their chips accordingly.
-        */
-       dp->bridge_dev = br;
-
-       err = dsa_port_notify(dp, DSA_NOTIFIER_BRIDGE_JOIN, &info);
-
-       /* The bridging is rolled back on error */
-       if (err)
-               dp->bridge_dev = NULL;
-
-       return err;
-}
-
-static void dsa_port_bridge_leave(struct dsa_port *dp, struct net_device *br)
-{
-       struct dsa_notifier_bridge_info info = {
-               .sw_index = dp->ds->index,
-               .port = dp->index,
-               .br = br,
-       };
-       int err;
-
-       /* Here the port is already unbridged. Reflect the current configuration
-        * so that drivers can program their chips accordingly.
-        */
-       dp->bridge_dev = NULL;
-
-       err = dsa_port_notify(dp, DSA_NOTIFIER_BRIDGE_LEAVE, &info);
-       if (err)
-               pr_err("DSA: failed to notify DSA_NOTIFIER_BRIDGE_LEAVE\n");
-
-       /* Port left the bridge, put in BR_STATE_DISABLED by the bridge layer,
-        * so allow it to be in BR_STATE_FORWARDING to be kept functional
-        */
-       dsa_port_set_state_now(dp, BR_STATE_FORWARDING);
-}
-
 static int dsa_slave_port_attr_get(struct net_device *dev,
                                   struct switchdev_attr *attr)
 {