Skip to content

Commit f2bc23d

Browse files
authored
fix(uart): ci uart test fail on esp32s2 after uart break
1 parent bda7c48 commit f2bc23d

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

tests/validation/uart/uart.ino

+4-3
Original file line numberDiff line numberDiff line change
@@ -365,12 +365,13 @@ void change_pins_test(void) {
365365

366366
if (TEST_UART_NUM == 1) {
367367
UARTTestConfig &config = *uart_test_configs[0];
368+
// internal loopback causes UART BREAK on ESP32 and ESP32-S2
369+
// setting it before changing the pins solves it
370+
uart_internal_loopback(config.uart_num, NEW_RX1);
368371
config.serial.setPins(NEW_RX1, NEW_TX1);
369372
TEST_ASSERT_EQUAL(NEW_RX1, uart_get_RxPin(config.uart_num));
370373
TEST_ASSERT_EQUAL(NEW_TX1, uart_get_TxPin(config.uart_num));
371-
372-
uart_internal_loopback(config.uart_num, NEW_RX1);
373-
config.transmit_and_check_msg("using new pins");
374+
config.transmit_and_check_msg("using new uart#1 pins");
374375
} else {
375376
for (int i = 0; i < TEST_UART_NUM; i++) {
376377
UARTTestConfig &config = *uart_test_configs[i];

0 commit comments

Comments
 (0)