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