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

3 cases where the application is never awakened #10

Closed
wants to merge 3 commits into from

Conversation

dfrosch
Copy link

@dfrosch dfrosch commented Aug 5, 2015

The code freezes in 3 cases

  • bad address
  • bad MIC
  • bad frame
    Better after applying the patch but not sure its enough.

@dfrosch dfrosch changed the title 3 cases where the application was never awakened 3 cases where the application is never awakened Aug 5, 2015
@mluis1
Copy link
Contributor

mluis1 commented Aug 5, 2015

Thanks for the findings.
I have one request could you please try to separate the pull requests in the future.

Concerning the Channels initialization you are right the only mandatory ones are the 3 first ones. But, in order to simplify the tests and make usage of the 8 channels provided by the gateways it has been decided to hard code by default the frequencies from LC4 up to LC9.
The current frequencies choice corresponds to the recommended frequency plan for Europe.
If for some reason one is using other frequencies he can always change them on his own project.

Concerning the 3 cases of firmware freezing I have applied your recommendation for the next release of the firmware.

@mluis1 mluis1 closed this Aug 7, 2015
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