Skip to content

I would like to reopen the issue. Encountered this again. #10152

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

Closed
khryssmar opened this issue Aug 13, 2024 · 5 comments
Closed

I would like to reopen the issue. Encountered this again. #10152

khryssmar opened this issue Aug 13, 2024 · 5 comments
Assignees
Labels
Peripheral: UART Resolution: Wontfix Arduino ESP32 team will not fix the issue

Comments

@khryssmar
Copy link

          I would like to reopen the issue. Encountered this again. Please fix. @SuGlider 

image
image

Originally posted by @khryssmar in #9500 (comment)

@khryssmar khryssmar changed the title I would like to reopen the issue. Encountered this again. Please fix. @SuGlider I would like to reopen the issue. Encountered this again. Aug 13, 2024
@lbernstone
Copy link
Contributor

Please fill out the issue details. This looks like it is still correct in 2.0.17 and v2.x branch. The change in v3.0 is intentional.

@Jason2866
Copy link
Collaborator

Which issue in general? Some defaults are changed for Core 3.0.x as designed.
Since UART IOs can freely assigned to nearly any GPIO just change as you want.

@SuGlider SuGlider self-assigned this Aug 13, 2024
@SuGlider SuGlider added Resolution: Wontfix Arduino ESP32 team will not fix the issue Peripheral: UART labels Aug 13, 2024
@SuGlider
Copy link
Collaborator

@khryssmar - this is intentional and it is documented in https://docs.espressif.com/projects/arduino-esp32/en/latest/migration_guides/2.x_to_3.0.html#id14

In case that the sketch needs to use other pins, Serial1 and Serial2 can change their pins by using Serial1.begin(baud_rate, mode, rxPin, txPin) or by Serial2.setPins(rxPin, txPin).
Setting the pin to -1 will force it to keep the default or to keep it unchanged.
Example: Serial2.setPins(-1, 17); will only change the TX2 pin to 17.

The problem with the former ESP32 Serial2 pins is that those are used to control the PSRAM within the ESP32-WROVER module. In case that the sketch uses PSRAM, it would crash its execution.

@SuGlider
Copy link
Collaborator

For the records:

This pins change is valid for Arduino Core 3.x.x. But it was commited by mistake into 2.0.15 -- 2.0.17.
Explanation in #10152 (comment)


Arduino Core 2.0.15, 2.0.16 and 2.0.17 had a mistake using new pins.
Arduino Core 2.0.14 and lower use the original pins.

Arduino Core from release/v2.x branch uses the same pins as before:
Fixed with #9502

@SuGlider
Copy link
Collaborator

SuGlider commented Aug 13, 2024

Arduino Core branch release/v2.x will continue to receive new contributions as usual.
But it is very improbable that we will release a 2.0.18 version.

It is possible to create a sketchbook board from a cloned branch, like release/v2.x
For that, please follow the instructions from
https://docs.espressif.com/projects/arduino-esp32/en/latest/installing.html#windows-manual-installation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Peripheral: UART Resolution: Wontfix Arduino ESP32 team will not fix the issue
Projects
None yet
Development

No branches or pull requests

4 participants