Re: [PATCH net-next v2 2/3] phy: Add Open Alliance helpers for the PHY framework
From: Andrew Lunn
Date: Sat Aug 10 2024 - 11:46:36 EST
> > > 6.4 Polarity Detection and Correction (POL)
> > > 6.4.1 Polarity Detection (DET)
> > > 6.4.2 Polarity Correction (COR)
> >
> > Could these be mapped to ETH_TP_MDI* ?
>
> Yes, but it will look confusing in the user space. To make better
> representation in ethtool we will probably need a new port type. For
> example instead of PORT_TP it will be PORT_STP (single twiste pair) or
> PORT_SPE (single pair ethernet). What do you think?
Thinking about this some more....
The Marvell PHYs can indicate per pair if the polarity is wrong and
has been corrected, for 10BaseT and 1000BaseT. The datasheet indicates
that for 100BaseTX polarity does not matter.
So i agree, MDI crossover is the wrong way to indicate this. We should
add polarity indication, for up to 4 pairs.
Andrew