Thread (30 messages) 30 messages, 4 authors, 2025-11-07
STALE246d

[PATCH net-next v3 1/4] dt-bindings: net: ftgmac100: Add delay properties for AST2600

From: Jacky Chou <jacky_chou@aspeedtech.com>
Date: 2025-11-04 09:55:00
Also in: linux-aspeed, linux-devicetree, lkml, netdev

Hi Krzysztof

Thank you for your reply.
quoted
Create the new compatibles to identify AST2600 MAC0/1 and MAC3/4.
Add conditional schema constraints for Aspeed AST2600 MAC controllers:
- For "aspeed,ast2600-mac01", require rx/tx-internal-delay-ps properties
  with 45ps step.
- For "aspeed,ast2600-mac23", require rx/tx-internal-delay-ps properties
  with 250ps step.
That difference does not justify different compatibles. Basically you said they
have same programming model, just different hardware characteristics, so
same compatible.
This change was originally based on feedback from a previous review discussion.
At that time, another reviewer suggested introducing separate compatibles for 
MAC0/1 and MAC2/3 on AST2600, since the delay characteristics differ and they 
might not be fully compatible.

Thanks,
Jacky
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help