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

[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-07 01:12:13
Also in: linux-arm-kernel, linux-aspeed, linux-devicetree, lkml

Hi Andrew
quoted
There are four MACs in the AST2600. In the DT bindings and DTS files,
what would be the recommended way to identify which MAC is which?
In version 3 of my patches, I used the aliases in the DTSI file to
allow the driver to get the MAC index.
It is a bit ugly, but you are working around broken behaviour, so sometimes you
need to accept ugly. The addresses are fixed. You know
1e660000 is mac0, 1e680000 is mac1, etc. Put the addresses into the driver,
for compatible aspeed,ast2600-mac.
I used this fixed address as MAC index in the first version of this series.
But the other reviewer mentioned maybe there has the other better way to 
identify index.
https://lore.kernel.org/all/20250317095229.6f8754dd@fedora.home/ (local)
I find the "aliase", on preparing the v2 and v3, I think it may be a way to
do that. But I am not sure.
So, I would like to confirm the other good way before submitting the next
version.

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