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: gpio: gpio_more_loops: Enable test on 54L* cpuflpr #21012

Conversation

nordic-segl
Copy link
Contributor

Add overlay files required to run the test on the FLPR core of nRF54L15, nRF54L09, nRF54L20.

@nordic-segl nordic-segl requested review from a team as code owners March 18, 2025 12:54
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Mar 18, 2025
Copy link
Contributor

@masz-nordic masz-nordic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rename common overlays

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Mar 18, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 746b467bfe6a31dbd8f3721db090d658fd0a9357

more details

sdk-nrf:

PR head: 746b467bfe6a31dbd8f3721db090d658fd0a9357
merge base: 425087097f238e696b8b1a8e667f9558f2dfdea2
target head (main): 622e086912a9f013a2e42ad56cb77927711f8af9
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 (10)
tests
│  ├── drivers
│  │  ├── gpio
│  │  │  ├── gpio_more_loops
│  │  │  │  ├── boards
│  │  │  │  │  ├── nrf54l09pdk_nrf54l09_common.dtsi
│  │  │  │  │  ├── nrf54l09pdk_nrf54l09_cpuapp.overlay
│  │  │  │  │  ├── nrf54l09pdk_nrf54l09_cpuflpr.overlay
│  │  │  │  │  ├── nrf54l15dk_nrf54l15_common.dtsi
│  │  │  │  │  ├── nrf54l15dk_nrf54l15_cpuapp.overlay
│  │  │  │  │  ├── nrf54l15dk_nrf54l15_cpuflpr.overlay
│  │  │  │  │  ├── nrf54l20pdk_nrf54l20_common.dtsi
│  │  │  │  │  ├── nrf54l20pdk_nrf54l20_cpuapp.overlay
│  │  │  │  │  │ nrf54l20pdk_nrf54l20_cpuflpr.overlay
│  │  │  │  │ testcase.yaml

Outputs:

Toolchain

Version: 4ffa2202d5
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:4ffa2202d5_e579f9fbe6

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister - Skipped: Skipping Build & Test as it succeeded in a previous run: 2
  • ✅ Integration tests
    • ✅ test-low-level
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-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_cloud
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • 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-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-ps
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread
    • test-fw-nrfconnect-zigbee
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • 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-segl
Copy link
Contributor Author

Rename common overlays

L09 FLPR includes L09 APP (so overlay for APP is common for both APP and FLPR on nrf54l09).
L15 FLPR includes L15 APP (so overlay for APP is common for both APP and FLPR on nrf54l15).
L20 FLPR includes L20 APP (so overlay for APP is common for both APP and FLPR on nrf54l20).

I can create common for L15 and include it without changes by both cpuapp and cpuflpr.
Same for L09 and L20.

Is this what You intended?

@nordic-segl nordic-segl requested a review from masz-nordic March 19, 2025 12:26
Add overlay files required to run the test on the FLPR core of
nRF54L15, nRF54L09, nRF54L20.

Signed-off-by: Sebastian Głąb <sebastian.glab@nordicsemi.no>
@nordic-segl nordic-segl force-pushed the NRFX-7216_Enable-gpio_more_loops-on-cpuflpr branch from e34610f to 746b467 Compare March 19, 2025 13:40
@nordic-segl
Copy link
Contributor Author

nordic-segl commented Mar 19, 2025

14:46:36  INFO    - 1/2 nrf54l20pdk/nrf54l20/cpuapp drivers.gpio.gpio_more_loops                       PASSED (device: 821013703, 11.533s <zephyr>)
14:47:02  INFO    - 2/2 nrf54l20pdk/nrf54l20/cpuflpr drivers.gpio.gpio_more_loops                       PASSED (device: 821013703, 12.979s <zephyr>)
14:45:24  INFO    - 1/1 nrf54l09pdk/nrf54l09/cpuapp drivers.gpio.gpio_more_loops                       PASSED (device: 175104789, 11.573s <zephyr>)
14:49:07  INFO    - 1/2 nrf54l15dk/nrf54l15/cpuapp drivers.gpio.gpio_more_loops                       PASSED (device: 1057735411, 12.432s <zephyr>)
14:49:16  INFO    - 2/2 nrf54l15dk/nrf54l15/cpuflpr drivers.gpio.gpio_more_loops                       PASSED (device: 1057793721, 13.789s <zephyr>)

@masz-nordic masz-nordic merged commit d0c3d7f into nrfconnect:main Mar 21, 2025
13 checks passed
@nordic-segl nordic-segl deleted the NRFX-7216_Enable-gpio_more_loops-on-cpuflpr branch March 21, 2025 11:21
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants