Re: [PATCH RFC] Providing a helper for PCS inband negotiation

Russell King (Oracle)
2023-05-15T15:35:42Z

On Mon, May 15, 2023 at 03:58:09PM +0200, Andrew Lunn wrote:
> > 2. XLGMII.. Looking at the XPCS driver, it's unclear whether Clause 73
> > AN gets used for this. A quick scan of IEEE 802.3 suggests that
> > XLGMII doesn't have any support for any inband signalling, and it's
> > just an intermediary protocol between the MAC (more specifically the
> > RS, but for the purposes of this I'll just refer to MAC) and the
> > attached PCS, and any autonegotiation happens after the XLGMII link.
>
> So isn't XLGMII then a generic PHY thing, not a phylink thing?

Honestly, I'm really not sure. It feels more like an internal-SoC
thing. See my last diagram why I think this.

> Or am i not correctly understanding how
> drivers/phy/marvell/phy-*-comphy.c and
> drivers/phy/microchip/*_serdev.c fit into the overall picture?

In the case of serdes-based protocols with an external PHY, the general
structure we have is:

------------------------------+
SoC |
+-----+ +-----+ +--------+ | +--------------+
| MAC +---+ PCS +--+ SERDES +------------+ Ethernet PHY +---- Media
+-----+ +-----+ +--------+ | ^ +--------------+
------------------------------+ |
|
PHY_INTERFACE_MODE_xxx has referred to this bit, and this is the bit
where inband can occur.

In the case of multi-rate implementations, there can be one of many
different PCS that can be placed there, and the SERDES handles
converting the data stream from the PCS into its appropriate
electrical form, essentially covering the PMA and PMD functions.
(That same SERDES block generally can also handle PCIe and SATA.)

In the case of non-serdes based protocols, then it's essentially the
same as the above, but with the PCS and SERDES blocks removed.

For Fibre based connections:

------------------------------+
SoC |
+-----+ +-----+ +--------+ |
| MAC +---+ PCS +--+ SERDES +------------ Media
+-----+ +-----+ +--------+ | ^
------------------------------+ |
|
PHY_INTERFACE_MODE_xxx has referred to this bit, and 1000BASE-X runs
negotiation on this. 10GBASE-R also used for fibre has no negotiation,
but there's still the 10GBASE-R PCS and the 802.3 PMA/PMD are subsumed
by the SERDES block.

What I think seems to be the case with XPCS is:

------------------------------+
SoC |
+-----+ +-----+ +--------+ |
| MAC +---+ PCS +--+ SERDES +----------- Media
+-----+ ^ +-----+ +--------+ |
--------|---------------------+
|
PHY_INTERFACE_MODE_xxx seems to be referring to this bit. When clause 73
negotiation is used, that happens where I've stated "media" above, and
that's involved with negotiating backplane protocols e.g. 40GBASE-KR4,
40GBASE-CR4, 25GBASE-KR, 10GBASE-KR, 1000BASE-KX etc on the bit I've
called "Media" above.

However, we can also have this for a fibre link:

------------------------------+
SoC |
+-----+ +-----+ +--------+ |
| MAC +---+ PCS +--+ SERDES +----------- Fibre
+-----+ ^ +-----+ +--------+ | ^
--------|---------------------+ |
| |
XLGMII 40GBASE-R

Given that in this case, we'd want PHY_INTERFACE_MODE_xxx to say
40GBASE-R, using the existing PHY_INTERFACE_MODE_xxx to specify at
where I've pointed XLGMII just makes things confused... but in the
case above with clause 73 negotiation, we wouldn't have a standard
PHY_INTERFACE_MODE_xxx specifier for the external "media" side
because that's dependent on the result of the negotiation.

So... this seems to be a right can of wriggly things.