>> + if (!netif_running(netdev)) {
>> + ldr->reason = ETHTOOL_LINK_NETDEV_CARRIER_DOWN;
>> + return 0;
>> + }
> This is generic, will work for any interface. The same is true for
> ADMIN_DOWN. Either it is not required at all, since the information is
> available via other means, or it should be in the core.I agree, will move it to the core. > > Andrew
