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

hw/mcu/dialog: Add callbacks on enter/exit sleep #2297

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

andrzej-kaczmarek
Copy link
Contributor

This patch adds callbacks to be called before and after deep sleep.
Those are intended to be registered by BSP which can then decide to
save/restore state of certain peripherals and release/acquire power
domains in order to save some power while M33 goes into deep sleep.

Note that similar power saving can usually be achieved if application
uses os_dev_open/close consistently, i.e. opens device only when
needed, which should properly acquire and release power domains
(although the actual behavior depends on a driver implementation).
However, there may still be some cases where quick save/restore is
possible and this can be handled in new callbacks.

This patch adds callbacks to be called before and after deep sleep.
Those are intended to be registered by BSP which can then decide to
save/restore state of certain peripherals and release/acquire power
domains in order to save some power while M33 goes into deep sleep.

Note that similar power saving can usually be achieved if application
uses os_dev_open/close consistently, i.e. opens device only when
needed, which should properly acquire and release power domains
(although the actual behavior depends on a driver implementation).
However, there may still be some cases where quick save/restore is
possible and this can be handled in new callbacks.
@apache-mynewt-bot
Copy link

Style check summary

No suggestions at this time!

Copy link
Contributor

@kasjer kasjer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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.

3 participants