Messages in this thread | | | Date | Mon, 27 Nov 2023 14:58:32 +0100 | From | Andrew Lunn <> | Subject | Re: [PATCH 0/3] net: microchip_t1s: additional phy support and collision detect handling |
| |
> Collision detection > This has been tested on a setup where one ARM system with a LAN8650 > mac-phy is daisy-chained to 8 mcus using lan8670 phys. Without the patch we > were limited to really short cables, about 1m per node, but we were > still getting a lot of connection drops. > With the patch we could increase the total cable length to at least 40M.
Did you do any testing of collision detection enabled, PLCA disabled?
You say you think this is noise related. But the noise should be the same with or without PLCA. I'm just thinking maybe collision detection is just plain broken and should always be disabled?
I've not read much about T1S, but if we assume it is doing good old fashioned CSMA/CD, with short cables the CS bit works well and the CD is less important. CD was needed when you have 1000m cable, and you can fit 64 bytes on the 1000m cable. So always turning of CD might be appropriate.
Andrew
| |