Linux-ARM-Kernel Archive on lore.kernel.org
 help / color / mirror / Atom feed
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




  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