> +static int __init phylink_init(void) > +{ > + __set_bit(PHY_INTERFACE_MODE_USXGMII, phylink_sfp_interfaces); > + __set_bit(PHY_INTERFACE_MODE_10GBASER, phylink_sfp_interfaces); > + __set_bit(PHY_INTERFACE_MODE_10GKR, phylink_sfp_interfaces); > + __set_bit(PHY_INTERFACE_MODE_5GBASER, phylink_sfp_interfaces); > + __set_bit(PHY_INTERFACE_MODE_2500BASEX, phylink_sfp_interfaces); > + __set_bit(PHY_INTERFACE_MODE_SGMII, phylink_sfp_interfaces); > + __set_bit(PHY_INTERFACE_MODE_1000BASEX, phylink_sfp_interfaces); Do we need to include PHY_INTERFACE_MODE_100BASEX here for 100BaseFX? Not sure we actually have any systems using it. Andrew