Commit 70180f72 authored by Nicolai Buchwitz's avatar Nicolai Buchwitz Committed by Jakub Kicinski
Browse files

net: phy: microchip: enable downshift by default on LAN88xx 



Enable auto-downshift from 1000BASE-T to 100BASE-TX after 2 failed
auto-negotiation attempts by default. This ensures that links with
faulty or missing cable pairs (C and D) fall back to 100Mbps without
requiring userspace configuration.

The downshift count is stored in the driver's private data and applied
in config_init, so user changes via ethtool are preserved across
suspend/resume cycles.

Users can override or disable downshift at runtime:

  ethtool --set-phy-tunable eth0 downshift off

Signed-off-by: default avatarNicolai Buchwitz <nb@tipi-net.de>
Link: https://patch.msgid.link/20260401123848.696766-3-nb@tipi-net.de


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent e417ac73

drivers/net/phy/microchip.c

+15 −2
Original line number Diff line number Diff line
@@ -26,6 +26,7 @@ struct lan88xx_priv { 
	int	chip_id;

	int	chip_rev;

	__u32	wolopts;

	u8	downshift_cnt;

};



static int lan88xx_read_page(struct phy_device *phydev)
@@ -247,9 +248,15 @@ static int lan88xx_get_tunable(struct phy_device *phydev, 
static int lan88xx_set_tunable(struct phy_device *phydev,

			       struct ethtool_tunable *tuna, const void *data)

{

	struct lan88xx_priv *priv = phydev->priv;

	int ret;



	switch (tuna->id) {

	case ETHTOOL_PHY_DOWNSHIFT:

		return lan88xx_set_downshift(phydev, *(const u8 *)data);

		ret = lan88xx_set_downshift(phydev, *(const u8 *)data);

		if (!ret)

			priv->downshift_cnt = *(const u8 *)data;

		return ret;

	default:

		return -EOPNOTSUPP;

	}
@@ -267,6 +274,7 @@ static int lan88xx_probe(struct phy_device *phydev) 
		return -ENOMEM;



	priv->wolopts = 0;

	priv->downshift_cnt = 2;



	len = of_property_read_variable_u32_array(dev->of_node,

						  "microchip,led-modes",
@@ -346,7 +354,8 @@ static void lan88xx_set_mdix(struct phy_device *phydev) 


static int lan88xx_config_init(struct phy_device *phydev)

{

	int val;

	struct lan88xx_priv *priv = phydev->priv;

	int val, err;



	/*Zerodetect delay enable */

	val = phy_read_mmd(phydev, MDIO_MMD_PCS,
@@ -359,6 +368,10 @@ static int lan88xx_config_init(struct phy_device *phydev) 
	/* Config DSP registers */

	lan88xx_config_TR_regs(phydev);



	err = lan88xx_set_downshift(phydev, priv->downshift_cnt);

	if (err < 0)

		return err;



	return 0;

}