Skip to content
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

Add getResetReason() by palmerr23 #2516

Merged
merged 5 commits into from
Oct 5, 2024

Conversation

retsifp
Copy link
Contributor

@retsifp retsifp commented Oct 4, 2024

Add code from @palmerr23 in #1843, slight changes were necessary in order to work and make the compiler happy.
Don't have a debug probe here, so I leave that part as-is, maybe someone finds some time to test it.

@retsifp
Copy link
Contributor Author

retsifp commented Oct 4, 2024

@earlephilhower This turns out to be harder than expected, but I hope what I now added for RP2350 makes sense.
I don't have any RP2350 available, but I hope what I looked up in powman.h makes sense.
Any tips appreciated! :)

@earlephilhower
Copy link
Owner

Thanks for putting in the effort to package this! I have plenty of RP2350s, so will give that side a try and fix anything I find for the merge. Thx again!

@earlephilhower
Copy link
Owner

Quick check on the RP2350 shows it works pretty well, thanks! Good to go!

@earlephilhower earlephilhower merged commit 2f55223 into earlephilhower:master Oct 5, 2024
26 checks passed
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.

2 participants