[AUTv6x] Detect and reset overcurrent conditions #25989
Draft
+94
−0
Conversation
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
alexcekay
force-pushed
the
pr-reset-overcurrent-v6x
branch
from
8127094 to
5a38db6
Compare
Contributor
Author
|
The reason I'm not merging this yet, is that the eFuses try up to 15 times to restart the power supply after an overcurrent event. So I'm trying to understand why that is not enough. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Solved Problem
When an overcurrent condition was detected, the eFuse never retried later.
This is a Skynode specific problem, as the eFuse part we used is the latching kind.
In addition the assembly option maps the 5V_PERIPH and 5V_CAN1_GPS1 enable lines into one pin, even though we have two pins to check the overcurrent conditions separately. So now logically we have to treat them as one power rail.
Solution
Poll in software and reset the fuse.
Alternatives
Maybe it should stop retrying after 10 attempts or so?
Test coverage
Tested on a Skynode X.