>> +    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

Reply via email to