Add T-Beam BPF (144-148 Mhz LoRa)#9703
Merged
Merged
Conversation
Member
|
EU (ITU Region 1) 2m Band is 144-146MHz - Can we either make 2 presets or (assuming a HAM knows what they are doing) make a tighter default preset and tell people to deliberately set a custom frequency if they wanna go outside? This way an EU operator may not unknowingly exceed their boundaries. |
Contributor
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Adds initial support for the LilyGo T-Beam BPF (144–148 MHz LoRa) ESP32-S3 variant, including board/variant definitions and new ham 2m region handling to keep region selection consistent with the device’s RF front-end.
Changes:
- Added new ESP32-S3 board + variant definitions for T-Beam BPF (pins, build env, board JSON).
- Introduced ham 2m region profiles (ITU1_2M, ITU23_2M) and enforced hardware/region compatibility checks.
- Added AXP2101 rail initialization for TBEAM_BPF and RF95 power-enable handling.
Reviewed changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| variants/esp32s3/t-beam-bpf/variant.h | Defines T-Beam BPF hardware wiring (GPS, OLED, LoRa, SD, PMU) and ham-only flag. |
| variants/esp32s3/t-beam-bpf/platformio.ini | Adds PlatformIO environment for building the new T-Beam BPF target. |
| variants/esp32s3/t-beam-bpf/pins_arduino.h | Defines Arduino core pin mappings (UART/I2C/SPI/SD). |
| boards/t-beam-bpf.json | Adds PlatformIO board definition for the new hardware. |
| src/platform/esp32/architecture.h | Maps T_BEAM_BPF define to protobuf hardware model enum. |
| src/mesh/RadioInterface.cpp | Adds ham 2m regions/profile and enforces region ↔ hardware compatibility. |
| src/mesh/RF95Interface.cpp | Adds RF95 power-enable toggling during init/sleep. |
| src/mesh/NodeDB.cpp | Sets default licensed-mode for ham-only hardware on first install. |
| src/mesh/MeshRadio.h | Exposes PROFILE_HAM. |
| src/graphics/draw/MenuHandler.cpp | Limits region picker options on ham-only hardware. |
| src/Power.cpp | Adds AXP2101 rail enablement for GNSS + MicroSD on TBEAM_BPF. |
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Member
Author
|
Blocked by #10120 (enabling frequency overrides, narrow presets) |
This is an appropriate default in the USA but not the EU. The slot override really should follow the region itself, not the regionprofile.
Member
Author
|
Depends on protobuf regen that includes: |
Member
Author
|
Moving this to a branch in-org |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add LilyGo T-Beam BPF, the first / only 144-148Mhz LoRa device 🍖
Requires: