staging: dpaa2-ethsw: move port switchdev notifier per ethsw
authorIoana Ciornei <ioana.ciornei@nxp.com>
Mon, 11 Nov 2019 16:50:56 +0000 (18:50 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 12 Nov 2019 23:10:15 +0000 (00:10 +0100)
Register a different switchdev 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-3-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 b255201..abbfcbf 100644 (file)
@@ -1314,10 +1314,6 @@ static int port_switchdev_blocking_event(struct notifier_block *unused,
        return NOTIFY_DONE;
 }
 
-static struct notifier_block port_switchdev_nb = {
-       .notifier_call = port_switchdev_event,
-};
-
 static struct notifier_block port_switchdev_blocking_nb = {
        .notifier_call = port_switchdev_blocking_event,
 };
@@ -1334,7 +1330,8 @@ static int ethsw_register_notifier(struct device *dev)
                return err;
        }
 
-       err = register_switchdev_notifier(&port_switchdev_nb);
+       ethsw->port_switchdev_nb.notifier_call = port_switchdev_event;
+       err = register_switchdev_notifier(&ethsw->port_switchdev_nb);
        if (err) {
                dev_err(dev, "Failed to register switchdev notifier\n");
                goto err_switchdev_nb;
@@ -1349,7 +1346,7 @@ static int ethsw_register_notifier(struct device *dev)
        return 0;
 
 err_switchdev_blocking_nb:
-       unregister_switchdev_notifier(&port_switchdev_nb);
+       unregister_switchdev_notifier(&ethsw->port_switchdev_nb);
 err_switchdev_nb:
        unregister_netdevice_notifier(&ethsw->port_nb);
        return err;
@@ -1499,7 +1496,7 @@ static void ethsw_unregister_notifier(struct device *dev)
                dev_err(dev,
                        "Failed to unregister switchdev blocking notifier (%d)\n", err);
 
-       err = unregister_switchdev_notifier(&port_switchdev_nb);
+       err = unregister_switchdev_notifier(&ethsw->port_switchdev_nb);
        if (err)
                dev_err(dev,
                        "Failed to unregister switchdev notifier (%d)\n", err);
index 39a3f54..e050589 100644 (file)
@@ -68,6 +68,7 @@ struct ethsw_core {
        bool                            learning;
 
        struct notifier_block           port_nb;
+       struct notifier_block           port_switchdev_nb;
 };
 
 #endif /* __ETHSW_H */