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

US-like regions don't process JoinAccept CFList so don't adjust channel mask to match network #739

Closed
terrillmoore opened this issue May 30, 2021 · 0 comments · Fixed by #741
Assignees
Labels

Comments

@terrillmoore
Copy link
Member

terrillmoore commented May 30, 2021

Yikes. Looks like a big hole in the compliance tests (and in my testing, sigh; but at least I've found it now).

Not only does the function processJoinAccept() not cater for US-style CFLists, but it hard-codes that all CFLists are EU style.

Result: in US and AU regions, the CFList (channel mask) is ignored. So if the LMIC is not configured for the network using LMIC_selectSubBand(), it will generally use the wrong channels for uplinks.

Solution is to add bandplan-specific CFList processing routines, and change processJoinAccept to call the bandplan handler instead of doing it itself.

Even with this fixed, users should be aware that TTN V2 does not send a CFList with JoinAccept in the US. Devices will function -- once the first regular uplink message hits subband 1, the network will download a proper netmask -- but it will take quite a while after the join. I observed delays of about 8 uplinks. You will have to check your network manual to confirm whether JoinAccept sends a CFList, and also confirm whether it tries to send the network mask as a reply to subsequent Class A uplinks. I have not yet checked TTN V3 behavior.

@terrillmoore terrillmoore self-assigned this May 30, 2021
@terrillmoore terrillmoore changed the title US-like regions don't process JoinAccept CFList US-like regions don't process JoinAccept CFList so don't adjust channel mask to match network May 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant