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

Variable/fixed packet length mode config for RF69/SX1231 #56

Merged
merged 3 commits into from
Nov 8, 2019

Conversation

phretor
Copy link
Contributor

@phretor phretor commented Nov 8, 2019

Related idea: is it possible to just call RF69::begin within SX1X1231::begin and just add the if (_chipRevision == SX1231_CHIP_REVISION_2_A) block before returning?

src/modules/RF69.cpp Outdated Show resolved Hide resolved
@phretor
Copy link
Contributor Author

phretor commented Nov 8, 2019

Changeset 39a4614 should address the latest request.

@jgromes
Copy link
Owner

jgromes commented Nov 8, 2019

is it possible to just call RF69::begin within SX1X1231::begin

Unforutnately it's not that easy - SX1231 has different values in the version register. RF69::begin would have to be generalized, a bit like SX127x::begin

@jgromes jgromes merged commit e61e9af into jgromes:master Nov 8, 2019
@jgromes
Copy link
Owner

jgromes commented Nov 8, 2019

Looks good, thanks!

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