Re: [PATCH net-next v2 4/7] net: mscc: Add initial Ocelot switch support

From: Andrew Lunn
Date: Thu Apr 26 2018 - 17:09:27 EST


> +/* Checks if the net_device instance given to us originate from our driver. */
> +static bool ocelot_netdevice_dev_check(const struct net_device *dev)
> +{
> + return dev->netdev_ops == &ocelot_port_netdev_ops;
> +}

This is probably O.K. now, but when you add support for controlling
the switch over PCIe, i think it breaks. A board could have two
switches...

It might be possible to do something with dev->parent. All ports of a
switch should have the same parent.

Andrew