LLCC68 SF can't be larger than 9

jgromes / RadioLib

Universal wireless communication library for embedded devices

https://jgromes.github.io/RadioLib/

MIT License

1.48k stars 373 forks source link

LLCC68 SF can't be larger than 9 #315

Closed xixixihaha closed 3 years ago

xixixihaha commented 3 years ago

According to the spec, LLCC68 support SF5~9 @ 125K BW, SF5~10 @250K BW, SF5~11 @ 500K BW When I was playing, I found it always failed with code -20 if SF >9 when it was doing initialization. Looks like there are still differences between SX1268 and LLCC68.

Could you help to check? Thanks!

Additional info (please complete):

MCU: ESP32
Wireless module type LLCC68
VScode + PlatformIO
Library version 4.4.2

jgromes commented 3 years ago

Indeed, looks like it won't be enough to just keep it as alias. I added a full class implementation for LLCC68, so now it should check bandwidth and spreading factor properly.

mrx23dot commented 3 months ago

The only diff in datasheet between the two is one line: Not all SF are available for any bandwidth with the LLCC68

But it doesn't say which combination is valid. Any idea?

LLCC68 868 is easier to buy than SX1262 868 on ali

jgromes commented 3 months ago

@mrx23dot I would suggest seaching the datasheet again: it does say which combinations are valid.

EDIT: I would also like to point out that the LLCC68 has some issues for LoRaWAN precisely because of this limited selection of SFs, so if that's your use case, just buy an SX1268 and save yourself the hassle.

Screenshot_96

mrx23dot commented 3 months ago

Oh yeah I missed that one in diff, thanks! That means it's unsuitable for default preset in Meshstastic.