Skip to content

ExternalWakeup: LowPowerMode parameter SLEEP_MODE should be used #55

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

Merged
merged 1 commit into from
Apr 7, 2021

Conversation

ABOSTM
Copy link
Contributor

@ABOSTM ABOSTM commented Apr 6, 2021

ExternalWakeup: LowPowerMode parameter SLEEP_MODE should be used

Last parameter of attachInterruptWakeup(), aka "LowPowerMode",
should match lowpower state used (in this case LowPower.sleep())

Fixes #54

@fpistm
Copy link
Member

fpistm commented Apr 6, 2021

@ABOSTM
I think the README.md should be updated to describe this.

Last parameter of attachInterruptWakeup(), aka "LowPowerMode",
should match lowpower state used (in this case LowPower.sleep())

Fixes stm32duino#54

Signed-off-by: Alexandre Bourdiol <alexandre.bourdiol@st.com>
@ABOSTM ABOSTM force-pushed the EXTERNALWAKEUP_LOWPOWERMODE_SLEEP branch from bbf1549 to 3142e3c Compare April 6, 2021 13:24
@ABOSTM
Copy link
Contributor Author

ABOSTM commented Apr 6, 2021

@ABOSTM
I think the README.md should be updated to describe this.

Right. Done.

@fpistm
Copy link
Member

fpistm commented Apr 7, 2021

Note: I've updated the GH action compile example to fix build issue while STM32 core 2.0.0 is not released.

@fpistm fpistm merged commit 15c56c0 into stm32duino:master Apr 7, 2021
@fpistm fpistm added this to the 1.1.0 milestone Apr 7, 2021
@fpistm fpistm added fix 🩹 Bug fix and removed bug labels Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix 🩹 Bug fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ExternalWakeup - add deepSleep() example - leak of current
2 participants