Skip to content

Conversation

Quency-D
Copy link
Contributor

  1. Use the same PA chip as heltec_v4.

🤝 Attestations

  • I have tested that my proposed changes behave as described.
  • I have tested that my proposed changes do not cause any obvious regressions on the following devices:
    • Heltec (Lora32) V4
    • Heltec Tracker V2

@fifieldt fifieldt added the enhancement New feature or request label Sep 29, 2025
@fifieldt fifieldt force-pushed the dev-heltec-tracker-v2 branch from cdbee89 to 8d323a1 Compare September 29, 2025 22:20
@fifieldt
Copy link
Member

@Quency-D , I added a common definition for the power amp (USE_GC1109_PA), so in case there are more devices in the future that use it you can use it more easily. Hope that's OK, if not, please feel free to revert :)

@Quency-D
Copy link
Contributor Author

@Quency-D , I added a common definition for the power amp (USE_GC1109_PA), so in case there are more devices in the future that use it you can use it more easily. Hope that's OK, if not, please feel free to revert :)

Thanks, I did a simple test and it works fine. I'm doing more detailed testing.

@thebentern thebentern requested a review from Copilot October 2, 2025 10:52
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for the Heltec Wireless Tracker V2 board by creating a new board variant that shares the same GC1109 power amplifier configuration as the Heltec V4.

  • Creates complete board definition with pin mappings, display configuration, and LoRa settings
  • Refactors power amplifier code to use USE_GC1109_PA define instead of board-specific conditionals
  • Updates screen initialization logic to include the new tracker variant

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
variants/esp32s3/heltec_wireless_tracker_v2/variant.h Defines hardware pin mappings, display settings, and LoRa configuration for the new board
variants/esp32s3/heltec_wireless_tracker_v2/platformio.ini PlatformIO build configuration for the tracker v2 variant
variants/esp32s3/heltec_wireless_tracker_v2/pins_arduino.h Arduino-style pin definitions and constants for the board
variants/esp32s3/heltec_v4/variant.h Adds USE_GC1109_PA define to enable shared power amplifier code
src/sleep.cpp Updates conditional compilation to use USE_GC1109_PA instead of HELTEC_V4
src/platform/esp32/architecture.h Adds hardware model definition for the new tracker variant
src/mesh/SX126xInterface.cpp Refactors power amplifier control to use USE_GC1109_PA define
src/graphics/Screen.cpp Includes tracker v2 in screen initialization logic
src/configuration.h Updates power amplifier configuration to use USE_GC1109_PA
boards/heltec_wireless_tracker_v2.json Board definition file for PlatformIO with hardware specifications

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@thebentern thebentern added the hardware-support Add hardware support for new devices or modules label Oct 2, 2025
@thebentern thebentern merged commit 775595c into meshtastic:develop Oct 3, 2025
83 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request hardware-support Add hardware support for new devices or modules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants