Thread (5 messages) flat view 5 messages, 3 authors, 15d ago

Re: [PATCH net-next v11] net: phy: Add driver for Motorcomm Quad 2.5GbE phy

From: Andrew Lunn <andrew@lunn.ch>
Date: 2026-08-18 01:13:26
Also in: lkml

+	if (link) {
+		phydev->link = 1;
+		phydev->pause = !!(lpa & BIT(10));
+		phydev->asym_pause = !!(lpa & BIT(11));
This is wrong.

Where do you see any other driver doing this?

phy_resolve_aneg_pause() will set phydev->pause and
phydev->asym_pause.

You should be trying to use genphy_read_status() to do most of the
work, and then mix in the 2.5G status information afterwards.

    Andrew
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help