Skip to content

Conversation

@entr0p1
Copy link

@entr0p1 entr0p1 commented Dec 30, 2025

Several fixes/improvements for SX1262 radios:

  1. Implemented radio resets for some TX and RX fault conditions. Radio will reset, then re-init and go back to receive state. Fault codes that will trigger reset:
    RADIOLIB_ERR_TX_TIMEOUT
    RADIOLIB_ERR_SPI_CMD_FAILED
    RADIOLIB_ERR_SPI_CMD_INVALID
    RADIOLIB_ERR_SPI_WRITE_FAILED
    RADIOLIB_ERR_SPI_READ_FAILED
    RADIOLIB_ERR_CHIP_NOT_FOUND
    RADIOLIB_ERR_DEVICE_BUSY

On SX1262-based variants without the P_LORA_RESET pin configured, this should NO-OP.

  1. Added SX1262 DC-DC enable code (separate to the board's DC-DC) and platform.ini flag to enable it on supported nodes. Enabled on RAK4631 as this is confirmed supported.

Copy link
Contributor

@fschrempf fschrempf left a comment

Choose a reason for hiding this comment

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

Great work! Here are some things I've noticed.

And maybe you could clean up the commits a little bit so there is one commit per logical change. You can just force-push if you cleaned up the history.

@entr0p1
Copy link
Author

entr0p1 commented Dec 30, 2025

Great work! Here are some things I've noticed.

Thanks heaps for your time reviewing this! Great points all around and I'll respond/fix accordingly.

And maybe you could clean up the commits a little bit so there is one commit per logical change. You can just force-push if you cleaned up the history.

Good call - I seem to have a concrete skull when it comes to learning how Git works haha :) will fix that up too, cheers

@entr0p1 entr0p1 force-pushed the sx1262-improvements branch 4 times, most recently from 33856bc to 7401417 Compare December 31, 2025 00:41
@entr0p1 entr0p1 force-pushed the sx1262-improvements branch from 7401417 to 25bbb0b Compare December 31, 2025 00:43
@entr0p1
Copy link
Author

entr0p1 commented Dec 31, 2025

@fschrempf would you mind taking another look for me now? All your comments should be resolved.

Copy link
Contributor

@fschrempf fschrempf left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@entr0p1 entr0p1 changed the title Sx1262 Improvements - LoRa Radio reset and DC-DC regulator Sx1262 Improvements - LoRa Radio reset Dec 31, 2025
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