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

add option to change RX/TX pins if blocked by USB bridge #3

Merged
merged 3 commits into from
Jul 28, 2024

Conversation

ALfuhrmann
Copy link
Contributor

Since on many 8266 boards the hardware UART is already used by the USB-bridge, I added the option to swap the serial pins to GPIO15 (TX) and GPIO13 (RX) .
(See https://arduino.esp8266.com/Arduino/versions/2.1.0-rc2/doc/reference.html#serial)
I added a #define in config_esp8266.h for this.

Tested it on an Wemos D1 mini, works fine at my home!

@ALfuhrmann ALfuhrmann closed this Oct 26, 2023
@ALfuhrmann ALfuhrmann reopened this Oct 26, 2023
@aldadic
Copy link
Owner

aldadic commented Nov 1, 2023

Hi @ALfuhrmann!
Thank you for your contribution! I think it is a very good idea to add this as a config option. Since I am abroad probably until Christmas, I am not able to test your addition, but the code looks good to me. However, before merging I would like to document and explain this option in the setup part of the README, which I am only able to do once I am back home.

@aldadic aldadic merged commit d2fbad3 into aldadic:main Jul 28, 2024
2 checks passed
@aldadic
Copy link
Owner

aldadic commented Jul 28, 2024

Thank you again for your contribution. I finally had the time to test your addition. I wasn't aware of the Serial.swap() functionality. By shorting GPIO2 (D4) to TX, it is possible to use Serial1 to send data to the serial monitor via USB, which is useful for logging purposes. I will change the documentation, such that this becomes the recommended setup for ESP8266.

@ALfuhrmann
Copy link
Contributor Author

You are welcome, glad to contribute!

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