On Tue, Jun 09, 2020 at 03:11:52PM +0200, Sascha Hauer wrote: > The Marvell MVNETA Ethernet controller supports a 2.5Gbps SGMII mode > called DRSGMII. Depending on the Port MAC Control Register0 PortType > setting this seems to be either an overclocked SGMII mode or 2500BaseX. > > This patch adds the necessary Serdes Configuration setting for the > 2.5Gbps modes. There is no phy interface mode define for overclocked > SGMII, so only 2500BaseX is handled for now. > > As phy_interface_mode_is_8023z() returns true for both > PHY_INTERFACE_MODE_1000BASEX and PHY_INTERFACE_MODE_2500BASEX we > explicitly test for 1000BaseX instead of using > phy_interface_mode_is_8023z() to differentiate the different > possibilities.
Hi Sascha This seems like it should have a Fixes: tag, and be submitted to the net tree. Please see the Networking FAQ. Otherwise it looks O.K. Andrew