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

SX1268 module support and base class for SX126x module family #865

Merged
merged 11 commits into from
Sep 18, 2021

Conversation

osmanovv
Copy link
Contributor

  • SX1268 module support. Define USE_SX1268 and SX126X_[CS|DIO1|RESET|BUSY|etc.] pinouts to enable it.
  • New base class SX126xInterface for SX126x modules with common realization. Child modules specifies LoRa adapter from RadioLib in template, e.g. class SX1262Interface : public SX126xInterface<SX1262>.
  • The RadioInterface::freq member was encapsulated with the RadioInterface::getFreq() function, which could be overridden in child classes for some LoRa-modules.

Class diagram for these changes (view in mermaid editor):
ClassDiagram

Otherwise, we get critical error 3 with result code -12 (ERR_INVALID_FREQUENCY):
The supplied frequency value is invalid for this module.
The `RadioInterface::freq` member was encapsulated with the `RadioInterface::getFreq()` function,
which could be overridden in child classes for some LoRa-modules.
The frequency could be overridden in `RadioInterface::getFreq()` for some modules.
@geeksville
Copy link
Member

AWESOME!

@geeksville geeksville merged commit 2f7e200 into meshtastic:master Sep 18, 2021
@osmanovv osmanovv deleted the sx1268-support branch September 25, 2021 03:33
@osmanovv osmanovv mentioned this pull request Nov 18, 2021
2 tasks
@geeksville
Copy link
Member

This pull request has been mentioned on Meshtastic. There might be relevant details there:

https://meshtastic.discourse.group/t/sx1268-modules-are-not-yet-supported/3659/10

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.

None yet

2 participants