Re: [PATCH net-next 2/6] net: dsa: simplify netdevice events handling

[Vivien Didelot]

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