Skip to content

Conversation

@OverkillFPV
Copy link

Added in an RSSI offset compensation which will reduce the reported noise floor and rssi values for nodes with an LNA, e.g. Heltec V4

…is will correct the noise floor and recieved rssi values
Copilot AI review requested due to automatic review settings January 14, 2026 13:27
Copy link

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 pull request adds RSSI (Received Signal Strength Indicator) offset compensation to account for external LNA (Low Noise Amplifier) gain on devices like the Heltec V4 which uses a GC1109 LNA with 17dB gain. The compensation adjusts reported RSSI and noise floor values to reflect true signal strength.

Changes:

  • Added RSSI_OFFSET build flag (-17 dB) in Heltec V4 platformio configuration
  • Implemented RSSI_OFFSET macro with default value of 0 in CustomSX1262Wrapper.h
  • Applied offset compensation to getCurrentRSSI() and getLastRSSI() methods

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
variants/heltec_v4/platformio.ini Defines RSSI_OFFSET=-17 for GC1109 LNA gain compensation specific to Heltec V4 hardware
src/helpers/radiolib/CustomSX1262Wrapper.h Adds RSSI_OFFSET macro and applies compensation to RSSI getter methods

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link

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

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


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

1 participant