CCU Send CAN Messages Faster#509
Merged
Merged
Conversation
dchansen06
added
Enhancement
dchansen06
force-pushed
the
ChargingTest
branch
5 times, most recently
from
72bbc2c to
af5d761
Compare
Using ECU value of `1000 >> 4 = 625` Signed-off-by: Daniel Hansen <dchansen06@gmail.com>
Signed-off-by: Daniel Hansen <dchansen06@gmail.com>
Signed-off-by: Daniel Hansen <dchansen06@gmail.com>
Signed-off-by: Daniel Hansen <dchansen06@gmail.com>
Signed-off-by: Daniel Hansen <dchansen06@gmail.com>
dchansen06
changed the title
dchansen06
commented
Jun 22, 2026
dchansen06
left a comment
dchansen06
left a comment
Contributor
Author
There was a problem hiding this comment.
PR only really changes CAN_TIMER_SEND_PERIOD_US to a still safe value. Main CCU has not been formally tested and this PR does not change that status (and if it worked previously it will continue to work etc)
dchansen06
changed the title
Contributor
Author
Per Discord This PR is now irrelevant but this change is fine and "just in case" it is better to not throw things away yet |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Increase CCU CAN Send
Problem and Scope
Need to send CAN faster
Description
Set CAN send speed to match ECU
Gotchas and Limitations
No testing previously or now of any formal degree. If it worked before it will continue to work etc.
Testing
Testing Details
None
Larger Impact
Match ECU
Additional Context and Ticket
None