RIOT-OS/RIOT
RIOT-OS/RIOT
1
0
Fork 0
mirror of https://github.com/RIOT-OS/RIOT.git synced 2024-12-29 04:50:03 +01:00
Code Releases Activity

Merge pull request #20083 from maribu/tests/periph/selftest_shield

Browse Source 
tests/periph/selftest_shield: re-enable ADC test
This commit is contained in:
Marian Buschsieweke 2023-11-16 12:04:00 +00:00 committed by GitHub
commit 14ff958659
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
tests/periph/selftest_shield/main.c
View File

@ -26,6 +26,7 @@
#include <stdlib.h> #include <stdlib.h>
#include <string.h> #include <string.h>
#include "architecture.h"
#include "arduino_iomap.h" #include "arduino_iomap.h"
#include "macros/units.h" #include "macros/units.h"
#include "macros/utils.h" #include "macros/utils.h"
@ -40,9 +41,6 @@
#include "stdio_uart.h" /* for STDIO_UART_DEV */ #include "stdio_uart.h" /* for STDIO_UART_DEV */
/* BEGIN: controls of the behavior of the testing app: */ /* BEGIN: controls of the behavior of the testing app: */
#define ENABLE_DEBUG 1
#include "debug.h"
#ifndef STOP_ON_FAILURE #ifndef STOP_ON_FAILURE
#define STOP_ON_FAILURE 0 #define STOP_ON_FAILURE 0
#endif #endif
@ -93,10 +91,9 @@
* - periph_adc support (so that we have something to test) * - periph_adc support (so that we have something to test)
* - Arduino I/O mapping for ADC (so that we know which line to test) * - Arduino I/O mapping for ADC (so that we know which line to test)
* - The PCF857x driver (so that we can control the R-2R resistor ladder * - The PCF857x driver (so that we can control the R-2R resistor ladder
* connected to the GPIO expander. * connected to the GPIO expander).
*/ */
#if defined(MODULE_PERIPH_ADC) && defined(ARDUINO_A0) && defined(MODULE_PCF857X) && 0 #if defined(MODULE_PERIPH_ADC) && defined(ARDUINO_A0) && defined(MODULE_PCF857X)
// TODO: Re-anble when PCB is fixed
# define ENABLE_ADC_TEST 1 # define ENABLE_ADC_TEST 1
#else #else
# define ENABLE_ADC_TEST 0 # define ENABLE_ADC_TEST 0
@ -716,8 +713,8 @@ static bool periph_uart_rxtx_test(uint32_t symbolrate)
/* expecting actual duration within 75% to 200% of the expected. */ /* expecting actual duration within 75% to 200% of the expected. */
failed |= TEST(stop - start > duration_ticks - (duration_ticks >> 2)); failed |= TEST(stop - start > duration_ticks - (duration_ticks >> 2));
failed |= TEST(stop - start < (duration_ticks << 1)); failed |= TEST(stop - start < (duration_ticks << 1));
if (failed) { if (failed && DETAILED_OUTPUT) {
DEBUG("%" PRIu32 " Bd, expected %" PRIu16 " ticks, got %" PRIu16 printf("%" PRIu32 " Bd, expected %" PRIu16 " ticks, got %" PRIu16
" ticks\n", " ticks\n",
symbolrate, duration_ticks, (uint16_t)(stop - start)); symbolrate, duration_ticks, (uint16_t)(stop - start));
} }
@ -979,9 +976,9 @@ static bool periph_adc_test(void)
uint16_t sample = adc_sample(ARDUINO_A0, ADC_RES_10BIT); uint16_t sample = adc_sample(ARDUINO_A0, ADC_RES_10BIT);
uint16_t expected = i << 6; uint16_t expected = i << 6;
/* the resistors are said to be rather accurate, so lets be a bit /* The resistors on board v0.3 are not that accurate, so allow for 10%
* more strict here */ * error margin */
const uint16_t delta = 16; const uint16_t delta = 1024 / 10;
uint16_t lower = expected <= delta ? 0 : expected - delta; uint16_t lower = expected <= delta ? 0 : expected - delta;
uint16_t upper = MIN(1023, expected + delta); uint16_t upper = MIN(1023, expected + delta);
bool test_failed = TEST((lower <= sample) && (upper >= sample)); bool test_failed = TEST((lower <= sample) && (upper >= sample));