Skip to content

Conversation

@jbrazio
Copy link
Contributor

@jbrazio jbrazio commented Nov 30, 2025

This pull request adds support for configuring and persisting radio current limit and RX boosted gain settings for SX12xx series radios (SX1262, SX1268, SX1276) across the codebase. These settings are now stored in preferences, can be set and queried via CLI commands, and are applied during radio initialization.

@fschrempf
Copy link
Contributor

Instead of duplicating the setters and getters for each target, can we directly call the functions from the app code using the global radio_driver variable, similar to what was done here: 07e7e2d?

@fschrempf
Copy link
Contributor

Can you explain why this is needed and what advantages these changes have? In what cases do these parameters need to be changed?

@jbrazio
Copy link
Contributor Author

jbrazio commented Nov 30, 2025

duplicating the setters and getters for each target

Didn't want to change the existing architecture: radio_set_params, radio_set_tx_power and radio_new_identity are all duplicated. If @ripplebiz agrees with it, I will refactor.

why this is needed and what advantages these changes have

Are you serious on this one ?

@fschrempf
Copy link
Contributor

If @ripplebiz agrees with it, I will refactor.

Got it, thanks!

Are you serious on this one ?

Absolutely. Why does this need to be a user-facing parameter? Why is a hardcoded value per variant not enough? Sorry for my lack of knowledge here. I'm not an expert, just interested in what this is about.

@jbrazio
Copy link
Contributor Author

jbrazio commented Nov 30, 2025

I would call it more "sysadmin facing" as the main target is repeaters: You might hang a repeater [with a proper antenna] in a difficult to access place to find out that there is a lot of QRM and RX is deaf; most of the situations where I faced this, disabling the Boosted Gain helped a lot with the problem.

When we use small out-of-tune antennas (the ones shipping with the boards) having a boost on the RX helps, this is usually the client situation. For repeater where we usually have proper antennas it's a question of testing with it ON and OFF and check what gives you better performance.

For the OCP there is no big use case but I had a couple of situations where tweaking the OCP helped with random resets of the radio.

@fschrempf
Copy link
Contributor

Thanks for the detailed explanations. Makes sense to me. I didn't know that disabling the RX boost can be benefitial for repeaters with proper antennas.

@fdlamotte
Copy link
Collaborator

These enhancements are good (should be done for all radios ...)

but this is really up to @ripplebiz ;)

1 similar comment
@fdlamotte
Copy link
Collaborator

These enhancements are good (should be done for all radios ...)

but this is really up to @ripplebiz ;)

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.

3 participants