> +static int cmis_fw_update_reset(struct net_device *dev) > +{ > + __u32 reset_data = ETH_RESET_PHY; > + > + return dev->ethtool_ops->reset(dev, &reset_data); Is there a test somewhere that this op is actually implemented? Maybe the next patch. Andrew