Skip to content

Commit

Permalink
Merge pull request #21042 from maribu/tests/periph/selftest_shield/fi…
Browse files Browse the repository at this point in the history
…x-uart

tests/periph/selftest_shield: Fix UART test
  • Loading branch information
maribu authored Nov 26, 2024
2 parents d0e4f4d + a51d4bd commit 3b238f7
Showing 1 changed file with 9 additions and 2 deletions.
11 changes: 9 additions & 2 deletions tests/periph/selftest_shield/main.c
Original file line number Diff line number Diff line change
Expand Up @@ -55,10 +55,14 @@
/* In order to run the periph_uart test all of the following needs to be true:
* - periph_uart needs to be used
* - an I/O mapping for the UART at D0/D1 needs to be provided
* - this UART dev is not busy with stdio
* - this UART dev is not busy with stdio (either not using `stdio_uart` or a different UART is used for stdio)

Check warning on line 58 in tests/periph/selftest_shield/main.c

View workflow job for this annotation

GitHub Actions / static-tests

line is longer than 100 characters
*/
#if defined(ARDUINO_UART_D0D1) && defined(MODULE_PERIPH_UART)
# define ENABLE_UART_TEST (STDIO_UART_DEV != ARDUINO_UART_D0D1)
# if MODULE_STDIO_UART
# define ENABLE_UART_TEST (STDIO_UART_DEV != ARDUINO_UART_D0D1)
# else
# define ENABLE_UART_TEST 1
# endif
# define UART_TEST_DEV ARDUINO_UART_D0D1
#else
# define ENABLE_UART_TEST 0
Expand Down Expand Up @@ -790,6 +794,9 @@ static bool periph_uart_rxtx_test(uint32_t symbolrate, uint32_t timer_freq)
failed |= TEST(memcmp(testdata, serial_buf.data, sizeof(serial_buf.data)) == 0);
failed |= TEST(serial_buf.pos == sizeof(testdata));

/* disable UART again, in case it is a shared peripheral */
uart_poweroff(UART_TEST_DEV);

return failed;
}

Expand Down

0 comments on commit 3b238f7

Please sign in to comment.