Skip to content

Commit 49bcafc

Browse files
mnkpMaureenHelm
authored andcommitted
tests: remove pinmux_basic_api test
The pinmux_basic_api test relies on a hardware loopback, manual shorting of two GPIO pins. The Zephyr test framework does not allow currently to define such pins in a generic way. The pinmux_basic_api test hard codes pin numbers specific to a few evaluation boards. The test has a few more flaws and limitations: - it verifies that pin configured to function A can be controlled by GPIO driver. It doesn't verify that pin configured to function B can be contorolled by a corresponding peripheral driver. - the test relies on level sensitive interrupt support which is not always available. - the test will pass even if there are erros when pin is configured to function B. - the test allows to configure both test pins as an output therefore shorting the outputs. Considering the flaws and limited coverage of the test as well as missing features of the Zephyr test framework this commit removes the testcase. It is not currently possible to write a generic pinmux testcase that supports multiple boards and can be used to assess the quality of the driver. Instead, to ensure the driver code will not degrade, we need to rely on implicit testing done by the board initialization code located in boards/ folder. Signed-off-by: Piotr Mienkowski <[email protected]>
1 parent c793c0e commit 49bcafc

File tree

5 files changed

+0
-164
lines changed

5 files changed

+0
-164
lines changed

tests/drivers/pinmux/pinmux_basic_api/CMakeLists.txt

Lines changed: 0 additions & 8 deletions
This file was deleted.

tests/drivers/pinmux/pinmux_basic_api/prj.conf

Lines changed: 0 additions & 3 deletions
This file was deleted.

tests/drivers/pinmux/pinmux_basic_api/src/main.c

Lines changed: 0 additions & 17 deletions
This file was deleted.

tests/drivers/pinmux/pinmux_basic_api/src/pinmux_gpio.c

Lines changed: 0 additions & 130 deletions
This file was deleted.

tests/drivers/pinmux/pinmux_basic_api/testcase.yaml

Lines changed: 0 additions & 6 deletions
This file was deleted.

0 commit comments

Comments
 (0)