[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID:
<SEYPR06MB513443D43C99A04B3493ABDB9DD82@SEYPR06MB5134.apcprd06.prod.outlook.com>
Date: Thu, 20 Mar 2025 08:06:23 +0000
From: Jacky Chou <jacky_chou@...eedtech.com>
To: Krzysztof Kozlowski <krzk@...nel.org>, Andrew Lunn <andrew@...n.ch>
CC: Russell King <linux@...linux.org.uk>, Maxime Chevallier
<maxime.chevallier@...tlin.com>, "andrew+netdev@...n.ch"
<andrew+netdev@...n.ch>, "davem@...emloft.net" <davem@...emloft.net>,
"edumazet@...gle.com" <edumazet@...gle.com>, "kuba@...nel.org"
<kuba@...nel.org>, "pabeni@...hat.com" <pabeni@...hat.com>, "robh@...nel.org"
<robh@...nel.org>, "krzk+dt@...nel.org" <krzk+dt@...nel.org>,
"conor+dt@...nel.org" <conor+dt@...nel.org>, "joel@....id.au"
<joel@....id.au>, "andrew@...econstruct.com.au"
<andrew@...econstruct.com.au>, "ratbert@...aday-tech.com"
<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>, BMC-SW <BMC-SW@...eedtech.com>
Subject:
回覆: 回覆: 回覆: [net-next 4/4] net: ftgmac100: add RGMII delay for AST2600
Hi Krzysztof,
>
> git grep multipleOf:
>
> e.g.
> oneOf:
> - minimum: 45
> maximum: ...
> multipleOf: 45
> - minimum: 1500
> maximum: ...
> multipleOf: 250
>
> >
> > Lets see what the DT Maintainers say, but it could be you need two
> > different compatibles for mac0/1 to mac2/3 because they are not
> > actually compatible! You can then have a list per compatible.
> If this is the only, *only* difference, then just go with vendor property matching
> register value... but oh, wait, how person reading and writing the DTS would
> understand if "0x2" means 90 ps or 1750 ps? I don't see how the original
> binding was helping here in total. Just moving the burden from driver
> developer to DTS developer. :/
>
> If different instances are not the same, means the devices are not the same, so
> two compatibles seem reasonable.
Thank you for your reply and information.
We will modify this section by adding more descriptions and referencing other yaml.
Thanks,
Jacky
Powered by blists - more mailing lists