jgromes / RadioLib

Universal wireless communication library for embedded devices
https://jgromes.github.io/RadioLib/
MIT License
1.56k stars 389 forks source link

v5.7 bugs #699

Closed EliasA97 closed 1 year ago

EliasA97 commented 1 year ago

After v.5.7 the ebyte sx1262 chips aren't identifying (-2 error) and code takes much more space. I'm switching back to 5.6 which has worked fine.

jgromes commented 1 year ago

This is not very helpful.

  1. If you have issues with some chips, post the debug output. It is possible they are being misidentified, or they are nto actually what you think they are.

  2. What platform are you using? How significant is the increase? Again, you gave me almost no information here ...

EliasA97 commented 1 year ago

Sorry my bad...

I use both arduino uno and mega as transceivers with ebyte E22 400m30s.

The uno specifically used about 89% of program space. After the update it got up to 99%. That raised a flag. When I updated to 5.7 yesterday suddenly the lora wasn't recognised in both boards. That got me thinking it may be a bug which explains the memory increase(?), so I downgraded to 5.6 and everything continue working.

Did you changed anything command-wise?

If not I'll post you later a debug output.

Thank's

jgromes commented 1 year ago

In 5.7 a check was introduced to make sure the module you have connected is actually what it claims to be. This is displayed in the debug output, so please post it.

The report template exists for a reason ;)

EliasA97 commented 1 year ago

Sorry Jan, I'm in a bit of a rush and my mind is slipping away.

This is the output (verbose and debug) of your SX126x-Settings example (v5.7)


[SX1262] Initializing ... CMD   80  
DATW    0   AA  

CMD 80  
DATW    0   A2  

CMD C0  
DATR    0   22  0   22  

CMD 1D  3   20  
DATR    0   A2  0   53  0   58  0   31  0   32  0   36  0   38  0   20  0   56  0   32  0   46  0   20  0   32  0   46  0   30  0   32  0   0   

SX126x not found! (1 of 10 tries) RADIOLIB_SX126X_REG_VERSION_STRING:
0000320 53 58 31 32 36 38 20 56 32 46 20 32 46 30 32 00 | SX1268 V2F 2F02.
Expected string: SX1261
CMD 80  
DATW    0   AA  

CMD 80  
DATW    0   A2  

CMD C0  
DATR    0   22  0   22  

CMD 1D  3   20  
DATR    0   A2  0   53  0   58  0   31  0   32  0   36  0   38  0   20  0   56  0   32  0   46  0   20  0   32  0   46  0   30  0   32  0   0   

SX126x not found! (2 of 10 tries) RADIOLIB_SX126X_REG_VERSION_STRING:
0000320 53 58 31 32 36 38 20 56 32 46 20 32 46 30 32 00 | SX1268 V2F 2F02.
Expected string: SX1261
CMD 80  
DATW    0   AA  

CMD 80  
DATW    0   A2  

CMD C0  
DATR    0   22  0   22  

CMD 1D  3   20  
DATR    0   A2  0   53  0   58  0   31  0   32  0   36  0   38  0   20  0   56  0   32  0   46  0   20  0   32  0   46  0   30  0   32  0   0   

SX126x not found! (3 of 10 tries) RADIOLIB_SX126X_REG_VERSION_STRING:
0000320 53 58 31 32 36 38 20 56 32 46 20 32 46 30 32 00 | SX1268 V2F 2F02.
Expected string: SX1261
CMD 80  
DATW    0   AA  

CMD 80  
DATW    0   A2  

CMD C0  
DATR    0   22  0   22  

CMD 1D  3   20  
DATR    0   A2  0   53  0   58  0   31  0   32  0   36  0   38  0   20  0   56  0   32  0   46  0   20  0   32  0   46  0   30  0   32  0   0   

SX126x not found! (4 of 10 tries) RADIOLIB_SX126X_REG_VERSION_STRING:
0000320 53 58 31 32 36 38 20 56 32 46 20 32 46 30 32 00 | SX1268 V2F 2F02.
Expected string: SX1261
CMD 80  
DATW    0   AA  

CMD 80  
DATW    0   A2  

CMD C0  
DATR    0   22  0   22  

CMD 1D  3   20  
DATR    0   A2  0   53  0   58  0   31  0   32  0   36  0   38  0   20  0   56  0   32  0   46  0   20  0   32  0   46  0   30  0   32  0   0   

