mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-05-09 09:01:36 -04:00
net: phy: realtek: change order of calls in C22 read_status()
Always call rtlgen_read_status() first, so genphy_read_status() which is called by it clears bits in case auto-negotiation has not completed. Also clear 10GBT link-partner advertisement bits in case auto-negotiation is disabled or has not completed. Suggested-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk> Signed-off-by: Daniel Golle <daniel@makrotopia.org> Link: https://patch.msgid.link/b15929a41621d215c6b2b57393368086589569ec.1728565530.git.daniel@makrotopia.org Signed-off-by: Paolo Abeni <pabeni@redhat.com>
This commit is contained in:
committed by
Paolo Abeni
parent
081c9c0265
commit
68d5cd09e8
@@ -949,17 +949,25 @@ static void rtl822xb_update_interface(struct phy_device *phydev)
|
|||||||
|
|
||||||
static int rtl822x_read_status(struct phy_device *phydev)
|
static int rtl822x_read_status(struct phy_device *phydev)
|
||||||
{
|
{
|
||||||
if (phydev->autoneg == AUTONEG_ENABLE) {
|
int lpadv, ret;
|
||||||
int lpadv = phy_read_paged(phydev, 0xa5d, 0x13);
|
|
||||||
|
|
||||||
if (lpadv < 0)
|
ret = rtlgen_read_status(phydev);
|
||||||
return lpadv;
|
if (ret < 0)
|
||||||
|
return ret;
|
||||||
|
|
||||||
mii_10gbt_stat_mod_linkmode_lpa_t(phydev->lp_advertising,
|
if (phydev->autoneg == AUTONEG_DISABLE ||
|
||||||
lpadv);
|
!phydev->autoneg_complete) {
|
||||||
|
mii_10gbt_stat_mod_linkmode_lpa_t(phydev->lp_advertising, 0);
|
||||||
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
return rtlgen_read_status(phydev);
|
lpadv = phy_read_paged(phydev, 0xa5d, 0x13);
|
||||||
|
if (lpadv < 0)
|
||||||
|
return lpadv;
|
||||||
|
|
||||||
|
mii_10gbt_stat_mod_linkmode_lpa_t(phydev->lp_advertising, lpadv);
|
||||||
|
|
||||||
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int rtl822xb_read_status(struct phy_device *phydev)
|
static int rtl822xb_read_status(struct phy_device *phydev)
|
||||||
|
|||||||
Reference in New Issue
Block a user