> +int hclge_mac_mdio_config(struct hclge_dev *hdev)> +{..> +}> +> +int hclge_mac_start_phy(struct hclge_dev *hdev)> +{> +}> +> +void hclge_mac_stop_phy(struct hclge_dev *hdev)> +{> +}> -- These are not static functions. So i would expect them to be in aheader file somewhere.... Andrew