SX126x not found! (5 of 10 tries) RADIOLIB_SX126X_REG_VERSION_STRING:
0000320 53 58 31 32 36 38 20 56 32 46 20 32 46 30 32 00 | SX1268 V2F 2F02.
Expected string: SX1261
CMD 80  
DATW    0   AA  

CMD 80  
DATW    0   A2  

CMD C0  
DATR    0   22  0   22  

CMD 1D  3   20  
DATR    0   A2  0   53  0   58  0   31  0   32  0   36  0   38  0   20  0   56  0   32  0   46  0   20  0   32  0   46  0   30  0   32  0   0   

SX126x not found! (6 of 10 tries) RADIOLIB_SX126X_REG_VERSION_STRING:
0000320 53 58 31 32 36 38 20 56 32 46 20 32 46 30 32 00 | SX1268 V2F 2F02.
Expected string: SX1261
CMD 80  
DATW    0   AA  

CMD 80  
DATW    0   A2  

CMD C0  
DATR    0   22  0   22  

CMD 1D  3   20  
DATR    0   A2  0   53  0   58  0   31  0   32  0   36  0   38  0   20  0   56  0   32  0   46  0   20  0   32  0   46  0   30  0   32  0   0   

SX126x not found! (7 of 10 tries) RADIOLIB_SX126X_REG_VERSION_STRING:
0000320 53 58 31 32 36 38 20 56 32 46 20 32 46 30 32 00 | SX1268 V2F 2F02.
Expected string: SX1261
CMD 80  
DATW    0   AA  

CMD 80  
DATW    0   A2  

CMD C0  
DATR    0   22  0   22  

CMD 1D  3   20  
DATR    0   A2  0   53  0   58  0   31  0   32  0   36  0   38  0   20  0   56  0   32  0   46  0   20  0   32  0   46  0   30  0   32  0   0   

SX126x not found! (8 of 10 tries) RADIOLIB_SX126X_REG_VERSION_STRING:
0000320 53 58 31 32 36 38 20 56 32 46 20 32 46 30 32 00 | SX1268 V2F 2F02.
Expected string: SX1261
CMD 80  
DATW    0   AA  

CMD 80  
DATW    0   A2  

CMD C0  
DATR    0   22  0   22  

CMD 1D  3   20  
DATR    0   A2  0   53  0   58  0   31  0   32  0   36  0   38  0   20  0   56  0   32  0   46  0   20  0   32  0   46  0   30  0   32  0   0   

SX126x not found! (9 of 10 tries) RADIOLIB_SX126X_REG_VERSION_STRING:
0000320 53 58 31 32 36 38 20 56 32 46 20 32 46 30 32 00 | SX1268 V2F 2F02.
Expected string: SX1261
CMD 80  
DATW    0   AA  

CMD 80  
DATW    0   A2  

CMD C0  
DATR    0   22  0   22  

CMD 1D  3   20  
DATR    0   A2  0   53  0   58  0   31  0   32  0   36  0   38  0   20  0   56  0   32  0   46  0   20  0   32  0   46  0   30  0   32  0   0   

SX126x not found! (10 of 10 tries) RADIOLIB_SX126X_REG_VERSION_STRING:
0000320 53 58 31 32 36 38 20 56 32 46 20 32 46 30 32 00 | SX1268 V2F 2F02.
Expected string: SX1261
No SX126x found!
failed, code -2

This is the same example but with v.5.6


[SX1262] Initializing ... M SX126x
CMD 80  
DATW    0   AA  

CMD 80  
DATW    0   A2  

CMD C0  
DATR    0   22  0   22  

CMD 80  
DATW    0   A2  

CMD C0  
DATR    0   22  0   22  

CMD 8F  
DATW    0   A2  0   A2  

CMD C0  
DATR    0   22  0   22  

CMD 8A  
DATW    1   A2  

CMD C0  
DATR    0   22  0   22  

CMD 93  
DATW    20  A2  

CMD C0  
DATR    0   22  0   22  

CMD 88  
DATW    3   A2  16  A2  A   A2  0   A2  0   A2  0   A2  0   A2  

CMD C0  
DATR    0   22  0   22  

CMD 2   
DATW    43  A2  FF  A2  

CMD C0  
DATR    0   22  0   22  

