netdev - [net-next v2 0/4] Add AST2600 RGMII delay into ftgmac100

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID:
 <SEYPR06MB51342BAA627D12DA4DC32D6E9D39A@SEYPR06MB5134.apcprd06.prod.outlook.com>
Date: Tue, 26 Aug 2025 01:41:18 +0000
From: Jacky Chou <jacky_chou@...eedtech.com>
To: Andrew Lunn <andrew@...n.ch>
CC: Andrew Lunn <andrew+netdev@...n.ch>, "David S . Miller"
	<davem@...emloft.net>, Eric Dumazet <edumazet@...gle.com>, Jakub Kicinski
	<kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>, Rob Herring
	<robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
	<conor+dt@...nel.org>, Joel Stanley <joel@....id.au>, Andrew Jeffery
	<andrew@...econstruct.com.au>, Simon Horman <horms@...nel.org>, Heiner
 Kallweit <hkallweit1@...il.com>, Uwe Kleine-König
	<u.kleine-koenig@...libre.com>, Po-Yu Chuang <ratbert@...aday-tech.com>,
	"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux-arm-kernel@...ts.infradead.org"
	<linux-arm-kernel@...ts.infradead.org>, "linux-aspeed@...ts.ozlabs.org"
	<linux-aspeed@...ts.ozlabs.org>, "taoren@...a.com" <taoren@...a.com>
Subject: [net-next v2 0/4] Add AST2600 RGMII delay into ftgmac100

Hi Andrew,

> > > > This patch series adds support for configuring RGMII internal
> > > > delays for the Aspeed AST2600 FTGMAC100 Ethernet MACs.
> > >
> > > So i think you are doing things in the wrong order. You first need
> > > to sort out the mess of most, if not all, AST2600 have the wrong
> > > phy-mode, because the RGMII delay configuration is hidden, and set
> wrongly.
> > >
> > > Please fix that first.
> > >
> >
> > The RGMII delay is configured in U-boot stage, and it is not changed
> > when booting to Linux. I want to know whether the first thing to
> > correct here is whether the phy-mode in aspeed-ast2600-evb.dts is
> > rgmii-id. Our AST2600 EVB, there is no delay on board, so, I need to change
> the phy-mode to "rgmii-id" to meet the RGMII usage first?
> 
> If there is no delay on the PCB, then phy-mode is "rgmii-id".

I would like to discuss with you how we fix the RGMII of AST2600 in this thread.
And thank you for your patience in reviewing our code.

Currently, the RGMII delay in AST2600 is configured in U-boot stage, not in Linux.
The ftgmac driver will not use the phy-mode to configure the RGMII delay on MAC side.

I list the parts that I think need to be modified.
1. Change the phy-mode to "rgmii-id" in aspeed-ast2600-evb.dts.
2. Add the tx/rx-internal-delay-ps in dts.
3. Add RGMII delay configuration in ftgmac driver. If the tx/rx-internal-delay-ps has not existed,
  according to the phy-mode to configure default value.

These are the fix items I can think of.
Could you point out what I miss or a clear direction to correct the RGMII mode on AST2600?

Thanks,
Jacky