Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Class C tested for EU_868 #32

Merged
merged 7 commits into from
Mar 17, 2020
Merged

Class C tested for EU_868 #32

merged 7 commits into from
Mar 17, 2020

Conversation

novvere
Copy link
Contributor

@novvere novvere commented Mar 10, 2020

Low datarate optimization on AGC auto on for DR11 and DR12, otherwise CRC_ERROR is detected by RFM in CLASS C continuos mode.
Other minor changes.

@novvere
Copy link
Contributor Author

novvere commented Mar 10, 2020

Tested with ChirpStack, TTN Madrid V7 board with ABP and OTAA. As this board is still in development I will try testing with V2 board based on Arduino Nano 328P

Copy link
Member

@sabas1080 sabas1080 left a comment

Choose a reason for hiding this comment

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

Wow, good look for me. @wero1414

@novvere
Copy link
Contributor Author

novvere commented Mar 13, 2020

Tested in ATMega 328P using TTN MAdrid v2 node (https://github.com/IoTopenTech/Nodo_TTN_MAD_V2_2)
On ARM micros just using 'const' puts constants in Flash memory but AVR needs PROGMEM macro. I added PROGMEM to be compatible with both microprocessors.

This was referenced Mar 17, 2020
@novvere
Copy link
Contributor Author

novvere commented Mar 17, 2020

Some of the problems I found testing Class C where related to the regional parameters, frequencies and data rates even working with TTN. I can only test with EU_868 so, if something goes wrong in other regions, activate DEBUG and check values sent to FRM just before going continuous receive to see if it meets correct freq and data rate.

@sabas1080 sabas1080 self-requested a review March 17, 2020 18:27
@sabas1080 sabas1080 merged commit 9a6f039 into ElectronicCats:master Mar 17, 2020
@mattjbailey
Copy link

I have not seen a good example of a working class C code yet. The default settings for this code are US915, yet no working examples for this band. Am I missing something? I have searched and searched with no luck.

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.

4 participants