From: "Gupta, Suraj" <suraj.gupta2@amd.com>
To: Andrew Lunn <andrew@lunn.ch>
Cc: Radhey Shyam Pandey <radhey.shyam.pandey@amd.com>,
Andrew Lunn <andrew+netdev@lunn.ch>,
"David S . Miller" <davem@davemloft.net>,
Eric Dumazet <edumazet@google.com>,
Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
Michal Simek <michal.simek@amd.com>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Russell King <linux@armlinux.org.uk>,
netdev@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org, devicetree@vger.kernel.org
Subject: Re: [PATCH net-next v3 2/6] dt-bindings: net: xlnx,xxv-ethernet: Add Xilinx XXV 10G/25G Ethernet
Date: Tue, 1 Sep 2026 22:31:47 +0530 [thread overview]
Message-ID: <320e31b8-7e26-404e-a1bc-d19ecb59ff3d@amd.com> (raw)
In-Reply-To: <b5126d72-734b-4f4d-b953-807579c8583e@lunn.ch>
On 9/1/2026 8:01 PM, Andrew Lunn wrote:
> On Tue, Sep 01, 2026 at 08:48:18AM +0530, Gupta, Suraj wrote:
>>
>>
>> On 9/1/2026 7:57 AM, Andrew Lunn wrote:
>>>> + The Xilinx XXV Ethernet Subsystem implements a 10G/25G Ethernet MAC with an
>>>> + in-core PCS/PMA driving a high-speed serial transceiver. It supports the
>>>> + 10GBASE-R and 25GBASE-R line rates, with 25G operation following the 25G
>>>> + Ethernet Consortium specification. It has no external MDIO PHY
>>>
>>> What about MDIO bus?
>>>
>>> Andrew
>>
>> The 10G/25G (XXV) Ethernet Subsystem IP does not provide an MDIO master.
>
I'll try to make the overall architecture clear, please let me know if I
miss something and additional details needed. The datapath is:
DMA (AXIDMA/ MCDMA) <-> (XXV MAC + PCS/PMA) <-> GT serial transceiver
<-> optical module / backplane.
MAC's in-core PCS/PMA drives a hardened GT (SerDes) transceiver directly
at 10GBASE-R / 25GBASE-R, and the GT is configured over its DRP
interface (the "drp" clock in this binding). There is no discrete PHY
chip and no MDIO or I2C master inside the IP.
XXV ethernet PG: https://docs.amd.com/r/en-US/pg210-25g-ethernet
> O.K. Please change the wording to make that clear. There are plenty to
> 10G PHYs available, but it seems like you do not consider that a use
> case.
>
Correct, the IP has no MDIO bus master, so an externally MDIO-managed
PHY is not a supported use case. I will reword the description in v4 to
state that explicitly, e.g.:
"The subsystem has an in-core PCS/PMA that drives a GT serial
transceiver; it has no MDIO bus master and is therefore not managed
through an external MDIO PHY. Link state is taken from the MAC's own
status registers."
> Has the MDIO bus master been replaced with I2C? I assume you intend
> this device to be paired by an SFP cage, so an I2C bus will be
> required. Is that a different IP which needs to be licensed?
>
The XXV Ethernet core contains neither an MDIO nor an I2C master. When a
hardware design pairs it with an SFP/SFP+ cage, only the high-speed data
path runs through this IP: the in-core PCS/PMA drives the GT
transceiver, whose serial TX/RX lanes connect to the module. The module
management side-band -- I2C for the module EEPROM/diagnostics is not
part of this IP. It is provided by separate board logic, i.e. a
different IP: a standard AXI IIC controller. Those are general-purpose
soft cores from the standard Vivado IP catalog; each has its own
devicetree node and binding, so the SFP management path looks outside
the scope of this Ethernet binding.
Please let me know if I should describe sfp in this binding.
>> The contents of the appropriate MDIO registers are available in the IP control &
>> status registers.
>
> What does that mean?
>
> Andrew
I meant that the PCS/link state a driver would normally read from a PHY
over MDIO (link up, resolved speed, block-lock/RX status) is exposed
directly in the XXV IP's control & status registers, because the PCS/PMA
is in-core.
Thanks,
Suraj
next prev parent reply other threads:[~2026-09-01 17:02 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-31 15:08 [PATCH net-next v3 0/6] net: xilinx: axienet: Add 10G/25G (XXV) ethernet support Suraj Gupta
2026-08-31 15:08 ` [PATCH net-next v3 1/6] net: xilinx: axienet: Introduce axienet_config for MAC-specific ops Suraj Gupta
2026-08-31 15:08 ` [PATCH net-next v3 2/6] dt-bindings: net: xlnx,xxv-ethernet: Add Xilinx XXV 10G/25G Ethernet Suraj Gupta
2026-09-01 2:27 ` Andrew Lunn
2026-09-01 3:18 ` Gupta, Suraj
2026-09-01 14:31 ` Andrew Lunn
2026-09-01 17:01 ` Gupta, Suraj [this message]
2026-08-31 15:08 ` [PATCH net-next v3 3/6] net: xilinx: axienet: Add 10G/25G (XXV) ethernet support Suraj Gupta
2026-09-01 2:52 ` Andrew Lunn
2026-09-01 3:27 ` Gupta, Suraj
2026-08-31 15:08 ` [PATCH net-next v3 4/6] net: xilinx: axienet: Make axienet_rmon_ranges non-static for reuse Suraj Gupta
2026-08-31 15:08 ` [PATCH net-next v3 5/6] net: xilinx: axienet: Dispatch statistics through axienet_config ops Suraj Gupta
2026-08-31 15:08 ` [PATCH net-next v3 6/6] net: xilinx: axienet: Add statistics support for XXV ethernet Suraj Gupta
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=320e31b8-7e26-404e-a1bc-d19ecb59ff3d@amd.com \
--to=suraj.gupta2@amd.com \
--cc=andrew+netdev@lunn.ch \
--cc=andrew@lunn.ch \
--cc=conor+dt@kernel.org \
--cc=davem@davemloft.net \
--cc=devicetree@vger.kernel.org \
--cc=edumazet@google.com \
--cc=krzk+dt@kernel.org \
--cc=kuba@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=michal.simek@amd.com \
--cc=netdev@vger.kernel.org \
--cc=pabeni@redhat.com \
--cc=radhey.shyam.pandey@amd.com \
--cc=robh@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox