Esp32-S3 Baud Rate Detection missing

[imwhocodes]

Related area

UART Baud Rate Detection

Hardware specification

Esp32-S3

Is your feature request related to a problem?

Auto Baud Rate feature is not supported/wrapped in arduino-esp32 for the esp32-S3
Hardware implementation / register are slightly different from previous esp32 revisions but it don't seem nothing drastic

(And there is no log-error advising user or the programmer that feature is not ready yet)

Describe the solution you'd like

Same behaviour as previous esp32 revisions

Describe alternatives you've considered

No response

Additional context

I tried to implement it by myself but I'm unable to compute the correct value, while reading the reference this line is not clear to me: UART_HIGHPULSE_MIN_CNT stores minimum high-pulse width
I'm unable to find what is the unit of measurement used for this value

I have checked existing list of Feature requests and the Contribution Guide

• I confirm I have checked existing list of Feature requests and Contribution Guide.

[SuGlider]

@imwhocodes -
It looks like you are using the ESP32 reference manual with the ESP32-S3.
Is it correct?

There is a code for it in
https://github.com/espressif/arduino-esp32/blob/master/cores/esp32/HardwareSerial.cpp#L380-L400
https://github.com/espressif/arduino-esp32/blob/master/cores/esp32/esp32-hal-uart.c#L690-L747

Last time I tried it with ESP32-S3, it worked fine.

You can try this Arduino code:

   // this will start UART in 9600 baud and then it will try to adjuts itself to the transmiter baud rate
   // if it can't detect the transmiter baudrate, it will issue a log error message. Turn on Log to Error Level in the IDE.
   Serial.begin(0);  

[imwhocodes]

Sorry I linked the wrong revision/model manual

I'm gonna try again, but what I did is using the same code that was correctly detecting the baud rate on ESP32 and tested on ESP32-S3 without any positive result (it always used default baudrate)

The thing that seem strange is that here nothing is enabled for the ESP32-S3
https://github.com/espressif/arduino-esp32/blob/master/cores/esp32/esp32-hal-uart.c#L681-L687

And here enabling is skipped too:
https://github.com/espressif/arduino-esp32/blob/master/cores/esp32/esp32-hal-uart.c#L716-L722

As I said i tried to fix it by myself, updating the struct name of the various parameter (they have different name in the ESP32-S3 HAL), it compile, return me a value but it is not correct

[SuGlider]

Thanks @imwhocodes - I'll try to make it work here as well.
Maybe there is some missing code as you pointed out.
Let's keep investigating the issue.

[imwhocodes]

Hi @SuGlider I submitted a PR #7782 addressing this issue

Everything is explained there

For the moment I'm leaving issue open until completion