staging: dpaa2-ethsw: move port notifier per ethsw
authorIoana Ciornei <ioana.ciornei@nxp.com>
Mon, 11 Nov 2019 16:50:55 +0000 (18:50 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 12 Nov 2019 23:10:14 +0000 (00:10 +0100)
Register a different net_device notifier block per ethsw instance.
When probing multiple dpaa2-ethsw instances, without this the register
will fail.

Signed-off-by: Ioana Ciornei <ioana.ciornei@nxp.com>
Link: https://lore.kernel.org/r/1573491058-24766-2-git-send-email-ioana.ciornei@nxp.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/fsl-dpaa2/ethsw/ethsw.c
drivers/staging/fsl-dpaa2/ethsw/ethsw.h

index 14a9eeb..b255201 100644 (file)
@@ -1174,10 +1174,6 @@ static int port_netdevice_event(struct notifier_block *unused,
        return notifier_from_errno(err);
 }
 
-static struct notifier_block port_nb __read_mostly = {
-       .notifier_call = port_netdevice_event,
-};
-
 struct ethsw_switchdev_event_work {
        struct work_struct work;
        struct switchdev_notifier_fdb_info fdb_info;
@@ -1328,9 +1324,11 @@ static struct notifier_block port_switchdev_blocking_nb = {
 
 static int ethsw_register_notifier(struct device *dev)
 {
+       struct ethsw_core *ethsw = dev_get_drvdata(dev);
        int err;
 
-       err = register_netdevice_notifier(&port_nb);
+       ethsw->port_nb.notifier_call = port_netdevice_event;
+       err = register_netdevice_notifier(&ethsw->port_nb);
        if (err) {
                dev_err(dev, "Failed to register netdev notifier\n");
                return err;
@@ -1353,7 +1351,7 @@ static int ethsw_register_notifier(struct device *dev)
 err_switchdev_blocking_nb:
        unregister_switchdev_notifier(&port_switchdev_nb);
 err_switchdev_nb:
-       unregister_netdevice_notifier(&port_nb);
+       unregister_netdevice_notifier(&ethsw->port_nb);
        return err;
 }
 
@@ -1491,6 +1489,7 @@ static int ethsw_port_init(struct ethsw_port_priv *port_priv, u16 port)
 
 static void ethsw_unregister_notifier(struct device *dev)
 {
+       struct ethsw_core *ethsw = dev_get_drvdata(dev);
        struct notifier_block *nb;
        int err;
 
@@ -1505,7 +1504,7 @@ static void ethsw_unregister_notifier(struct device *dev)
                dev_err(dev,
                        "Failed to unregister switchdev notifier (%d)\n", err);
 
-       err = unregister_netdevice_notifier(&port_nb);
+       err = unregister_netdevice_notifier(&ethsw->port_nb);
        if (err)
                dev_err(dev,
                        "Failed to unregister netdev notifier (%d)\n", err);
index 3ea8a0a..39a3f54 100644 (file)
@@ -66,6 +66,8 @@ struct ethsw_core {
 
        u8                              vlans[VLAN_VID_MASK + 1];
        bool                            learning;
+
+       struct notifier_block           port_nb;
 };
 
 #endif /* __ETHSW_H */