CMD 8   
DATW    0   A2  0   A2  0   A2  0   A2  0   A2  0   A2  0   A2  0   A2  

CMD C0  
DATR    0   22  0   22  

CMD 89  
DATW    7F  A2  

CMD 80  
DATW    0   AA  

CMD 17  
DATR    0   A2  0   0   0   20  

CMD C0  
DATR    0   22  0   22  

CMD 97  
DATW    0   A2  0   A2  1   A2  40  A2  

CMD C0  
DATR    0   22  0   22  

CMD 11  
DATR    0   A2  0   1   

CMD C0  
DATR    0   22  0   22  

Symbol length: 1.02 ms
CMD 8B  
DATW    9   A2  6   A2  3   A2  0   A2  

CMD C0  
DATR    0   22  0   22  

CMD 11  
DATR    0   A2  0   1   

CMD C0  
DATR    0   22  0   22  

CMD D   7   40  
DATW    14  A2  24  A2  

CMD C0  
DATR    0   22  0   22  

CMD 11  
DATR    0   A2  0   1   

CMD C0  
DATR    0   22  0   22  

CMD 1D  7   36  
DATR    0   A2  0   D   

CMD C0  
DATR    0   22  0   22  

CMD D   7   36  
DATW    D   A2  

CMD C0  
DATR    0   22  0   22  

CMD 8C  
DATW    0   A2  8   A2  0   A2  FF  A2  1   A2  0   A2  

CMD C0  
DATR    0   22  0   22  

CMD D   8   E7  
DATW    18  A2  

CMD C0  
DATR    0   22  0   22  

CMD 9D  
DATW    1   A2  

CMD C0  
DATR    0   22  0   22  

CMD 96  
DATW    1   A2  

CMD C0  
DATR    0   22  0   22  

CMD 11  
DATR    0   A2  0   1   

CMD C0  
DATR    0   22  0   22  

Symbol length: 1.02 ms
CMD 8B  
DATW    9   A2  6   A2  3   A2  0   A2  

CMD C0  
DATR    0   22  0   22  

CMD 11  
DATR    0   A2  0   1   

CMD C0  
DATR    0   22  0   22  

Symbol length: 4.10 ms
CMD 8B  
DATW    9   A2  4   A2  3   A2  0   A2  

CMD C0  
DATR    0   22  0   22  

CMD 98  
DATW    6B  A2  6F  A2  

CMD C0  
DATR    0   22  0   22  

CMD 86  
DATW    1B  A2  20  A2  0   A2  0   A2  

CMD C0  
DATR    0   22  0   22  

CMD 1D  8   D8  
DATR    0   A2  0   E8  

CMD C0  
DATR    0   22  0   22  

CMD D   8   D8  
DATW    FE  A2  

CMD C0  
DATR    0   22  0   22  

CMD 1D  8   E7  
DATR    0   A2  0   18  

CMD C0  
DATR    0   22  0   22  

CMD 95  
DATW    4   A2  7   A2  0   A2  1   A2  

CMD C0  
DATR    0   22  0   22  

CMD 8E  
DATW    A   A2  4   A2  

CMD C0  
DATR    0   22  0   22  

CMD D   8   E7  
DATW    18  A2  

CMD C0  
DATR    0   22  0   22  

success!
[SX1268] Initializing ... M SX126x
failed, code -705

Note that I only have SX1262 and I didn't erase the second instance.

Now, on arduino uno with v5.6 I get 54% used program space and 50% ram. With v5.7 I get 66% program and 53% ram. As you can see there's not a problem with my setup because it works fine when using v5.6

jgromes commented 1 year ago
SX126x not found! (5 of 10 tries) RADIOLIB_SX126X_REG_VERSION_STRING:
0000320 53 58 31 32 36 38 20 56 32 46 20 32 46 30 32 00 | SX1268 V2F 2F02.
Expected string: SX1261

The module clearly identifies itself as SX1268. The Ebyte page for E22 400M30S also states that there's an SX1268 inside it.

Note that I only have SX1262

You don't have an SX1262 - you have an SX1268. This check was introduced in 5.7.0, so before that there was nothing stoppping you from using an incorrect module type.

Regarding the increased memory footprint, I will check if there's something that can be done about it.

EliasA97 commented 1 year ago

That's odd. I thought I had a SX1262 version all that time. Changing SX1262 to SX1268 seems to fix the problem.