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

OS Pwr Manager doesn't put nrf52 into LPS_1 #12025

Closed
bearsh opened this issue Dec 11, 2018 · 7 comments
Closed

OS Pwr Manager doesn't put nrf52 into LPS_1 #12025

bearsh opened this issue Dec 11, 2018 · 7 comments
Assignees
Labels
area: Power Management bug The issue is a bug, or the PR is fixing a bug platform: nRF Nordic nRFx priority: low Low impact/importance bug
Milestone

Comments

@bearsh
Copy link
Contributor

bearsh commented Dec 11, 2018

Describe the bug
Using a very simple app (while (1) {k_sleep(MSEC_PER_SEC);} in main thread only) with os pwr manager enabled, the lowest current consumption I get is around 700uA, which looks more like LPS_0 than LPS_1

Expected behavior
A sleeping SoC for almost 1s with an expected current consumption of around ~5uA rather than 700uA

Additional context
Running the pwr manager demo, the systems sometimes (not reliable) enters LPS_1.

@bearsh bearsh added the bug The issue is a bug, or the PR is fixing a bug label Dec 11, 2018
@carlescufi carlescufi added priority: medium Medium impact/importance bug area: Power Management platform: nRF Nordic nRFx labels Dec 12, 2018
@tautologyclub
Copy link
Contributor

If you read the code for the power management example it's pretty clear this isn't a bug. You have to implement the suspend hook yourself. Suggest closing.

@bearsh
Copy link
Contributor Author

bearsh commented Jan 25, 2019

@tautologyclub do you mean pwer_mgr? I can't find any reference to a hook I have to implement. Just to make it clear, this issue is about 'OS Power Management' which should enter the appropriate sleep state 'by itself', see also power.c

@pizi-nordic
Copy link
Collaborator

There are no low power states in nRF5x SoCs. The https://github.com/zephyrproject-rtos/zephyr/blob/master/samples/subsys/power/power_mgr example introduces them artificially in order to show the OS-based policies.

There is no magic switch which reduces power consumption to XX uA in your design.
And there never will be one.

@bearsh
Copy link
Contributor Author

bearsh commented Mar 26, 2019

@pizi-nordic ok, let's not name them LPS_x (as I just notived, commit f04a4c9 removed that naming). but without any magic switch, running the simple application mentioned in the description and have the pm handled by the kernel (logging etc. disabled) shouldn't end up in a current consumption of 700uA.

@nashif nashif added priority: low Low impact/importance bug and removed priority: medium Medium impact/importance bug labels Apr 2, 2019
@carlescufi
Copy link
Member

@pizi-nordic @pabigot is this still a valid issue?

@pabigot
Copy link
Collaborator

pabigot commented Aug 26, 2019

IMO yes.

@pabigot pabigot added this to the v2.3.0 milestone Mar 13, 2020
@carlescufi carlescufi modified the milestones: v2.3.0, v2.4.0 Jun 5, 2020
@pabigot
Copy link
Collaborator

pabigot commented Aug 24, 2020

A year later I don't think this is worth keeping open. Currently controlling power is mostly manual, and the referenced sample has been replaced by samples/boards/system_off which shows (crudely) how to do it for Nordic. At some point this will be done automatically, but it's an ongoing process being tracked elsewhere (e.g. entry point at #24228).

@pabigot pabigot closed this as completed Aug 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Power Management bug The issue is a bug, or the PR is fixing a bug platform: nRF Nordic nRFx priority: low Low impact/importance bug
Projects
None yet
Development

No branches or pull requests

6 participants