Skip to content

stm32f4discovery long-term deep sleep#3

Open
southernbear wants to merge 8 commits into
OlegHahm:southernbear-masterfrom
southernbear:longterm-sleep
Open

stm32f4discovery long-term deep sleep#3
southernbear wants to merge 8 commits into
OlegHahm:southernbear-masterfrom
southernbear:longterm-sleep

Conversation

@southernbear

Copy link
Copy Markdown

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

OlegHahm commented Feb 5, 2014

Copy link
Copy Markdown
Owner

@haukepetersen, could look at this?

@ghost

ghost commented Feb 20, 2014

Copy link
Copy Markdown

Merged build triggered.

@ghost

ghost commented Feb 20, 2014

Copy link
Copy Markdown

Merged build started.

@ghost

ghost commented Feb 20, 2014

Copy link
Copy Markdown

Merged build finished.

@haukepetersen

Copy link
Copy Markdown
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
Copy Markdown
Contributor

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

@haukepetersen

Copy link
Copy Markdown
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

ghost commented May 16, 2014

Copy link
Copy Markdown

Can anyone ask me to verify this patch?

@miri64

miri64 commented May 17, 2014

Copy link
Copy Markdown
Contributor

@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