Skip to content

Commit b32f0d8

Browse files
committed
tests: drivers: adc: enhance test filtering and config
Make tests and test scenaries depend on the existence of required DT elements. Now that tests are filtered automatically depending on DT, remove platform_allow/exclude from generic test scenarios. Add more integration platforms to cover most ADC IPs/drivers. Signed-off-by: Abderrahmane JARMOUNI <[email protected]>
1 parent a1c597c commit b32f0d8

File tree

6 files changed

+30
-54
lines changed

6 files changed

+30
-54
lines changed

boards/renesas/ek_ra8m1/ek_ra8m1.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,4 +15,5 @@ supported:
1515
- counter
1616
- i2s
1717
- i3c
18+
- adc
1819
vendor: renesas

tests/drivers/adc/adc_accuracy_test/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
cmake_minimum_required(VERSION 3.20.0)
44

55
find_package(Zephyr REQUIRED HINTS $ENV{ZEPHYR_BASE})
6-
project(dac_accuracy)
6+
project(adc_accuracy)
77

88
target_sources(app PRIVATE src/main.c)
99
target_sources_ifdef(CONFIG_REFERENCE_VOLTAGE_TEST app PRIVATE src/ref_volt.c)

tests/drivers/adc/adc_accuracy_test/Kconfig

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,16 @@ source "Kconfig.zephyr"
1111
ZEPHYR_USER := zephyr,user
1212

1313
config DAC_SOURCE_TEST
14-
bool
14+
bool "Test using a DAC source"
1515
select DAC
16-
default y if $(dt_node_has_prop,/$(ZEPHYR_USER),dac)
16+
depends on $(dt_node_has_prop,/$(ZEPHYR_USER),dac)
17+
depends on $(dt_node_has_prop,/$(ZEPHYR_USER),dac-channel-id)
18+
depends on $(dt_node_has_prop,/$(ZEPHYR_USER),dac-resolution)
1719

1820
config REFERENCE_VOLTAGE_TEST
19-
bool
20-
default y if $(dt_node_has_prop,/$(ZEPHYR_USER),reference-mv)
21+
bool "Test using a ref voltage source"
22+
depends on $(dt_node_has_prop,/$(ZEPHYR_USER),reference-mv)
23+
depends on $(dt_node_has_prop,/$(ZEPHYR_USER),expected-accuracy)
2124

2225
config NUMBER_OF_PASSES
2326
int "Number of passes"

tests/drivers/adc/adc_accuracy_test/testcase.yaml

Lines changed: 13 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -4,45 +4,29 @@ common:
44
- drivers
55
depends_on:
66
- adc
7+
filter: dt_node_has_prop("/zephyr,user","io-channels")
8+
79
tests:
810
drivers.adc.accuracy.dac_source:
911
depends_on:
1012
- dac
13+
extra_configs:
14+
- CONFIG_DAC_SOURCE_TEST=y
15+
# Test scenario is filtered if Kconfig dependencies are not satisfied
16+
filter: CONFIG_DAC_SOURCE_TEST
1117
harness_config:
1218
fixture: dac_adc_loopback
13-
platform_allow:
19+
integration_platforms:
1420
- frdm_k64f
1521
drivers.adc.accuracy.ref_volt:
22+
extra_configs:
23+
- CONFIG_REFERENCE_VOLTAGE_TEST=y
24+
# Test scenario is filtered if Kconfig dependencies are not satisfied
25+
filter: CONFIG_REFERENCE_VOLTAGE_TEST
1626
harness_config:
1727
fixture: adc_ref_volt
18-
platform_allow:
19-
- frdm_kl25z
20-
- ek_ra8m1
21-
- frdm_mcxc242
22-
- frdm_mcxc444
23-
- nrf52840dk/nrf52840
24-
- nrf54h20dk/nrf54h20/cpuapp
25-
- nrf54l15dk/nrf54l15/cpuapp
26-
- nrf54lm20dk/nrf54lm20a/cpuapp
27-
- ophelia4ev/nrf54l15/cpuapp
28-
- ek_ra8d1
29-
- mck_ra8t1
30-
- ek_ra6e2
31-
- ek_ra6m1
32-
- ek_ra6m2
33-
- ek_ra6m3
34-
- ek_ra6m4
35-
- ek_ra6m5
36-
- fpb_ra6e1
37-
- fpb_ra6e2
38-
- ek_ra4e2
39-
- ek_ra4m2
40-
- ek_ra4m3
41-
- ek_ra4w1
42-
- xg24_dk2601b
43-
- xg24_rb4187c
44-
- xg27_dk2602a
45-
- xg29_rb4412a
4628
integration_platforms:
29+
- ek_ra8m1
4730
- frdm_kl25z
4831
- nrf54l15dk/nrf54l15/cpuapp
32+
- xg24_dk2601b

