[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <fc3f159a-0919-46d1-9fd8-8dc263391691@lunn.ch>
Date: Tue, 4 Nov 2025 04:55:19 +0100
From: Andrew Lunn <andrew@...n.ch>
To: Jacky Chou <jacky_chou@...eedtech.com>
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>,
Po-Yu Chuang <ratbert@...aday-tech.com>,
Joel Stanley <joel@....id.au>,
Andrew Jeffery <andrew@...econstruct.com.au>,
netdev@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-aspeed@...ts.ozlabs.org, taoren@...a.com
Subject: Re: [PATCH net-next v3 4/4] net: ftgmac100: Add RGMII delay support
for AST2600
> + rgmii_tx_delay = DIV_ROUND_CLOSEST(rgmii_tx_delay, rgmii_delay_unit);
> + if (rgmii_tx_delay >= 32) {
> + dev_err(&pdev->dev,
> + "The index %u of TX delay setting is out of range\n",
> + rgmii_tx_delay);
The index is not really interesting here, it is not something a DT
author uses. It is the delay in ps in the .dts file which is too big.
Andrew
Powered by blists - more mailing lists