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

ESP32 - use HardwareSerial instead of SoftwareSerial #38

Closed
wants to merge 4 commits into from
Closed

ESP32 - use HardwareSerial instead of SoftwareSerial #38

wants to merge 4 commits into from

Conversation

beegee-tokyo
Copy link
Contributor

ESP32 has no working SoftwareSerial. With a simple #ifdef ESP32 a hardware serial will be used instead.
Minimum changes without influence on existing installations or usage.

ESP32 has no working SoftwareSerial. With a simple #ifdef ESP32 a hardware serial will be used instead.    
Minimum changes without influence on existing installations or usage.
@beegee-tokyo
Copy link
Contributor Author

I know you are not targeting the ESP32. But your library is the best LoRa library I found. Would be a pitty if the ESP32 would be left out.
Unfortunately ESP32 has no SoftwareSerial support. The existing EspSoftwareSerial does not compile for me and is on top outdated.

@Pablo2048
Copy link

The same thing is valid for SAMD platform (Arduino Zero), so +1 for this PR and use it like #if defined(ESP32) || defined(SAMD_SERIES) ...

ESP32 has no working SoftwareSerial. With a simple #ifdef ESP32 a hardware serial will be used instead.    
Minimum changes without influence on existing installations or usage.
@beegee-tokyo
Copy link
Contributor Author

@Pablo2048
I cannot test on SAMD, I have no board with a SAMD, only one a Sparkfun LoRa board with an SAMD 21, is that the same?
Because that compiles without problems with this library.

Anyway, I somehow messed up this pull request, will retry.

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