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

cpu/nrf5x/timer: fix high current consumption in powered off state #18953

Merged
merged 1 commit into from
Nov 23, 2022

Conversation

jue89
Copy link
Contributor

@jue89 jue89 commented Nov 22, 2022

Contribution description

While testing #17607 I realized that the timer peripheral consumes significant amounts of current even in stopped state. I had a look into the implementation of our Zephyr friends and found this Errata PDF.

Testing procedure

TBD

Issues/PRs references

This is a workaround for errata 78 that causes increased current consumption even in the stopped state.
@jue89 jue89 added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Area: cpu Area: CPU/MCU ports labels Nov 22, 2022
@github-actions github-actions bot added the Platform: ARM Platform: This PR/issue effects ARM-based platforms label Nov 22, 2022
@aabadie aabadie added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Nov 23, 2022
@riot-ci
Copy link

riot-ci commented Nov 23, 2022

Murdock results

✔️ PASSED

a7e2182 cpu/nrf5x/timer: run task SHUTDOWN instead of STOP

Success Failures Total Runtime
117849 0 117849 01h:55m:05s

Artifacts

@maribu maribu merged commit cefef8f into RIOT-OS:master Nov 23, 2022
@jue89 jue89 deleted the fix/nrf5x-timer-lowpower branch November 23, 2022 16:39
@jue89
Copy link
Contributor Author

jue89 commented Nov 23, 2022

Thank you :)

@kaspar030 kaspar030 added this to the Release 2023.01 milestone Jan 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: cpu Area: CPU/MCU ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: ARM Platform: This PR/issue effects ARM-based platforms Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants