Skip to content
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.

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

tests: drivers: flash: common: run on SDP MSPI #20542

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

jaz1-nordic
Copy link
Contributor

No description provided.

@jaz1-nordic jaz1-nordic requested review from a team as code owners February 20, 2025 15:47
@github-actions github-actions bot added manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Feb 20, 2025
Copy link

After documentation is built, you will find the preview for this PR here.

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Feb 20, 2025

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@1bf500b nrfconnect/sdk-zephyr#2541 nrfconnect/sdk-zephyr#2541/files

DNM label due to: 1 project with PR revision

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Feb 20, 2025

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 3

Inputs:

Sources:

sdk-nrf: PR head: dba12eb00d932458546f147a06211ff4003c59a3
zephyr: PR head: 979bc8d88aa1f5314e39f5464a3315d0e4e07c0f

more details

sdk-nrf:

PR head: dba12eb00d932458546f147a06211ff4003c59a3
merge base: c2e0a5880f68268b0974e7af56a65c40ee7de635
target head (main): c2e0a5880f68268b0974e7af56a65c40ee7de635
Diff

zephyr:

PR head: 979bc8d88aa1f5314e39f5464a3315d0e4e07c0f
merge base: 1bf500b0c51d8e04336b76d39437d408e3348956
target head (main): 6147494082221c0d8e03f0fc2190f0e1c15db233
Diff

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (17)
applications
│  ├── sdp
│  │  ├── mspi
│  │  │  │ Kconfig
scripts
│  ├── twister
│  │  ├── alt
│  │  │  ├── zephyr
│  │  │  │  ├── tests
│  │  │  │  │  ├── drivers
│  │  │  │  │  │  ├── flash
│  │  │  │  │  │  │  ├── common
│  │  │  │  │  │  │  │  │ testcase.yaml
snippets
│  ├── sdp
│  │  ├── mspi
│  │  │  ├── soc
│  │  │  │  │ nrf54l15_cpuapp.overlay
west.yml
zephyr
│  ├── drivers
│  │  ├── clock_control
│  │  │  │ clock_control_nrf2_fll16m.c
│  │  ├── flash
│  │  │  ├── flash_mspi_nor.c
│  │  │  │ spi_nor.h
│  │  ├── misc
│  │  │  ├── coresight
│  │  │  │  │ nrf_etr.c
│  ├── dts
│  │  ├── bindings
│  │  │  ├── clock
│  │  │  │  │ nordic,nrf-fll16m.yaml
│  │  ├── common
│  │  │  ├── nordic
│  │  │  │  │ nrf54h20.dtsi
│  ├── include
│  │  ├── zephyr
│  │  │  ├── logging
│  │  │  │  ├── log_frontend_stmesp.h
│  │  │  │  │ log_frontend_stmesp_demux.h
│  ├── subsys
│  │  ├── logging
│  │  │  ├── frontends
│  │  │  │  ├── log_frontend_stmesp.c
│  │  │  │  │ log_frontend_stmesp_demux.c
│  ├── tests
│  │  ├── drivers
│  │  │  ├── clock_control
│  │  │  │  ├── nrf_clock_control
│  │  │  │  │  ├── src
│  │  │  │  │  │  │ main.c
│  │  │  ├── flash
│  │  │  │  ├── common
│  │  │  │  │  ├── src
│  │  │  │  │  │  │ main.c
│  │  ├── kernel
│  │  │  ├── usage
│  │  │  │  ├── thread_runtime_stats
│  │  │  │  │  ├── src
│  │  │  │  │  │  │ test_thread_runtime_stats.c

Outputs:

Toolchain

Version: aedb4c0245
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:aedb4c0245_bece0367df

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 508
    • sdk-zephyr test count: 2476
  • ❌ Integration tests
    • ✅ test-sdk-audio
    • ✅ test-fw-nrfconnect-nfc
    • ✅ test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • ✅ test-fw-nrfconnect-nrf-iot_thingy91
    • ✅ test-fw-nrfconnect-rs
    • ✅ test-fw-nrfconnect-fem
    • ✅ test-sdk-find-my
    • ❌ test-low-level
    • ✅ test-sdk-dfu
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-boot
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-nrf-iot_cloud
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_mosh
    • test-fw-nrfconnect-nrf-iot_positioning
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-ps
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread
    • test-fw-nrfconnect-zigbee
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-sidewalk
    • test-sdk-wifi
    • test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

@nordic-piks
Copy link
Contributor

New alt-config for zephyr/tests/drivers/flash/common/testcase.yaml added at CI.

@jaz1-nordic jaz1-nordic force-pushed the nrfx-7100_sdp_mspi_flash_test_fail branch from a599a79 to fb48824 Compare February 25, 2025 13:12
@jaz1-nordic jaz1-nordic requested a review from a team as a code owner February 25, 2025 13:12
@jaz1-nordic jaz1-nordic changed the title drivers: sdp: mspi: fix for stuck in semaphore timeouts tests: drivers: flash: common: run on SDP MSPI Feb 25, 2025
Current memory configuration is SINGLE mode. We need to remove
unused pins in this mode until SDP app supports it properly.
Otherwise it causes problems with reset ans WP pins.

Signed-off-by: Jakub Zymelka <[email protected]>
Increase RX buffer size to be able to run flash/common test.

Signed-off-by: Jakub Zymelka <[email protected]>
Changes needed for SDP MSPI flash tests.

Signed-off-by: Jakub Zymelka <[email protected]>
Add yaml configuration to enable SDP MSPI API test.

Signed-off-by: Jakub Zymelka <[email protected]>
@jaz1-nordic jaz1-nordic force-pushed the nrfx-7100_sdp_mspi_flash_test_fail branch from fb48824 to dba12eb Compare February 26, 2025 13:32
Copy link

You can find the documentation preview for this PR here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. DNM manifest manifest-zephyr
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants