Skip to content

drivers: spi: spi_loopback: drivers.spi.* fail #90318

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
katgiadla opened this issue May 22, 2025 · 3 comments · May be fixed by #90321
Open

drivers: spi: spi_loopback: drivers.spi.* fail #90318

katgiadla opened this issue May 22, 2025 · 3 comments · May be fixed by #90321
Assignees
@nika-nordic
Labels
area: SPI SPI bus bug The issue is a bug, or the PR is fixing a bug platform: nRF Nordic nRFx priority: low Low impact/importance bug

Comments

@katgiadla
Copy link
Collaborator

Describe the bug
Tests:

  • tests/drivers/spi/spi_loopback/drivers.spi.nrf_pm_runtime
  • tests/drivers/spi/spi_loopback/drivers.spi.loopback
  • tests/drivers/spi/spi_loopback/drivers.spi.nrf54l_32mhz
  • tests/drivers/spi/spi_loopback/drivers.spi.nrf54l_16mhz
  • tests/drivers/spi/spi_loopback/drivers.spi.nrf54l_8mhz

fail.

Observed for nrf54l15dk/nrf54l15/cpuapp

To Reproduce
Steps to reproduce the behavior:

  1. have nrf54l15dk/nrf54l15/cpuapp connected
  2. connect jump wire like in overlay
  3. go to your zephyr dir
  4. call ./scripts/twister -T tests/subsys/debug/cpu_load -p nrf54l15dk/nrf54l15/cpuapp --device-testing --device-serial /dev/ttyACM1 -v --inline-logs --west-flash="--erase" --fixture spi_loopback
  5. See console output with error
    Expected behavior
    Valid console output

Impact
Not clear

Logs and console output

===================================================================
START - test_spi_word_size_16
E: Word sizes other than 8 bits are not supported
Spi config invalid for this controller - skip

    Assertion failed at zephyr/tests/drivers/spi/spi_loopback/src/spi.c:628: spi_loopback_test_word_size: (memcmp(compare_data, rx_buffer, buffer_size) is true)
16-bit word buffer contents are different
 FAIL - test_spi_word_size_16 in 0.029 seconds
===================================================================
START - test_spi_word_size_24
E: Word sizes other than 8 bits are not supported
Spi config invalid for this controller - skip

    Assertion failed at zephyr/tests/drivers/spi/spi_loopback/src/spi.c:628: spi_loopback_test_word_size: (memcmp(compare_data, rx_buffer, buffer_size) is true)
24-bit word buffer contents are different
 FAIL - test_spi_word_size_24 in 0.029 seconds
===================================================================
START - test_spi_word_size_32
E: Word sizes other than 8 bits are not supported
Spi config invalid for this controller - skip

    Assertion failed at zephyr/tests/drivers/spi/spi_loopback/src/spi.c:628: spi_loopback_test_word_size: (memcmp(compare_data, rx_buffer, buffer_size) is true)
32-bit word buffer contents are different
 FAIL - test_spi_word_size_32 in 0.029 seconds
===================================================================
...
===================================================================
START - test_spi_word_size_9
E: Word sizes other than 8 bits are not supported
Spi config invalid for this controller - skip

    Assertion failed at zephyr/tests/drivers/spi/spi_loopback/src/spi.c:628: spi_loopback_test_word_size: (memcmp(compare_data, rx_buffer, buffer_size) is true)
9-bit word buffer contents are different
 FAIL - test_spi_word_size_9 in 0.029 seconds
===================================================================
...

Environment (please complete the following information):

  • OS: Ubuntu 20.04.1 LTS
  • Toolchain Zephyr SDK 0.17.0
  • Commit SHA or Version used: f47f3fe
@katgiadla katgiadla added bug The issue is a bug, or the PR is fixing a bug priority: low Low impact/importance bug area: SPI SPI bus platform: nRF Nordic nRFx labels May 22, 2025
@github-project-automation github-project-automation bot moved this to To triage in nRF platform May 22, 2025
@nika-nordic
Copy link
Collaborator

@bjarki-andreasen I see you have recently applied some changes to Nordic SPI driver: #90047 . Any chance they introduced regression to L15 ?

@bjarki-andreasen
Copy link
Collaborator

Very unlikely, there was a PR which refactored and added new tests cases for different word sizes, so its probably just something that has always been broken :)

@bjarki-andreasen
Copy link
Collaborator

bjarki-andreasen commented May 22, 2025 •
edited
Loading

#87838 fixes the test suite, the driver works as expected, the test suite has had a new test added which did not properly skip unsupported spi configurations :)

@bjarki-andreasen bjarki-andreasen linked a pull request May 22, 2025 that will close this issue
Open
@nordic-piks nordic-piks moved this from To triage to Backlog in nRF platform May 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nika-nordic nika-nordic

Labels
area: SPI SPI bus bug The issue is a bug, or the PR is fixing a bug platform: nRF Nordic nRFx priority: low Low impact/importance bug
Projects
nRF platform
Status: Backlog
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

tests: drivers: spi: spi_loopback: skip tests if invalid config bjarki-andreasen/zephyr
3 participants
@nika-nordic @katgiadla @bjarki-andreasen