-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Toyota: 2021-23 RAV4 Prime #1287
Conversation
This should be G2G after the pandacan reference becomes available to update. |
@sshane What do you think about capping MAC fail logging like this? Seems unwise to cloudlog fails at dozens/sec indefinitely when someone starts up with a wrong/missing key, but not sure we have similar limits for counter/checksum fails. I do want to know if it happens during a drive, and I want to know if it happens sporadically during key reset/rollover, etc. |
Seems fine |
Credit to @pd0wm, just bringing commaai/openpilot#31179 current after the refactor.
Tests for SecOC? TBDlaterTODO in a future PR: