Re: [PATCH 5/9] ax88179_178a: Add support for ethtool pause parameter configuration

[Andrew Lunn <andrew@lunn.ch>]

On Wed, Jul 01, 2026 at 06:22:31PM +0200, Birger Koblitz wrote:
> Hi Andrew,
> 
> thanks for reviewing this patch-series! I will answer to the other questions later,
> so that the answers stay together. But it is probably best if I give this answer
> immediately:
> On 7/1/26 17:08, Andrew Lunn wrote:
> > > > +static void ax88179a_get_pauseparam(struct net_device *net, struct ethtool_pauseparam *pause)
> > > > +	if (!(bmcr & BMCR_ANENABLE)) {
> > > > +		pause->autoneg = 0;
> > > > +		pause->rx_pause = 0;
> > > > +		pause->tx_pause = 0;
> > > The best way to have this correct is to use phylink, but for that you'd need to
> > > have a proper PHY driver instead of using the mii_ API here.
> > 
> > I said the some to one of the other patches.
> > 
> > Do we know what PHYs are being used? Can register 2 and 3 be read to
> > get the PHY IDs?
> > 
> > 	Andrew
> 
> I tested
>   id1 = ax88179_mdio_read(dev->net, dev->mii.phy_id, MII_PHYSID1);
>   id2 = ax88179_mdio_read(dev->net, dev->mii.phy_id, MII_PHYSID2);
> 
> and got:

Thanks for these numbers.

> Renkforce AX88179A: ID1 7c9f, ID2 7061
> Delock AX88279  ID1 03a2, ID2 a411

air_en8811h.c:#define EN8811H_PHY_ID		0x03a2a411

> UGreen AX88772D ID1 e65b, ID2 2c61
> TP-Link AX88179A ID1 e65b, ID2 2c61

The two ID registers contain part of an OUI, but it has some bits
missing. So it is not so easy to look it up.

However, anything using the MII framework basically assumes a very
simple PHY and only looks at the 802.3 defined registers. So the
genphy generic PHY driver might be sufficient for when there is not a
specific driver. At lot depends on how much extra code there is
accessing the PHY registers in the driver.

	 Andrew