Re: [PATCH 0/3] phy: qcom: sgmii-eth: add Nord USXGMII support

From: Krzysztof Kozlowski

Date: Mon Sep 28 2026 - 06:23:47 EST


On Wed, Sep 23, 2026 at 04:58:03PM +0800, Zhangfei Gao wrote:
> On Wed, Sep 23, 2026 at 8:04 AM Mohd Ayaan Anwar
> <mohd.anwar@xxxxxxxxxxxxxxxx> wrote:
> >
> > This series adds support for the Ethernet SerDes found on the Qualcomm
> > Nord SoC.
> >
> > Nord uses a QMP v7 SerDes block, which is an updated version of the
> > SGMII SerDes PHY IP already handled by phy-qcom-sgmii-eth. The driver
> > currently supports the 1.25 Gbaud and 3.125 Gbaud configurations used by
> > SGMII, 1000BASE-X and 2500BASE-X. Nord additionally requires the SerDes
> > to operate at 10.3125 Gbaud for its 10 Gigabit Ethernet interfaces.
> >
> > The series can be broadly divided into three parts:
> >
> > 1. Add the qcom,nord-dwmac-sgmii-phy compatible to the binding.
> > 2. Prepare the driver for PHY modes which require different SerDes
> > programming and readiness checks. Existing SA8775P behaviour is
> > unchanged.
> > 3. Add the QMP v7 register definitions, 10.3125 Gbaud programming
> > sequence and readiness checks required for USXGMII and 10GBASE-R.
> >
> > The changes have been tested on the Nord Ride SX platform, which has a
> > MV-Q6223 switch, running in fixed link USXGMII 10G mode.
> >
> > Signed-off-by: Mohd Ayaan Anwar <mohd.anwar@xxxxxxxxxxxxxxxx>
> > ---
> > Mohd Ayaan Anwar (3):
> > dt-bindings: phy: qcom,sa8775p-dwmac-sgmii-phy: add Nord compatible
> > phy: qcom: sgmii-eth: prepare for additional PHY modes
> > phy: qcom: sgmii-eth: add Nord USXGMII support
> >
> > .../bindings/phy/qcom,sa8775p-dwmac-sgmii-phy.yaml | 1 +
> > drivers/phy/qualcomm/phy-qcom-qmp-pcs-sgmii.h | 5 +
> > drivers/phy/qualcomm/phy-qcom-qmp-qserdes-com-v7.h | 2 +
> > .../phy/qualcomm/phy-qcom-qmp-qserdes-txrx-v7.h | 9 +
> > drivers/phy/qualcomm/phy-qcom-sgmii-eth.c | 226 ++++++++++++++++++---
> > 5 files changed, 219 insertions(+), 24 deletions(-)
> > ---
> > base-commit: a8c591ed6b672915e0be57843f943a2a723aff40
> > change-id: 20260923-nord_ethernet_serdes-c486f715f383
> >
> > Best regards,
> > --
> > Mohd Ayaan Anwar <mohd.anwar@xxxxxxxxxxxxxxxx>
> >
>
> Good Job,
>
> Tested-by: Zhangfei Gao <zhangfei.gao@xxxxxxxxxxxxxxxx>

This has to stop.

Why again adding fake testing to the bindings? You cannot reply twice?

These two weeks it's some plague of such tags...

Best regards,
Krzysztof