[PATCH v4 2/6] staging: fsl-dpaa2/ethsw: Add Freescale DPAA2 Ethernet Switch driver
Razvan Stefanescu
razvan.stefanescu at nxp.com
Tue Mar 13 05:51:13 UTC 2018
> -----Original Message-----
> From: Andrew Lunn [mailto:andrew at lunn.ch]
> Sent: Monday, March 12, 2018 4:37 PM
> To: Razvan Stefanescu <razvan.stefanescu at nxp.com>
> Cc: gregkh at linuxfoundation.org; devel at driverdev.osuosl.org; linux-
> kernel at vger.kernel.org; netdev at vger.kernel.org; Alexander Graf
> <agraf at suse.de>; arnd at arndb.de; Alexandru Marginean
> <alexandru.marginean at nxp.com>; Ruxandra Ioana Ciocoi Radulescu
> <ruxandra.radulescu at nxp.com>; Ioana Ciornei <ioana.ciornei at nxp.com>;
> Laurentiu Tudor <laurentiu.tudor at nxp.com>; stuyoder at gmail.com
> Subject: Re: [PATCH v4 2/6] staging: fsl-dpaa2/ethsw: Add Freescale DPAA2
> Ethernet Switch driver
>
> > +static int port_netdevice_event(struct notifier_block *unused,
> > + unsigned long event, void *ptr)
> > +{
> > + struct net_device *netdev = netdev_notifier_info_to_dev(ptr);
> > + struct netdev_notifier_changeupper_info *info = ptr;
> > + struct net_device *upper_dev;
> > + int err = 0;
> > +
> > + if (netdev->netdev_ops != ðsw_port_ops)
> > + return NOTIFY_DONE;
> > +
> > + /* Handle just upper dev link/unlink for the moment */
> > + if (event == NETDEV_CHANGEUPPER) {
> > + upper_dev = info->upper_dev;
> > + if (netif_is_bridge_master(upper_dev)) {
> > + if (info->linking)
> > + err = port_bridge_join(netdev);
> > + else
> > + err = port_bridge_leave(netdev);
> > + }
> > + }
> > +
> > + return notifier_from_errno(err);
> > +}
>
> I could be missing something here, but don't you need to pass to
> port_bridge_join() which bridge the port is joining. There can be
> multiple bridges, so you need to ensure the port joins the correct
> bridge.
>
Thank you for noticing this. I'll add proper checks in next version.
Razvan
> Andrew
More information about the devel
mailing list