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

Radiolib sync upstream #850

Merged
merged 8 commits into from
Sep 8, 2021

Conversation

osmanovv
Copy link
Contributor

@osmanovv osmanovv commented Sep 2, 2021

To reflect changes in the updated RadioLib (#849) we need slightly modify these two adapters:

  • RadioLibRF95
  • SX1262Interface

as the argument currentLimit was removed from the begin method:
jgromes/RadioLib@e1141ca#diff-507c32190e4a29ffe411c8eab06c75b594c5d2a5fd92e5cb703b787e5f019589L568

@osmanovv osmanovv force-pushed the radiolib-sync-upstream branch from 4c7c56e to f40bd07 Compare September 2, 2021 13:11
@geeksville
Copy link
Member

Thanks so much! This looks great, but the native build has a bit of a problem with it. I'll check and fix whatever is busted in the next day or two before merging.

@geeksville geeksville merged commit b9443d8 into meshtastic:master Sep 8, 2021
@osmanovv osmanovv deleted the radiolib-sync-upstream branch September 9, 2021 20:25
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