Skip to content

pinMatrixInDetach() has wrong parameter #5385

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

Merged
merged 2 commits into from
Jul 14, 2021
Merged

pinMatrixInDetach() has wrong parameter #5385

merged 2 commits into from
Jul 14, 2021

Conversation

SuGlider
Copy link
Collaborator

This solves issue #5112
Call to pinMatrixInDetach() was changed from version 1.0.6 in version 2.0.0 injecting a bug as seen in cores/esp32/esp32-hal-uart.c
80418fa

This solves issue #5112
Call to pinMatrixInDetach() was changed from version 1.0.6 in version 2.0.0 injecting a bug as seen in cores/esp32/esp32-hal-uart.c
80418fa
@SuGlider SuGlider mentioned this pull request Jul 14, 2021
@CLAassistant
Copy link

CLAassistant commented Jul 14, 2021

CLA assistant check
All committers have signed the CLA.

@me-no-dev me-no-dev merged commit f4f1c89 into espressif:master Jul 14, 2021
TD-er added a commit to TD-er/ESPEasySerial that referenced this pull request Oct 13, 2021
ESP32 does have some issue where it is not properly detaching the RX/TX pins when calling `end()`
This happens especially when the RX pin receives data when calling `end()`.

Problem is that the serial port will no longer work after calling `end()` followed by `begin()` until it reboots.

This very messy work-around is to cache the used pins and not calling `begin()` on the port if using the same pins for a specific port.

Should be fixed with: espressif/arduino-esp32#5385
But that core version cannot yet be used due to other issues, like a bug with web authentication and issues with WiFi and ESP-NOW.
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.

3 participants