Skip to content

Commit d796f5a

Browse files
committed
fix(test): Fix UART test
1 parent b7acbb9 commit d796f5a

File tree

1 file changed

+30
-30
lines changed

1 file changed

+30
-30
lines changed

tests/validation/uart/uart.ino

+30-30
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@
4646
*
4747
*/
4848

49-
#if SOC_UART_NUM == 2
49+
#if SOC_UART_HP_NUM == 2
5050
// Used for the pin swap test
5151
#define NEW_RX1 9
5252
#define NEW_TX1 10
@@ -64,14 +64,14 @@ extern int8_t uart_get_TxPin(uint8_t uart_num);
6464

6565
// This function starts all the available test UARTs
6666
void start_serial(unsigned long baudrate = 115200) {
67-
#if SOC_UART_NUM >= 2
67+
#if SOC_UART_HP_NUM >= 2
6868
Serial1.begin(baudrate);
6969
while (!Serial1) {
7070
delay(10);
7171
}
7272
#endif
7373

74-
#if SOC_UART_NUM >= 3
74+
#if SOC_UART_HP_NUM >= 3
7575
Serial2.begin(baudrate);
7676
while (!Serial2) {
7777
delay(10);
@@ -81,26 +81,26 @@ void start_serial(unsigned long baudrate = 115200) {
8181

8282
// This function stops all the available test UARTs
8383
void stop_serial(bool hard_stop = false) {
84-
#if SOC_UART_NUM >= 2
84+
#if SOC_UART_HP_NUM >= 2
8585
Serial1.end(/*hard_stop*/);
8686
#endif
8787

88-
#if SOC_UART_NUM >= 3
88+
#if SOC_UART_HP_NUM >= 3
8989
Serial2.end(/*hard_stop*/);
9090
#endif
9191
}
9292

9393
// This function transmits a message and checks if it was received correctly
9494
void transmit_and_check_msg(const String msg_append, bool perform_assert = true) {
9595
delay(100); // Wait for some settings changes to take effect
96-
#if SOC_UART_NUM == 2
96+
#if SOC_UART_HP_NUM == 2
9797
Serial1.print("Hello from Serial1 (UART1) >>> via loopback >>> Serial1 (UART1) " + msg_append);
9898
Serial1.flush();
9999
delay(100);
100100
if (perform_assert) {
101101
TEST_ASSERT_EQUAL_STRING(("Hello from Serial1 (UART1) >>> via loopback >>> Serial1 (UART1) " + msg_append).c_str(), recv_msg.c_str());
102102
}
103-
#elif SOC_UART_NUM == 3
103+
#elif SOC_UART_HP_NUM == 3
104104
Serial1.print("Hello from Serial1 (UART1) >>> to >>> Serial2 (UART2) " + msg_append);
105105
Serial1.flush();
106106
delay(100);
@@ -126,9 +126,9 @@ void transmit_and_check_msg(const String msg_append, bool perform_assert = true)
126126
void task_delayed_msg(void *pvParameters) {
127127
HardwareSerial *selected_serial;
128128

129-
#if SOC_UART_NUM == 2
129+
#if SOC_UART_HP_NUM == 2
130130
selected_serial = &Serial;
131-
#elif SOC_UART_NUM == 3
131+
#elif SOC_UART_HP_NUM == 3
132132
selected_serial = &Serial1;
133133
#endif
134134

@@ -143,14 +143,14 @@ void task_delayed_msg(void *pvParameters) {
143143
// This function is automatically called by unity before each test is run
144144
void setUp(void) {
145145
start_serial(115200);
146-
#if SOC_UART_NUM == 2
146+
#if SOC_UART_HP_NUM == 2
147147
log_d("Setup internal loop-back from and back to Serial1 (UART1) TX >> Serial1 (UART1) RX");
148148

149149
Serial1.onReceive([]() {
150150
onReceive_cb(Serial1);
151151
});
152152
uart_internal_loopback(1, RX1);
153-
#elif SOC_UART_NUM == 3
153+
#elif SOC_UART_HP_NUM == 3
154154
log_d("Setup internal loop-back between Serial1 (UART1) <<--->> Serial2 (UART2)");
155155

156156
Serial1.onReceive([]() {
@@ -180,11 +180,11 @@ void onReceive_cb(HardwareSerial &selected_serial) {
180180

181181
if (&selected_serial == &Serial) {
182182
uart_num = 0;
183-
#if SOC_UART_NUM >= 2
183+
#if SOC_UART_HP_NUM >= 2
184184
} else if (&selected_serial == &Serial1) {
185185
uart_num = 1;
186186
#endif
187-
#if SOC_UART_NUM >= 3
187+
#if SOC_UART_HP_NUM >= 3
188188
} else if (&selected_serial == &Serial2) {
189189
uart_num = 2;
190190
#endif
@@ -225,7 +225,7 @@ void change_baudrate_test(void) {
225225
Serial1.updateBaudRate(9600);
226226
TEST_ASSERT_UINT_WITHIN(192, 9600, Serial1.baudRate());
227227

228-
#if SOC_UART_NUM == 3
228+
#if SOC_UART_HP_NUM == 3
229229
Serial2.updateBaudRate(9600);
230230
TEST_ASSERT_UINT_WITHIN(192, 9600, Serial2.baudRate());
231231
#endif
@@ -239,7 +239,7 @@ void change_baudrate_test(void) {
239239
//Baudrate error should be within 2% of the target baudrate
240240
TEST_ASSERT_UINT_WITHIN(2304, 115200, Serial1.baudRate());
241241

242-
#if SOC_UART_NUM == 3
242+
#if SOC_UART_HP_NUM == 3
243243
TEST_ASSERT_UINT_WITHIN(2304, 115200, Serial2.baudRate());
244244
#endif
245245

@@ -419,20 +419,20 @@ void change_pins_test(void) {
419419

420420
log_d("Disabling UART loopback");
421421

422-
#if SOC_UART_NUM == 2
422+
#if SOC_UART_HP_NUM == 2
423423
esp_rom_gpio_connect_out_signal(SOC_RX0, SIG_GPIO_OUT_IDX, false, false);
424-
#elif SOC_UART_NUM == 3
424+
#elif SOC_UART_HP_NUM == 3
425425
esp_rom_gpio_connect_out_signal(RX1, SIG_GPIO_OUT_IDX, false, false);
426426
esp_rom_gpio_connect_out_signal(RX2, SIG_GPIO_OUT_IDX, false, false);
427427
#endif
428428

429429
log_d("Swapping UART pins");
430430

431-
#if SOC_UART_NUM == 2
431+
#if SOC_UART_HP_NUM == 2
432432
Serial1.setPins(NEW_RX1, NEW_TX1);
433433
TEST_ASSERT_EQUAL(NEW_RX1, uart_get_RxPin(1));
434434
TEST_ASSERT_EQUAL(NEW_TX1, uart_get_TxPin(1));
435-
#elif SOC_UART_NUM == 3
435+
#elif SOC_UART_HP_NUM == 3
436436
Serial1.setPins(RX2, TX2);
437437
Serial2.setPins(RX1, TX1);
438438
TEST_ASSERT_EQUAL(RX2, uart_get_RxPin(1));
@@ -445,9 +445,9 @@ void change_pins_test(void) {
445445

446446
log_d("Re-enabling UART loopback");
447447

448-
#if SOC_UART_NUM == 2
448+
#if SOC_UART_HP_NUM == 2
449449
uart_internal_loopback(1, NEW_RX1);
450-
#elif SOC_UART_NUM == 3
450+
#elif SOC_UART_HP_NUM == 3
451451
uart_internal_loopback(1, RX1);
452452
uart_internal_loopback(2, RX2);
453453
#endif
@@ -467,10 +467,10 @@ void auto_baudrate_test(void) {
467467

468468
log_d("Stopping test serial. Using Serial2 for ESP32 and Serial1 for ESP32-S2.");
469469

470-
#if SOC_UART_NUM == 2
470+
#if SOC_UART_HP_NUM == 2
471471
selected_serial = &Serial1;
472472
uart_internal_loopback(0, RX1);
473-
#elif SOC_UART_NUM == 3
473+
#elif SOC_UART_HP_NUM == 3
474474
selected_serial = &Serial2;
475475
#endif
476476

@@ -485,7 +485,7 @@ void auto_baudrate_test(void) {
485485
selected_serial->begin(0);
486486
baudrate = selected_serial->baudRate();
487487

488-
#if SOC_UART_NUM == 2
488+
#if SOC_UART_HP_NUM == 2
489489
Serial.end();
490490
Serial.begin(115200);
491491
#endif
@@ -504,7 +504,7 @@ void periman_test(void) {
504504

505505
Wire.begin(RX1, TX1);
506506

507-
#if SOC_UART_NUM == 3
507+
#if SOC_UART_HP_NUM == 3
508508
Wire1.begin(RX2, TX2);
509509
#endif
510510

@@ -518,11 +518,11 @@ void periman_test(void) {
518518

519519
Serial1.setPins(RX1, TX1);
520520

521-
#if SOC_UART_NUM == 3
521+
#if SOC_UART_HP_NUM == 3
522522
Serial2.setPins(RX2, TX2);
523523
uart_internal_loopback(1, RX2);
524524
uart_internal_loopback(2, RX1);
525-
#elif SOC_UART_NUM == 2
525+
#elif SOC_UART_HP_NUM == 2
526526
uart_internal_loopback(1, RX1);
527527
#endif
528528

@@ -565,19 +565,19 @@ void setup() {
565565
while (!Serial) {
566566
delay(10);
567567
}
568-
log_d("SOC_UART_NUM = %d", SOC_UART_NUM);
568+
log_d("SOC_UART_HP_NUM = %d", SOC_UART_HP_NUM);
569569

570570
// Begin needs to be called before setting up the loopback because it creates the serial object
571571
start_serial(115200);
572572

573-
#if SOC_UART_NUM == 2
573+
#if SOC_UART_HP_NUM == 2
574574
log_d("Setup internal loop-back from and back to Serial1 (UART1) TX >> Serial1 (UART1) RX");
575575

576576
Serial1.onReceive([]() {
577577
onReceive_cb(Serial1);
578578
});
579579
uart_internal_loopback(1, RX1);
580-
#elif SOC_UART_NUM == 3
580+
#elif SOC_UART_HP_NUM == 3
581581
log_d("Setup internal loop-back between Serial1 (UART1) <<--->> Serial2 (UART2)");
582582

583583
Serial1.onReceive([]() {

0 commit comments

Comments
 (0)