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

stm32f4discovery long-term deep sleep #3

Open
wants to merge 8 commits into
base: southernbear-master
Choose a base branch
from

Conversation

southernbear
Copy link

Use deep sleep to save power consumption.

  • Enter Stop Mode when long-term sleeping(more than 1s) and wake up by RTC.
  • Provide automatic RTC calibration synchronized with TIM2.

@OlegHahm
Copy link
Owner

OlegHahm commented Feb 5, 2014

@haukepetersen, could look at this?

@ghost
Copy link

ghost commented Feb 20, 2014

Merged build triggered.

@ghost
Copy link

ghost commented Feb 20, 2014

Merged build started.

@ghost
Copy link

ghost commented Feb 20, 2014

Merged build finished.

@haukepetersen
Copy link
Contributor

  1. I dont quite understand the coupling of the RTC with TIM2, especially the part with searching for the nearest channel of TIM2
  2. The code formatting is not quite RIOT-style (way to many empty lines)
  3. Comments missing (page headers, doxygen) and some comments in the code could help

@thomaseichinger
Copy link
Contributor

@southernbear @haukepetersen
Regarding the future closing of this repository is this PR anyhow maintained?

@haukepetersen
Copy link
Contributor

Not sure - as I would be very interested in the functionality and it should be of course moved into the main repo as soon as the questions above are solved.

@ghost
Copy link

ghost commented May 16, 2014

Can anyone ask me to verify this patch?

@miri64
Copy link
Contributor

miri64 commented May 17, 2014

@RIOT-OS-Jenkins test this please

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.

5 participants