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

Reduce deep-sleep current for Heltec Wireless Paper #3545

Merged
merged 3 commits into from
Apr 4, 2024

Conversation

todd-herbert
Copy link
Contributor

@todd-herbert todd-herbert commented Apr 4, 2024

Addresses #3523

At present, deep-sleep current is 2.2mA @ 3.8V.
Changes lower the current draw to 18µA @ 3.8V.

These modifications are likely to be applicable to a number of ESP32 devices, but I don't have the hardware to test this.

Copy link
Member

@caveman99 caveman99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we test this PR on more devices before we accept it? This sounds like it's applicable to more than the wireless papers.

@todd-herbert
Copy link
Contributor Author

todd-herbert commented Apr 4, 2024

The code changes right now are intentionally limited to Wireless Paper, but I imagine many ESP32 / ESP32S3 devices could benefit here, if anyone with other hardware wants to check?

Ideally would need to check both:

  • ESP32-C3 (doesn't have SOC_RTCIO_HOLD_SUPPORTED)
  • Device which needs an internal pull-up on BUTTON_PIN

@caveman99 caveman99 merged commit f0b6ff9 into meshtastic:master Apr 4, 2024
69 checks passed
@todd-herbert todd-herbert deleted the paper-deepsleep-current branch April 4, 2024 09:48
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.

2 participants