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

KR921 Frequency Plan #7

Closed
wants to merge 1 commit into from
Closed

KR921 Frequency Plan #7

wants to merge 1 commit into from

Conversation

pctj101
Copy link
Contributor

@pctj101 pctj101 commented Oct 7, 2018

This is what I have so far in trying to match with the following frequency plan for TTN:
https://account.thethingsnetwork.org/api/v2/frequency-plans/KR_920_923

Not sure what copyright notices should be applied to the "new" files based on "existing" files.

For many of the "magic numbers", made my best guess.

Looks like KR921 doesn't have FSK, so I took that out.
https://www.thethingsnetwork.org/docs/lorawan/frequency-plans.html#kr920-923

Verified operation of LoRa Node -> LoRa Gateway with Join/Uplink/Downlink operation on TTN with Gateway using KR_920_923 as stated above.

@manuelbl
Copy link
Owner

manuelbl commented Oct 7, 2018

Thanks a lot for your contribution.

As this library is based on MCCI's Arduino LMIC library, I'd prefer if the Korean frequency plan was part of their library. Otherwise it will be difficult to keep up with the bug fixes and enhancement in that library.

Can you also create a pull request for mcci-catena/arduino-lmic?

@pctj101
Copy link
Contributor Author

pctj101 commented Oct 9, 2018

Sure I'll take a look and see if they're interested.

@terrillmoore
Copy link

I'm the maintainer of https://github.com/mcci-catena/arduino-lmic; would be happy to have a pull request.

@manuelbl
Copy link
Owner

manuelbl commented Dec 3, 2019

The dev branch now contains the latest version of the MCCI-Catena LMIC library that includes the frequency plan for South Korea. So if you don't mind, I'm going to close this pull request.

@pctj101 pctj101 closed this Dec 4, 2019
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.

3 participants