Skip to content

Sync master time on network time#374

Draft
leducp wants to merge 4 commits into
masterfrom
fix_dc_sync
Draft

Sync master time on network time#374
leducp wants to merge 4 commits into
masterfrom
fix_dc_sync

Conversation

@leducp
Copy link
Copy Markdown
Owner

@leducp leducp commented Apr 15, 2026

To test on real hardware before merge

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 15, 2026

File Coverage Lines Branches
All files 77% 80% 75%
lib/include/kickcat/Error.h 91% 100% 83%
lib/include/kickcat/SBufQueue.h 80% 98% 63%
lib/master/include/kickcat/SIIParser.h 91% 100% 83%
lib/master/src/Bus.cc 91% 92% 89%
lib/master/src/CoE.cc 61% 71% 51%
lib/master/src/Link.cc 98% 100% 97%
lib/master/src/MailboxSequencer.cc 97% 100% 94%
lib/master/src/MasterOD.cc 99% 99% 100%
lib/master/src/Prints.cc 95% 95% 95%
lib/master/src/Slave.cc 27% 42% 12%
lib/master/src/dc.cc 0% 0% 0%
lib/slave/src/AbstractESC.cc 92% 100% 84%
lib/slave/src/AbstractEmulatedEEPROM.cc 0% 0% 0%
lib/slave/src/ESMStates.cc 96% 98% 94%
lib/slave/src/ESM.cc 95% 100% 90%
lib/slave/src/PDO.cc 92% 97% 88%
lib/slave/src/ESC/EmulatedESC.cc 57% 63% 51%
lib/slave/src/ESC/Lan9252.cc 0% 0% 0%
lib/src/Frame.cc 98% 100% 97%
lib/src/Mailbox.cc 89% 93% 85%
lib/src/SIIParser.cc 40% 50% 30%
lib/src/protocol.cc 94% 92% 96%
lib/src/CoE/EsiParser.cc 93% 95% 90%
lib/src/CoE/OD.cc 98% 98% 98%
lib/src/CoE/protocol.cc 93% 93% 94%
lib/src/CoE/CiA/DS402/StateMachine.cc 93% 94% 92%
lib/src/CoE/mailbox/request.cc 80% 84% 76%
lib/src/CoE/mailbox/response.cc 91% 96% 87%
lib/src/EoE/protocol.cc 0% 0% 0%

Minimum allowed coverage is 75%

Generated by 🐒 cobertura-action against 87189b2

…that will always diverge.

Refactor timer class.
@leducp
Copy link
Copy Markdown
Owner Author

leducp commented Apr 21, 2026

Some PLLs slaves goes into errors after a few minutes: something wrong with the fix to let the master adjust to the network time.
To be noted that the first slave/reference clock is an ET1100 from Beckhoff so we know for sure that this one is compliant.

slave 1001 al_status=0x18 code=0x0000
slave 1002 al_status=0x08 code=0x0000
slave 1003 al_status=0x08 code=0x0000
slave 1004 al_status=0x08 code=0x0000
slave 1005 al_status=0x08 code=0x0000
slave 1006 al_status=0x08 code=0x0000
slave 1007 al_status=0x08 code=0x0000
slave 1008 al_status=0x08 code=0x0000
slave 1009 al_status=0x08 code=0x0000
slave 1010 al_status=0x08 code=0x0000
slave 1011 al_status=0x14 code=0x0032
slave 1012 al_status=0x08 code=0x0000
slave 1013 al_status=0x08 code=0x0000
slave 1014 al_status=0x08 code=0x0000
slave 1015 al_status=0x08 code=0x0000
slave 1016 al_status=0x08 code=0x0000
slave 1017 al_status=0x08 code=0x0000

leducp added 2 commits April 21, 2026 16:01
Add a function to monitor the DC of the network
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.

1 participant