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

Wrong assignment of wakeup bits to external interrupts in EnergySaving library (Seeeduino XIAO) #70

Closed
jonas-koeritz opened this issue Jan 11, 2024 · 2 comments
Assignees
Labels
ArduinoCore-samd Label for ArduinoCore-samd UAY Unassigned yet

Comments

@jonas-koeritz
Copy link

Inside EnergySaving::enable_eic_wake, there is a switch/case that assigns the input pin number to a WAKEUPENx bit. This function mangles them up in a way that will never wake a Seeeduino XIAO correctly. The WAKEUPENx bits are not mixed up like the function suggests. Waking up from external interrupts start's working as soon as you set the WAKEUPENx bit of the pin used to ::begin() the EnergySaving calls with.

@MatthewJeffson MatthewJeffson added UAY Unassigned yet ArduinoCore-samd Label for ArduinoCore-samd labels Oct 9, 2024
@Lesords Lesords self-assigned this Oct 14, 2024
@Lesords
Copy link

Lesords commented Oct 16, 2024

Hello,

I'm very sorry to have kept you waiting so long.

Do you still have this problem now?

@Lesords
Copy link

Lesords commented Nov 4, 2024

Hello,

I'm going to close this issue, feel free to re-open it if you have any other questions.

[Autocommentary]

@Lesords Lesords closed this as completed Nov 4, 2024
@github-project-automation github-project-automation bot moved this from Todo to Done in Issues and PR Assemble Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ArduinoCore-samd Label for ArduinoCore-samd UAY Unassigned yet
Projects
Status: Done
Development

No branches or pull requests

3 participants