On 07/18/2017 08:16 PM, Vivien Didelot wrote:
> Hi Arkadi,
>
> Arkadi Sharshevsky <arka...@mellanox.com> writes:
>
>> --- a/include/net/dsa.h
>> +++ b/include/net/dsa.h
>> @@ -451,6 +451,7 @@ void unregister_switch_driver(struct dsa_switch_driver
>> *type);
>> struct mii_bus *dsa_host_dev_to_mii_bus(struct device *dev);
>>
>> struct net_device *dsa_dev_to_net_device(struct device *dev);
>> +bool dsa_schedule_work(struct work_struct *work);
>
> You forgot to move this declaration to net/dsa/dsa_priv.h, since this is
> private to DSA core and does not need to be exposed to drivers ;-)
>
No problem, will move it to dsa_priv.h, thanks.
>> + err = unregister_netdevice_notifier(&dsa_slave_switchdev_notifier);
>> + if (err)
>> + pr_err("DSA: failed to unregister switchdev notifier (%d)\n",
>> err);
>
> I think you meant unregister_switchdev_notifier() here.
>
> Thanks,
>
> Vivien
>
