From nobody Thu May 2 20:58:36 2024 Delivered-To: importer2@patchew.org Received-SPF: pass (zohomail.com: domain of vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; envelope-from=linux-kernel-owner@vger.kernel.org; helo=vger.kernel.org; Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org ARC-Seal: i=1; a=rsa-sha256; t=1618230074; cv=none; d=zohomail.com; s=zohoarc; b=hgnwbx6R1gxCJQPb/BZ5xZkMWmDJj/q6ZI5X2a6l5rB6PEXbwBEh+re3B2NqUklQmTYVAkvKxhviKztSpKpl4EMAKLbqaeYQuou5FFwGf+uQG/LUvHIV7zjpyRCwr0r4/XWsqEGpWynFCNftbTHAfg7ScxuL0r/LDBWX+SuTY9I= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1618230074; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Id:MIME-Version:Message-ID:References:Subject:To; bh=yqtWDF/P2rEqZ8lDgxw9NumV6DV5wLbLDgBLsyv19iQ=; b=cc6UDX0AMfO1eFHbmoRcGwaocyW2q1t9yIh9s4PmXoSMsy1q12MlYwyiAGQuZ58LabMA/e6csS1p1FzWzpbmHuB3ksUlDpXM4BT+L16AWiqnHmSJxSqF4ZfPFykGvByllXQtR7l6i+Q+Ndf4m9JLp78krOjV8CSl32U+VAsb1Po= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mx.zohomail.com with SMTP id 1618230074927890.1700833234036; Mon, 12 Apr 2021 05:21:14 -0700 (PDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S240702AbhDLMV3 (ORCPT ); Mon, 12 Apr 2021 08:21:29 -0400 Received: from mail.pr-group.ru ([178.18.215.3]:50835 "EHLO mail.pr-group.ru" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S238059AbhDLMV0 (ORCPT ); Mon, 12 Apr 2021 08:21:26 -0400 Received: from localhost.localdomain ([178.70.223.189]) (authenticated user i.bornyakov@metrotek.ru) by mail.pr-group.ru with ESMTPSA (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256 bits)); Mon, 12 Apr 2021 15:20:56 +0300 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=metrotek.ru; s=mail; h=from:subject:date:message-id:to:cc:mime-version:content-transfer-encoding: in-reply-to:references; bh=oh4kxZwGwy53nO5b07oWCLMSgMZkgNXyPwI/EEEgYp0=; b=BVAfYqIVJvmgjXBbf+qX8BJu5es9xlstzTa5zY7BQIlxWrS655ykUZvvS2z2XUKbc1Ki/6X1FU4Ei C89ZyYKYd28R5ybUAKW4L2fqX3fckwIqVSoF+psbf4PINDjtV9zm6k7UmcDzZXyL6iZln3iqUyKrpu 2Cf1YliPKss7dIIPwfaHozvRkF53b3vnQTfpG7mT7sUtg1rbjrDXExAJ6h64drmhNpnAgNaN6bjHAl ZXvliT4PjWFTcX2PoXfkv1rbKLNUWauMZ+UkWYLFwAZIZjyMoyBdqjdrWmKx4macZbPwjpiP0me2l2 Eu3Cw6v3dZNuRA+QzzWvTmWM8aVeZ+w== X-Spam-Status: No, hits=0.0 required=3.4 tests=AWL: 0.000, BAYES_00: -1.665, CUSTOM_RULE_FROM: ALLOW, TOTAL_SCORE: -1.665,autolearn=ham X-Spam-Level: X-Footer: bWV0cm90ZWsucnU= From: Ivan Bornyakov Cc: Ivan Bornyakov , system@metrotek.ru, andrew@lunn.ch, hkallweit1@gmail.com, linux@armlinux.org.uk, davem@davemloft.net, kuba@kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH net-next 1/2] net: phy: marvell-88x2222: check that link is operational Date: Mon, 12 Apr 2021 15:16:59 +0300 Message-Id: <614b534f1661ecf1fff419e2f36eddfb0e6f066d.1618227910.git.i.bornyakov@metrotek.ru> X-Mailer: git-send-email 2.26.3 In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable To: unlisted-recipients:; (no To-header on input) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-ZohoMail-DKIM: fail (Header signature does not verify) Content-Type: text/plain; charset="utf-8" Some SFP modules uses RX_LOS for link indication. In such cases link will be always up, even without cable connected. RX_LOS changes will trigger link_up()/link_down() upstream operations. Thus, check that SFP link is operational before actual read link status. Signed-off-by: Ivan Bornyakov --- drivers/net/phy/marvell-88x2222.c | 26 ++++++++++++++++++++++++++ 1 file changed, 26 insertions(+) diff --git a/drivers/net/phy/marvell-88x2222.c b/drivers/net/phy/marvell-88= x2222.c index eca8c2f20684..fb285ac741b2 100644 --- a/drivers/net/phy/marvell-88x2222.c +++ b/drivers/net/phy/marvell-88x2222.c @@ -51,6 +51,7 @@ struct mv2222_data { phy_interface_t line_interface; __ETHTOOL_DECLARE_LINK_MODE_MASK(supported); + bool sfp_link; }; =20 /* SFI PMA transmit enable */ @@ -148,6 +149,9 @@ static int mv2222_read_status(struct phy_device *phydev) phydev->speed =3D SPEED_UNKNOWN; phydev->duplex =3D DUPLEX_UNKNOWN; =20 + if (!priv->sfp_link) + return 0; + if (priv->line_interface =3D=3D PHY_INTERFACE_MODE_10GBASER) link =3D mv2222_read_status_10g(phydev); else @@ -446,9 +450,31 @@ static void mv2222_sfp_remove(void *upstream) linkmode_zero(priv->supported); } =20 +static void mv2222_sfp_link_up(void *upstream) +{ + struct phy_device *phydev =3D upstream; + struct mv2222_data *priv; + + priv =3D (struct mv2222_data *)phydev->priv; + + priv->sfp_link =3D true; +} + +static void mv2222_sfp_link_down(void *upstream) +{ + struct phy_device *phydev =3D upstream; + struct mv2222_data *priv; + + priv =3D (struct mv2222_data *)phydev->priv; + + priv->sfp_link =3D false; +} + static const struct sfp_upstream_ops sfp_phy_ops =3D { .module_insert =3D mv2222_sfp_insert, .module_remove =3D mv2222_sfp_remove, + .link_up =3D mv2222_sfp_link_up, + .link_down =3D mv2222_sfp_link_down, .attach =3D phy_sfp_attach, .detach =3D phy_sfp_detach, }; --=20 2.26.3 From nobody Thu May 2 20:58:36 2024 Delivered-To: importer2@patchew.org Received-SPF: pass (zohomail.com: domain of vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; envelope-from=linux-kernel-owner@vger.kernel.org; helo=vger.kernel.org; Authentication-Results: mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org ARC-Seal: i=1; a=rsa-sha256; t=1618230075; cv=none; d=zohomail.com; s=zohoarc; b=D2awrrahh+xZ7WLEpzhWsfWH/yrm3O1uMrENPJ7qJS6RnLH/Ig6GipblOaLbgALcDcZ2J0NjtRk47BwLSfImRykIn6k9idNkAKnkPrA9aaJCoXs6XgQCma5l18fIyMDCcWY9mxLdasUdhdau+mzOTk55veJXVkSYr/ss0Yi7hu0= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1618230075; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Id:MIME-Version:Message-ID:References:Subject:To; bh=TqvhJrPk1fhO1mH1V6/SI7sRLyZ2ruRlKSi4Fbd0hrs=; b=S0cP5t+uqP8q7JuJGY4WtFzq41aLPcgIfuImEShJzS3KUVjhcnneGQuVPwa+XoXW9zAONwKg6EJ0x08gBGkNFj7aevp7TevoB+or4eWUUnu2zIZl6TW8v0i5Tm2yZSHaUUYQ0mE1l3N0QMNfdEGkTMyFLJoToCkgYtE2eP6ogzA= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=fail; spf=pass (zohomail.com: domain of vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mx.zohomail.com with SMTP id 1618230075249516.400957765601; Mon, 12 Apr 2021 05:21:15 -0700 (PDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S240862AbhDLMVc (ORCPT ); Mon, 12 Apr 2021 08:21:32 -0400 Received: from mail.pr-group.ru ([178.18.215.3]:50837 "EHLO mail.pr-group.ru" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S240434AbhDLMV3 (ORCPT ); Mon, 12 Apr 2021 08:21:29 -0400 Received: from localhost.localdomain ([178.70.223.189]) (authenticated user i.bornyakov@metrotek.ru) by mail.pr-group.ru with ESMTPSA (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256 bits)); Mon, 12 Apr 2021 15:20:58 +0300 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=metrotek.ru; s=mail; h=from:subject:date:message-id:to:cc:mime-version:content-transfer-encoding: in-reply-to:references; bh=XmQZwzG+bGgfx9OYVnQAZ/LvVz36gZRVbLw6sZ6ix1E=; b=D58qmRFbACV/V7/Xa+Aj070y/IOT1JurPYhf/+3Gwnzv+fKU0a7KHYjG+BEaShcXjkleP0ab9fO/p 5Q3N/Y8dpnA6vALcOCiGWcV24lvgoJwm/EU9U2fj8MpGlhkJUz+1yUE+jPdsajK9RKMuq+Z62liVEd +TvyKxGMAUWbGxivwgwmmdPzUwsxuqNQAmVAkAmV4/mLpgIxVcMQOTu3PoPNGPkWjSSBbJGmqe8SRU 8VLrnV5xRXjTdWWijXqqObxZYdcYQByz5kxHHZEIZYdjBqrfwi6AZ4M1SreBKPFahZwnnYZM7QypiX x5czSkAR4ToKOaO9qEdwfqrxa8VvATw== X-Spam-Status: No, hits=0.0 required=3.4 tests=AWL: 0.000, BAYES_00: -1.665, CUSTOM_RULE_FROM: ALLOW, TOTAL_SCORE: -1.665,autolearn=ham X-Spam-Level: X-Footer: bWV0cm90ZWsucnU= From: Ivan Bornyakov Cc: Ivan Bornyakov , system@metrotek.ru, andrew@lunn.ch, hkallweit1@gmail.com, linux@armlinux.org.uk, davem@davemloft.net, kuba@kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH net-next 2/2] net: phy: marvell-88x2222: swap 1G/10G modes on autoneg Date: Mon, 12 Apr 2021 15:17:00 +0300 Message-Id: X-Mailer: git-send-email 2.26.3 In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable To: unlisted-recipients:; (no To-header on input) Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-ZohoMail-DKIM: fail (Header signature does not verify) Content-Type: text/plain; charset="utf-8" Setting 10G without autonegotiation is invalid according to phy_ethtool_ksettings_set(). Thus, if autonegotiation can't complete for quite a time, but there is signal in line, switch line interface type to 10GBase-R, if supported, in hope for link to be established. And vice versa. Signed-off-by: Ivan Bornyakov --- drivers/net/phy/marvell-88x2222.c | 276 ++++++++++++++++++------------ 1 file changed, 168 insertions(+), 108 deletions(-) diff --git a/drivers/net/phy/marvell-88x2222.c b/drivers/net/phy/marvell-88= x2222.c index fb285ac741b2..d16c81f08334 100644 --- a/drivers/net/phy/marvell-88x2222.c +++ b/drivers/net/phy/marvell-88x2222.c @@ -48,6 +48,8 @@ #define MV_1GBX_PHY_STAT_SPEED100 BIT(14) #define MV_1GBX_PHY_STAT_SPEED1000 BIT(15) =20 +#define AUTONEG_TIMEOUT 3 + struct mv2222_data { phy_interface_t line_interface; __ETHTOOL_DECLARE_LINK_MODE_MASK(supported); @@ -82,89 +84,6 @@ static int mv2222_soft_reset(struct phy_device *phydev) 5000, 1000000, true); } =20 -/* Returns negative on error, 0 if link is down, 1 if link is up */ -static int mv2222_read_status_10g(struct phy_device *phydev) -{ - int val, link =3D 0; - - val =3D phy_read_mmd(phydev, MDIO_MMD_PCS, MDIO_STAT1); - if (val < 0) - return val; - - if (val & MDIO_STAT1_LSTATUS) { - link =3D 1; - - /* 10GBASE-R do not support auto-negotiation */ - phydev->autoneg =3D AUTONEG_DISABLE; - phydev->speed =3D SPEED_10000; - phydev->duplex =3D DUPLEX_FULL; - } - - return link; -} - -/* Returns negative on error, 0 if link is down, 1 if link is up */ -static int mv2222_read_status_1g(struct phy_device *phydev) -{ - int val, link =3D 0; - - val =3D phy_read_mmd(phydev, MDIO_MMD_PCS, MV_1GBX_STAT); - if (val < 0) - return val; - - if (!(val & BMSR_LSTATUS) || - (phydev->autoneg =3D=3D AUTONEG_ENABLE && - !(val & BMSR_ANEGCOMPLETE))) - return 0; - - link =3D 1; - - val =3D phy_read_mmd(phydev, MDIO_MMD_PCS, MV_1GBX_PHY_STAT); - if (val < 0) - return val; - - if (val & MV_1GBX_PHY_STAT_AN_RESOLVED) { - if (val & MV_1GBX_PHY_STAT_DUPLEX) - phydev->duplex =3D DUPLEX_FULL; - else - phydev->duplex =3D DUPLEX_HALF; - - if (val & MV_1GBX_PHY_STAT_SPEED1000) - phydev->speed =3D SPEED_1000; - else if (val & MV_1GBX_PHY_STAT_SPEED100) - phydev->speed =3D SPEED_100; - else - phydev->speed =3D SPEED_10; - } - - return link; -} - -static int mv2222_read_status(struct phy_device *phydev) -{ - struct mv2222_data *priv =3D phydev->priv; - int link; - - phydev->link =3D 0; - phydev->speed =3D SPEED_UNKNOWN; - phydev->duplex =3D DUPLEX_UNKNOWN; - - if (!priv->sfp_link) - return 0; - - if (priv->line_interface =3D=3D PHY_INTERFACE_MODE_10GBASER) - link =3D mv2222_read_status_10g(phydev); - else - link =3D mv2222_read_status_1g(phydev); - - if (link < 0) - return link; - - phydev->link =3D link; - - return 0; -} - static int mv2222_disable_aneg(struct phy_device *phydev) { int ret =3D phy_clear_bits_mmd(phydev, MDIO_MMD_PCS, MV_1GBX_CTRL, @@ -252,6 +171,24 @@ static bool mv2222_is_1gbx_capable(struct phy_device *= phydev) priv->supported); } =20 +static bool mv2222_is_sgmii_capable(struct phy_device *phydev) +{ + struct mv2222_data *priv =3D phydev->priv; + + return (linkmode_test_bit(ETHTOOL_LINK_MODE_1000baseT_Full_BIT, + priv->supported) || + linkmode_test_bit(ETHTOOL_LINK_MODE_1000baseT_Half_BIT, + priv->supported) || + linkmode_test_bit(ETHTOOL_LINK_MODE_100baseT_Full_BIT, + priv->supported) || + linkmode_test_bit(ETHTOOL_LINK_MODE_100baseT_Half_BIT, + priv->supported) || + linkmode_test_bit(ETHTOOL_LINK_MODE_10baseT_Full_BIT, + priv->supported) || + linkmode_test_bit(ETHTOOL_LINK_MODE_10baseT_Half_BIT, + priv->supported)); +} + static int mv2222_config_line(struct phy_device *phydev) { struct mv2222_data *priv =3D phydev->priv; @@ -271,7 +208,7 @@ static int mv2222_config_line(struct phy_device *phydev) } } =20 -static int mv2222_setup_forced(struct phy_device *phydev) +static int mv2222_swap_line_type(struct phy_device *phydev) { struct mv2222_data *priv =3D phydev->priv; bool changed =3D false; @@ -279,25 +216,23 @@ static int mv2222_setup_forced(struct phy_device *phy= dev) =20 switch (priv->line_interface) { case PHY_INTERFACE_MODE_10GBASER: - if (phydev->speed =3D=3D SPEED_1000 && - mv2222_is_1gbx_capable(phydev)) { + if (mv2222_is_1gbx_capable(phydev)) { priv->line_interface =3D PHY_INTERFACE_MODE_1000BASEX; changed =3D true; } =20 - break; - case PHY_INTERFACE_MODE_1000BASEX: - if (phydev->speed =3D=3D SPEED_10000 && - mv2222_is_10g_capable(phydev)) { - priv->line_interface =3D PHY_INTERFACE_MODE_10GBASER; + if (mv2222_is_sgmii_capable(phydev)) { + priv->line_interface =3D PHY_INTERFACE_MODE_SGMII; changed =3D true; } =20 break; + case PHY_INTERFACE_MODE_1000BASEX: case PHY_INTERFACE_MODE_SGMII: - ret =3D mv2222_set_sgmii_speed(phydev); - if (ret < 0) - return ret; + if (mv2222_is_10g_capable(phydev)) { + priv->line_interface =3D PHY_INTERFACE_MODE_10GBASER; + changed =3D true; + } =20 break; default: @@ -310,6 +245,29 @@ static int mv2222_setup_forced(struct phy_device *phyd= ev) return ret; } =20 + return 0; +} + +static int mv2222_setup_forced(struct phy_device *phydev) +{ + struct mv2222_data *priv =3D phydev->priv; + int ret; + + if (priv->line_interface =3D=3D PHY_INTERFACE_MODE_10GBASER) { + if (phydev->speed < SPEED_10000 && + phydev->speed !=3D SPEED_UNKNOWN) { + ret =3D mv2222_swap_line_type(phydev); + if (ret < 0) + return ret; + } + } + + if (priv->line_interface =3D=3D PHY_INTERFACE_MODE_SGMII) { + ret =3D mv2222_set_sgmii_speed(phydev); + if (ret < 0) + return ret; + } + return mv2222_disable_aneg(phydev); } =20 @@ -323,17 +281,9 @@ static int mv2222_config_aneg(struct phy_device *phyde= v) return 0; =20 if (phydev->autoneg =3D=3D AUTONEG_DISABLE || - phydev->speed =3D=3D SPEED_10000) + priv->line_interface =3D=3D PHY_INTERFACE_MODE_10GBASER) return mv2222_setup_forced(phydev); =20 - if (priv->line_interface =3D=3D PHY_INTERFACE_MODE_10GBASER && - mv2222_is_1gbx_capable(phydev)) { - priv->line_interface =3D PHY_INTERFACE_MODE_1000BASEX; - ret =3D mv2222_config_line(phydev); - if (ret < 0) - return ret; - } - adv =3D linkmode_adv_to_mii_adv_x(priv->supported, ETHTOOL_LINK_MODE_1000baseX_Full_BIT); =20 @@ -367,6 +317,120 @@ static int mv2222_aneg_done(struct phy_device *phydev) return (ret & BMSR_ANEGCOMPLETE); } =20 +/* Returns negative on error, 0 if link is down, 1 if link is up */ +static int mv2222_read_status_10g(struct phy_device *phydev) +{ + static int timeout; + int val, link =3D 0; + + val =3D phy_read_mmd(phydev, MDIO_MMD_PCS, MDIO_STAT1); + if (val < 0) + return val; + + if (val & MDIO_STAT1_LSTATUS) { + link =3D 1; + + /* 10GBASE-R do not support auto-negotiation */ + phydev->autoneg =3D AUTONEG_DISABLE; + phydev->speed =3D SPEED_10000; + phydev->duplex =3D DUPLEX_FULL; + } else { + if (phydev->autoneg =3D=3D AUTONEG_ENABLE) { + timeout++; + + if (timeout > AUTONEG_TIMEOUT) { + timeout =3D 0; + + val =3D mv2222_swap_line_type(phydev); + if (val < 0) + return val; + + return mv2222_config_aneg(phydev); + } + } + } + + return link; +} + +/* Returns negative on error, 0 if link is down, 1 if link is up */ +static int mv2222_read_status_1g(struct phy_device *phydev) +{ + static int timeout; + int val, link =3D 0; + + val =3D phy_read_mmd(phydev, MDIO_MMD_PCS, MV_1GBX_STAT); + if (val < 0) + return val; + + if ((phydev->autoneg =3D=3D AUTONEG_ENABLE && + !(val & BMSR_ANEGCOMPLETE))) { + timeout++; + + if (timeout > AUTONEG_TIMEOUT) { + timeout =3D 0; + + val =3D mv2222_swap_line_type(phydev); + if (val < 0) + return val; + + return mv2222_config_aneg(phydev); + } + + return 0; + } + + if (!(val & BMSR_LSTATUS)) + return 0; + + link =3D 1; + + val =3D phy_read_mmd(phydev, MDIO_MMD_PCS, MV_1GBX_PHY_STAT); + if (val < 0) + return val; + + if (val & MV_1GBX_PHY_STAT_AN_RESOLVED) { + if (val & MV_1GBX_PHY_STAT_DUPLEX) + phydev->duplex =3D DUPLEX_FULL; + else + phydev->duplex =3D DUPLEX_HALF; + + if (val & MV_1GBX_PHY_STAT_SPEED1000) + phydev->speed =3D SPEED_1000; + else if (val & MV_1GBX_PHY_STAT_SPEED100) + phydev->speed =3D SPEED_100; + else + phydev->speed =3D SPEED_10; + } + + return link; +} + +static int mv2222_read_status(struct phy_device *phydev) +{ + struct mv2222_data *priv =3D phydev->priv; + int link; + + phydev->link =3D 0; + phydev->speed =3D SPEED_UNKNOWN; + phydev->duplex =3D DUPLEX_UNKNOWN; + + if (!priv->sfp_link) + return 0; + + if (priv->line_interface =3D=3D PHY_INTERFACE_MODE_10GBASER) + link =3D mv2222_read_status_10g(phydev); + else + link =3D mv2222_read_status_1g(phydev); + + if (link < 0) + return link; + + phydev->link =3D link; + + return 0; +} + static int mv2222_resume(struct phy_device *phydev) { return mv2222_tx_enable(phydev); @@ -428,11 +492,7 @@ static int mv2222_sfp_insert(void *upstream, const str= uct sfp_eeprom_id *id) return ret; =20 if (mutex_trylock(&phydev->lock)) { - if (priv->line_interface =3D=3D PHY_INTERFACE_MODE_10GBASER) - ret =3D mv2222_setup_forced(phydev); - else - ret =3D mv2222_config_aneg(phydev); - + ret =3D mv2222_config_aneg(phydev); mutex_unlock(&phydev->lock); } =20 --=20 2.26.3