tests/drivers/adc/adc_api/testcase.yaml

Lines changed: 5 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -3,20 +3,14 @@ common:
33
- adc
44
- drivers
55
- userspace
6+
depends_on: adc
7+
filter: dt_node_has_prop("/zephyr,user","io-channels")
68
tests:
79
drivers.adc:
8-
depends_on: adc
910
min_flash: 40
10-
platform_exclude:
11-
- nucleo_u031r8
12-
- panb511evb/nrf54l15/cpuapp
13-
- panb511evb/nrf54l15/cpuapp/ns
14-
- nrf54l15dk/nrf54l15/cpuapp/ns
15-
- nrf54l15dk/nrf54l10/cpuapp/ns
16-
- bl54l15_dvk/nrf54l10/cpuapp/ns
17-
- bl54l15_dvk/nrf54l15/cpuapp/ns
18-
- bl54l15u_dvk/nrf54l15/cpuapp/ns
19-
- raytac_an54l15q_db/nrf54l15/cpuapp/ns
11+
integration_platforms:
12+
- native_sim
13+
- native_sim/native/64
2014
drivers.adc.b_u585i_iot02a_adc4:
2115
extra_args:
2216
- DTC_OVERLAY_FILE="boards/b_u585i_iot02a_adc4.overlay"
@@ -32,7 +26,6 @@ tests:
3226
extra_args:
3327
- EXTRA_CONF_FILE="overlay-dma-stm32.conf"
3428
depends_on:
35-
- adc
3629
- dma
3730
min_flash: 40
3831
platform_allow:
@@ -67,7 +60,6 @@ tests:
6760
extra_args:
6861
- EXTRA_CONF_FILE="overlay-dma-kinetis.conf"
6962
depends_on:
70-
- adc
7163
- dma
7264
min_flash: 40
7365
platform_allow:
@@ -80,7 +72,6 @@ tests:
8072
- DTC_OVERLAY_FILE="boards/esp32s3_devkitc_procpu.overlay"
8173
- EXTRA_CONF_FILE="overlay-dma-esp32.conf"
8274
depends_on:
83-
- adc
8475
- dma
8576
platform_allow:
8677
- esp32s3_devkitc/esp32s3/procpu

tests/drivers/adc/adc_error_cases/testcase.yaml

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,13 +2,10 @@ common:
22
tags:
33
- adc
44
- drivers
5+
depends_on: adc
6+
filter: dt_alias_exists("adc")
57

68
tests:
79
drivers.adc_error_cases:
8-
depends_on: adc
9-
platform_allow:
10+
integration_platforms:
1011
- nrf52840dk/nrf52840
11-
- nrf54l15dk/nrf54l15/cpuapp
12-
- nrf54lm20dk/nrf54lm20a/cpuapp
13-
- nrf54h20dk/nrf54h20/cpuapp
14-
- ophelia4ev/nrf54l15/cpuapp

0 commit comments

Comments
 (0)