lora: Multiple small fixes (including syncword, sx126x minimum config)

projectgus commented 7 months ago

Collection of small LoRa fixes in individual commits. See commit messages for more details.

SX126x: Fix invalid configuration after reset. Configuring with the documented minimal lora_cfg didn't produce the documented results, some configuration keys always needed to be set.
SX126x: Fix syncword setting. Closes #796.
SX127x: Fix syncword setting. Implementation was simply missing. :facepalm:
lora-sync: Fix a possible race where a send which failed immediately (or completed very quickly) would not return immediately.
SX126x: Cosmetic clean up of some struct formatting.

Test on SX1276, SX1262, STM32WL55.

projectgus commented 7 months ago

Build failure looks to be an unrelated regression from https://github.com/micropython/micropython/pull/13620

projectgus commented 7 months ago

@oeysteinhansen If you have the opportunity, then it's possible to install the packages from this PR and verify the fix by running mpremote mip install --index https://projectgus.github.io/micropython-lib/mip/bugfix/lora_sx126x lora-sx126x

dpgeorge commented 7 months ago

Thanks, merged with relevant manifest.py versions bumped at patch level.

projectgus commented 7 months ago

@dpgeorge Oops, good catch - thanks!

micropython / micropython-lib

lora: Multiple small fixes (including syncword, sx126x minimum config) #804