Thread (34 messages) 34 messages, 4 authors, 1d ago

Re: [PATCH v3 0/7] net: dsa: mt7530: support EcoNet EN751221

flat view

From: Caleb James DeLisle <cjd@cjdns.fr>
Date: 2026-10-28 16:33:29
Also in: linux-arm-kernel, linux-devicetree, linux-mediatek, lkml

Hello Christian,


Thank you for testing!


On 28/10/2026 17:03, Cristian Papa wrote:
Hi Caleb,

I tested v3 on a TP-Link Archer XR500v v1 (EN7526G: on-die switch plus the
MCM gigabit switch).

Setup: OpenWrt with kernel 6.18.44. I took the four mt7530 files from
econet_eth at 3a8bcd2e6398 (they contain every line added by 1/7, 3/7,
4/7, 5/7 and 7/7 of v3) and booted from RAM over TFTP, nothing written to
flash.

With the DT from en751221g.dtsi in openwrt#25330 (MCM "econet,en751221",
no "mediatek,mcm", dsa,member <0 1> on the on-die switch and <0 0> on the
MCM):

- 18/18 boots: all 10 TRGMII lanes found a window, no "calibration
   failed", "TRGMII inter-switch link initialized", DSA tree set up, ping
   over lan4.
- The chosen taps were stable across boots (+/-1): SoC->MCM 15-22,
   MCM->SoC 27-29. In the SoC->MCM direction the window always starts at
   tap 0.
- 6 of those boots ran iperf3 for 6 s each way through lan4: 0 RX CRC
   errors on on-die port 5 (~1.05M frames) and on MCM port 6 (~1.29M
   frames). In 4 of 10 boots MCM port 6 already had 1 CRC error before any
   traffic, probably from the calibration; it did not grow.

Following the binding in 6/7, the MCM does not come up on this board:

- "econet,en751221-gswitch": mt7530-mdio.c in 7/7 still matches
   "econet,en751221", so nothing probes the MCM and there are no LAN
   ports.

Whoops, I see I made an error on patch 7 and did not update the name to "gswitch", I will send a v4 soon.

quoted hunk ↗ jump to hunk
- "mediatek,mcm", which the binding requires for -gswitch: the probe
   fails with "Couldn't get our reset line", since nothing provides an
   "mcm" reset yet (the TODO in en751221g.dtsi).

Also expected as setting "mediatek,mcm" requires specifying the reset line, and reset for the MCM is on its own unique register and is not yet supported by en7523-clk.c reset controller. The OpenWrt implementation is a bit cheating by not bumping the reset line at all right now but in my testing it was not different when I manually bumped it so it seems okay for now

So I think 7/7 should match the new compatible, and the binding should not
require "mediatek,mcm" while there is no "mcm" reset for the EN751221.

Not covered yet: cold boots, and the other LAN ports beyond link-up on
lan3/lan4. I'm happy to give a Tested-by once the compatible strings line
up, and to rerun anything you'd like; the full boot logs are available.

I'll send a v4 and I will synchronize the PR on OpenWrt to it so you can test easily.


Thanks,

Caleb

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