Skip to content

allow rtos idle thread to call sleep(); #2443

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

Closed
wants to merge 2 commits into from
Closed

Conversation

maclobdell
Copy link
Contributor

@maclobdell maclobdell commented Aug 15, 2016

To enable power efficiency, the rtos thread should call sleep(). To my knowledge, this can be done regardless if the debugger interface is connected (at least it can on the Kinetis MCUs). If the debugger interface is connected, the device may not achieve as great of power savings, but it still can enter low power mode. Something is better than nothing.

To enable power efficiency, the rtos loop should call sleep().  To my knowledge, this can be done regardless if the debugger interface is connected (at least it can on the Kinetis MCUs).  If the debugger interface is connected, the device may not achieve as great of power savings, but it still can enter low power mode.  Something is better than nothing.
@maclobdell maclobdell changed the title allow rtos idle loop to call sleep(); allow rtos idle thread to call sleep(); Aug 15, 2016
@0xc0170
Copy link
Contributor

0xc0170 commented Aug 15, 2016

@maclobdell Did you test this on the lpc boards that have local filesystem? There's relevant issue here: #143

cc @Sissors @neilt6 @bogdanm

@bogdanm
Copy link
Contributor

bogdanm commented Aug 16, 2016

Cc @bulislaw (he's working on something related to this).

@bulislaw
Copy link
Member

I'm working on a tickless mode, where we'll stop the SysTick, set wakeup timer and then call sleep.

@0xc0170
Copy link
Contributor

0xc0170 commented Aug 16, 2016

I cant make lpc11u24 neither lpc1768 debugging work (uvision, iar, makefile). Anyone to test for those platforms this patch?

@bogdanm
Copy link
Contributor

bogdanm commented Aug 16, 2016

I'm working on a tickless mode, where we'll stop the SysTick, set wakeup timer and then call sleep.

I believe this will eventually be a better solution, since tickless mode allows the OS to keep track of the time even while the MCU is asleep.

@sg-
Copy link
Contributor

sg- commented Aug 26, 2016

Closing in favor of #2547

@maclobdell please make sure sleep() gets enabled by default for the non-tickless mode.

@sg- sg- closed this Aug 26, 2016
@sg- sg- removed the needs: review label Aug 26, 2016
@maclobdell
Copy link
Contributor Author

re-opening because #2547 is postponed.

@maclobdell maclobdell reopened this Oct 4, 2016
@sg- sg- added the needs: work label Oct 5, 2016
@sg-
Copy link
Contributor

sg- commented Oct 7, 2016

@maclobdell can you resolve the conflict?

@sg-
Copy link
Contributor

sg- commented Oct 10, 2016

@maclobdell github is still not happy :)

@maclobdell
Copy link
Contributor Author

created new pull request #3019 to replace this one. The change is identical. Trying to avoid potential git issue with this PR.

@maclobdell maclobdell closed this Oct 13, 2016
@sg- sg- deleted the maclobdell-patch-1 branch October 19, 2016 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants