Skip to content

[LR11x0] Expose instantaneous RSSI measurement#1714

Merged
jgromes merged 4 commits intojgromes:masterfrom
GUVWAF:LR11x0RSSI
Mar 1, 2026
Merged

[LR11x0] Expose instantaneous RSSI measurement#1714
jgromes merged 4 commits intojgromes:masterfrom
GUVWAF:LR11x0RSSI

Conversation

@GUVWAF
Copy link
Contributor

@GUVWAF GUVWAF commented Feb 26, 2026

The function getRssiInst() is used for the LR11x0 to get an instantaneous RSSI measurement, but it is protected. This exposes it by overloading getRSSI() with the bool packet parameter added just like is done for SX127x, SX126x, SX128x and LR2021.

I also rephrased Recorded Signal Strength Indicator to Received Signal Strength Indicator as I believe that's the more common term.

@jgromes
Copy link
Owner

jgromes commented Feb 28, 2026

Thanks for the contribution! In testing, does this return the some reasonable numbers? I am asking because on the other devices (most recently on LR2021), I had to make sure Rx mode was active, otherwise the return value was stuck at -128 dBm.

@GUVWAF
Copy link
Contributor Author

GUVWAF commented Feb 28, 2026

Yes, I get reasonable numbers, but indeed I just checked that's only when we're in receive mode. Do you want me to add the bool skipReceive parameter just like for LR2021 and let it go to receive mode if that value is false?

@jgromes
Copy link
Owner

jgromes commented Feb 28, 2026

I think that would be the best option, that way we will keep the same interface for all the LoRa modules.

@GUVWAF
Copy link
Contributor Author

GUVWAF commented Mar 1, 2026

Added in the latest commit, following the same approach as the LR2021.
I also slightly adjusted the description for this parameter for the LR2021.

@jgromes jgromes merged commit b0a3f2d into jgromes:master Mar 1, 2026
32 checks passed
@jgromes
Copy link
Owner

jgromes commented Mar 1, 2026

Looking good now - merged, thank you for the contribution!

@GUVWAF GUVWAF mentioned this pull request Mar 1, 2026
8 tasks
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