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

Add status_led_off to support turning status LED off (dark mode) #485

Merged
merged 1 commit into from
Apr 20, 2024

Conversation

ndoo
Copy link
Contributor

@ndoo ndoo commented Apr 20, 2024

What does this PR do?

Add status_led_off to support turning status LED off (dark mode).

This only affects the GPIO-controllable LED and may be desirable behavior for Meshtastic users who sleep in the vicinity of their node and are annoyed by the blinking LED, especially for boards that indicate charging when USB power is connected, resulting in a 1 second LED illumination cycle.

Most boards however, have a non-GPIO controlled power LED (i.e. hardwired to VBUS/VCC/etc.) that can only be turned off by desoldering or otherwise removing that LED. This can't be addressed by firmware changes.

[Feature Request]: Option to turn off LED inducators on ESP32 Heltec V3

Checklist before merging

  • All top level messages commented
  • All enum members have unique descriptions

ndoo added a commit to ndoo/meshtastic-firmware that referenced this pull request Apr 20, 2024
Fixes meshtastic#3635 and depends on [protobufs PR meshtastic#485](meshtastic/protobufs#485)

Signed-off-by: Andrew Yong <me@ndoo.sg>
@garthvh
Copy link
Member

garthvh commented Apr 20, 2024

Since this is not going to actually enable dark mode what is the point?

ndoo added a commit to ndoo/meshtastic-firmware that referenced this pull request Apr 20, 2024
Fixes meshtastic#3635 and depends on [protobufs PR meshtastic#485](meshtastic/protobufs#485)

Signed-off-by: Andrew Yong <me@ndoo.sg>
@ndoo
Copy link
Contributor Author

ndoo commented Apr 20, 2024

Since this is not going to actually enable dark mode what is the point?

It does, for the LED that is actually controllable in firmware.

The point is that there are users who do not want a blinking LED at night.

@ndoo
Copy link
Contributor Author

ndoo commented Apr 20, 2024

I updated the PR text with a slightly more detailed explanation

@ndoo ndoo marked this pull request as ready for review April 20, 2024 08:27
@thebentern thebentern merged commit f4be94a into meshtastic:master Apr 20, 2024
1 of 2 checks passed
ndoo added a commit to ndoo/meshtastic-firmware that referenced this pull request Apr 20, 2024
Fixes meshtastic#3635 and depends on [protobufs PR meshtastic#485](meshtastic/protobufs#485)

Signed-off-by: Andrew Yong <me@ndoo.sg>
@ndoo ndoo deleted the status-led-off branch April 21, 2024 10:39
thebentern added a commit to meshtastic/firmware that referenced this pull request Apr 22, 2024
* Add ability to turn off status LED blinking

Fixes #3635 and depends on [protobufs PR #485](meshtastic/protobufs#485)

Signed-off-by: Andrew Yong <me@ndoo.sg>

* led_heartbeat_disabled

* trunk

---------

Signed-off-by: Andrew Yong <me@ndoo.sg>
Co-authored-by: Ben Meadors <benmmeadors@gmail.com>
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.

3 participants