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

New .gitignore pattern, CC1101 streaming mode, RF69 SPI get/set/read/write overrides #279

Merged
merged 29 commits into from
Apr 22, 2021

Conversation

phretor
Copy link
Contributor

@phretor phretor commented Apr 13, 2021

  • new .gitignore pattern to prevent that .pio* files get sucked up (PlatformIO users will be happy)
  • [CC1101] new 255+ data length (almost like streaming mode) using FIFO
  • [RF69] SPI get/set read/write overrides to set bit for burst write and status registers access

aguglie and others added 22 commits February 16, 2020 21:53
Tested: CC1101_Receive, CC1101_Receive_Interrupt, CC1101_Transmit, CC1101_Transmit_Interrupt
* 'master' of github.com:rfquack/RadioLib: (135 commits)
  [CC1101] Added bitrate caching.
  [nRF24] Removed ambiguity, PR jgromes#119
  [nRF24] Minor fixes
  [nRF24] Added "isCarrierDetected()"
  Bump version to 3.3.1
  [SX127x] Set OOK parameter before setting the bitrate to avoid reading undefined variable
  [Module] Fixed incorrect arguments
  [Module] Fixed interface termination
  [SX1231] Removed old call to SPI end
  [RF69] Removed old call to SPI end
  [CC1101] Removed old call to SPI end
  [CC1101] Removed redundant null terminator
  [nRF24] Removed redundant null terminator
  [RF69] Removed redundant null terminator
  [CC1101] Requested changes PR jgromes#114
  [CC1101] Fixed interrupt direction. Tested: CC1101_Receive, CC1101_Receive_Interrupt, CC1101_Transmit, CC1101_Transmit_Interrupt
  [CC1101] Receive up to 255 bytes
  [CC1101] Transmit up to 255 bytes
  [CC1101] Added carrier sense as sync word qualifier
  [CC1101] Update cached len value.
  ...
* upstream/master: (249 commits)
  [SX127x] OOK functions style cleanup
  [RF69] OOK functions style cleanup
  styles and syntax changes
  added RF69::setOokThresholdType
  added RF69::setLnaTestBoost
  doc fix
  SX1278::getRSSI() modified for OOK so it wont require an activation
  SX127x::setOokPeakThresholdDecrement
  ignore jetbrain .idea
  SX1278::setGain() added support for FSK_OOK modes
  SX127x::setOokFixedOrFloorThreshold
  SX1278::setOokThresholdType()
  RF69::setOOK
  [SX127x] Fixed shaping in OOK (jgromes#268)
  [SX127x] Added FSK CRC mode configuration (jgromes#266)
  Bump version to 4.3.0
  [nRF24] Added low-level access macro
  [XBee] Added low-level access macro
  [Si443x] Added low-level access macro
  [SX128x] Added low-level access macro
  ...
* 'master' of github.com:rfquack/RadioLib:
  [RF69] Fixed setOOK()
  [CC1101] Fix in promiscuous mode
  Fixed compiling issues
Copy link
Owner

@jgromes jgromes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, I left some comments. Overall the biggest change is probably the stream mode for CC1101 - is that something that could be done more generically, e.g. using approach outlined in #201? And what's the current maximum data length for CC1101, is it unlimited now?

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

phretor commented Apr 13, 2021

Added a bunch of changes and comments. LMK what you think.

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

phretor commented Apr 21, 2021

Added some changes.

@jgromes jgromes merged commit 070b4f5 into jgromes:master Apr 22, 2021
@jgromes
Copy link
Owner

jgromes commented Apr 22, 2021

Merged, thanks for the continued contributions guys :) Release 4.4.0 coming up shortly.

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.

4 participants