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

Revisioned promiscious-mode implementation for CC1101 and RF69 #63 #66

Merged
merged 8 commits into from
Nov 13, 2019

Conversation

phretor
Copy link
Contributor

@phretor phretor commented Nov 13, 2019

Promiscuous mode is needed in sniffing scenarios, similarly to variable vs. fixed packet mode. I also made setSyncWord uniform, retaining the original signature for compatibility.

I'm not aware of a way to disable sync word detection alone in CC1101. The only way I know also disables preamble detection altogether. Therefore, disableSyncWordFiltering also disables preamble filtering.

If you know a better way to address this, I'll commit some changes.

* 'master' of https://github.com/jgromes/RadioLib:
  Fixed keyword separator
  Fixed SoftwareSerial for ESP8266 core 2.6.0
  [SX126x] Moved debug output out of SPI transaction
* 'master' of https://github.com/jgromes/RadioLib:
  Comment and error checking cleanup. Fixed power adjustment for SX1261
  Removed memory allocation and copying from SX126X read* and write* functions
  Implemented optimal PA settings for SX1268. Ensured OCP is always restored when changing power. Slight refactor to avoid duplicated SX1262 / SX1268 code.
  Fixed link to ESP32 package index
  Implemented power scaling in setOutputPower, in accordance with datasheet.
  Missed SX1261 PA Config flag in previous commit
  Adjusted setPaConfig in SX126x setOutputPower.
@jgromes jgromes merged commit e48275e into jgromes:master Nov 13, 2019
@jgromes
Copy link
Owner

jgromes commented Nov 13, 2019

Merged, release most likely later this week.

Thank you very much for this contribution, I really appreciate it.

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