Skip to content

Add ZX310S-4T2XT device support#215

Open
logicog wants to merge 3 commits into
mainfrom
2x10g
Open

Add ZX310S-4T2XT device support#215
logicog wants to merge 3 commits into
mainfrom
2x10g

Conversation

@logicog
Copy link
Copy Markdown
Owner

@logicog logicog commented May 4, 2026

The ZX310S-4T2XT is a managed device with 2x2.5GBit and 2x10GBit Ethernet.
In the OEM image, the system LED is configured manually, but it is possible to use the automatic LED control as for all the other devices. Still, one issue was the setup of RTL837X_PIN_MUX_0, which is set to 0x30000000 on reset for this device, which means that all LEDs are turned off (apart from the System-LED). In the custom section of the machine entry, therefore this is configured explicitly.

I attach a full trace of the OEM boot with all register/PHY/SDS settings done after the boot-loader is through and console output available. Such traces could in principle be used to patch the PHYs as is done in the OEM firmware:
trace_zx310s_4t2xt.txt

@logicog logicog force-pushed the 2x10g branch 2 times, most recently from 11d6c02 to 8696824 Compare May 7, 2026 13:23
@logicog logicog changed the title WIP: Add ZX310S-4T2XT device support Add ZX310S-4T2XT device support May 7, 2026
@logicog
Copy link
Copy Markdown
Owner Author

logicog commented May 7, 2026

I managed to clean up the LED configuration. This is now ready for review.

Comment thread doc/devices/ZX310S-4T2XT.md Outdated
The device is fully supported:
- All 4 2.5GBASE-T RJ45 ports work at 10/100/1000/2500 Mbps
- The 10GBit port works. TODO: Fix EEE, speed selection
- The SFP+ port supports 1G, 2.5G and 10G modules
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no sfp....

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops.

Comment thread machine.c Outdated
0x1d, 0x20, 0x21 },
.port_led_set = { 0, 0, 0, 1, 0, 0, 0, 0, 1},
/* Ports 1-4: Green: 2.5GBit, Orange: 10/100/1000MBit
* Port 5: Green: 10GBit, Orange: <= 5GBit
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and Port 6 probably

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops, will fix.

Comment thread doc/devices/ZX310S-4T2XT.md Outdated
### What works
The device is fully supported:
- All 4 2.5GBASE-T RJ45 ports work at 10/100/1000/2500 Mbps
- The 10GBit port works. TODO: Fix EEE, speed selection
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

still TODO?

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No. :-)

@logicog logicog force-pushed the 2x10g branch 2 times, most recently from c1c4307 to a6f3de2 Compare May 8, 2026 04:37
@logicog
Copy link
Copy Markdown
Owner Author

logicog commented May 8, 2026

Thanks for reviewing! Fixed all of the above and pushed the PR again.

Copy link
Copy Markdown
Collaborator

@feelfree69 feelfree69 May 8, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm confused, this file is for a -XH switch with one 10GBaseT and one SFP+ port, isn't it?

@logicog
Copy link
Copy Markdown
Owner Author

logicog commented May 8, 2026

No, this is the device with 2 10GBit Ethernet ports, ending in 2XT. I believe, I initially linked to the wrong photos.

@feelfree69
Copy link
Copy Markdown
Collaborator

No, this is the device with 2 10GBit Ethernet ports, ending in 2XT. I believe, I initially linked to the wrong photos.

But there are two .md Files in this PR.
A new one ending in XT -> ok.
And a modified one, ending in XH -> are these changes intentionally?

@logicog
Copy link
Copy Markdown
Owner Author

logicog commented May 8, 2026

Oh, no! I edited the wrong file when you asked for the changes in the .md file and then made it part of the PR. I will have to clean this up. Sorry about that. Will take a bit to disentangle the PR.

@logicog logicog force-pushed the 2x10g branch 2 times, most recently from 9956956 to 9c8901f Compare May 8, 2026 12:15
logicog added 3 commits May 8, 2026 14:17
The ZX310S-4T2XT is a managed device with 2x2.5GBit and 2x10GBit Ethernet.
It is sold, among others, by Horaco as an unbranded device.

CPU: RTL8372
Flash: 2MByte Winbond W25Q16DV (U3)
PHY 2x RTL8261BE
@logicog
Copy link
Copy Markdown
Owner Author

logicog commented May 8, 2026

Fixed and pushed again.

@feelfree69
Copy link
Copy Markdown
Collaborator

Perfect!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants