Skip to content

Heltec V3 screen/LED controls and display#1585

Open
Koreanman01 wants to merge 1 commit intomeshcore-dev:mainfrom
Koreanman01:heltec-v3-screen-led
Open

Heltec V3 screen/LED controls and display#1585
Koreanman01 wants to merge 1 commit intomeshcore-dev:mainfrom
Koreanman01:heltec-v3-screen-led

Conversation

@Koreanman01
Copy link

Summary

  • Heltec V3‑only screen/LED controls with persistence, UI, and CLI actions.
  • Heltec V3‑specific SSD1306 display driver with brightness control.
  • Heltec V3 overlay examples to avoid shared example edits.
  • VEXT polarity fix for correct brightness.
  • Persistence uses ESP32 NVS (no Preferences dependency).

Details

  • Adds board settings for screen/LED/brightness and applies them at runtime.
  • LED/brightness persist across reboots; screen defaults to on by design.
  • Introduces a Heltec V3 display class (contrast/precharge/VCOM handling).
  • Updates Heltec V3 build flags and overlay example selection.

Testing

  • Built and uploaded Heltec_v3_companion_radio_usb
  • Built and uploaded Heltec_v3_companion_radio_ble
  • Built and uploaded Heltec_v3_repeater

Copy link
Contributor

@oltaco oltaco left a comment

Choose a reason for hiding this comment

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

Thanks for this, brightness control will be a great addition. In future please base all PRs against dev branch, PRs against main branch will not be accepted.

I really think this should be implemented in the main SSD1306 display driver so that all devices with SSD1306 can benefit rather than copying the firmwares into the heltec v3 variant folder and adding the feature there.

It's also probably not appropriate to use EnvironmentSensorManager's getNumSettings(), getSettingName(), getSettingValue(), and setSettingValue() methods for setting the display brightness, I think it would require new CMD_GET/SET and PUSH_ codes for companion firmwares and could be integrated with settings in the on device UI.

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