Hi Florian, Florian Fainelli <f.faine...@gmail.com> writes:
>> - if (dsa_slave_dev_check(dev)) >> - return dsa_slave_port_event(dev, event, ptr); >> + if (dev->netdev_ops != &dsa_slave_netdev_ops) >> + return NOTIFY_DONE; > > Why not keep the dsa_slave_dev_check() here? I dropped it because that condition feels more readable to me than !dsa_slave_dev_check(dev). Thanks, Vivien