Re: [PATCH] r8169: add support for RTL8127ATF

[Andrew Lunn]

> +	if (tp->fiber_mode) {
> +		phy_remove_link_mode(tp->phydev, ETHTOOL_LINK_MODE_10baseT_Half_BIT);
> +		phy_remove_link_mode(tp->phydev, ETHTOOL_LINK_MODE_10baseT_Full_BIT);
> +		phy_remove_link_mode(tp->phydev, ETHTOOL_LINK_MODE_100baseT_Half_BIT);
> +		phy_remove_link_mode(tp->phydev, ETHTOOL_LINK_MODE_100baseT_Full_BIT);
> +		phy_remove_link_mode(tp->phydev, ETHTOOL_LINK_MODE_2500baseT_Full_BIT);
> +		phy_remove_link_mode(tp->phydev, ETHTOOL_LINK_MODE_5000baseT_Full_BIT);

An SFP module can support baseT modes, if the SFP module has a PHY
inside it. But it could also be it is a fibre module with a laser and
so uses 100baseFX, 1000baseX, 2500BaseX, etc.

To do this properly, you need to be able to read the SFP EERPOMs
content, to know what sort of SFP module you have plugged in. Then you
can list the correct modes.

	Andrew