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: zephyr: drivers: i2c: i2c_target_api: Enable original test on … #21179

Merged

Conversation

nordic-segl
Copy link
Contributor

…L20 APP

Include:
tests: drivers: i2c: i2c_target_api: Enable test on nRF54L20 cpuapp
nrfconnect/sdk-zephyr#2671
zephyrproject-rtos/zephyr#87727

Disable nrf.extended variant of the i2c_target_api test on nrf54l20pdk/nrf54l20/cpuapp.

@nordic-segl nordic-segl requested a review from a team as a code owner March 27, 2025 07:17
@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 27, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Mar 27, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 9234313d35a4fc3ab3bca5a3debfd7fe99c6e8d2

more details

sdk-nrf:

PR head: 9234313d35a4fc3ab3bca5a3debfd7fe99c6e8d2
merge base: d99cc8e41850c64279235154dc7e3492adc54bba
target head (main): d99cc8e41850c64279235154dc7e3492adc54bba
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 (3)
tests
│  ├── zephyr
│  │  ├── drivers
│  │  │  ├── i2c
│  │  │  │  ├── i2c_target_api
│  │  │  │  │  ├── boards
│  │  │  │  │  │  ├── nrf54l20pdk_nrf54l20_cpuapp.conf
│  │  │  │  │  │  │ nrf54l20pdk_nrf54l20_cpuapp.overlay
│  │  │  │  │  │ testcase.yaml

Outputs:

Toolchain

Version: 7dbe0e2658
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:7dbe0e2658_8bf7ca4353

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 29
  • ✅ 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_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-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-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-wifi
    • test-secdom-samples-public

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

@nordic-segl nordic-segl force-pushed the NRFX-6832_Enable-i2c-tests-on-L20-cpuapp branch from 18d870d to 136d90f Compare March 27, 2025 07:22
@nordic-segl nordic-segl force-pushed the NRFX-6832_Enable-i2c-tests-on-L20-cpuapp branch from 136d90f to 6a03fb7 Compare March 27, 2025 08:17
@nordic-segl nordic-segl requested review from a team as code owners March 27, 2025 08:17
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Mar 27, 2025

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

Name Old Revision New Revision Diff

All manifest checks OK

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

@nordic-segl
Copy link
Contributor Author

09:30:57 INFO - 1/1 nrf54l20pdk/nrf54l20/cpuapp tests/drivers/i2c/i2c_target_api/drivers.i2c.target_api.single_role PASSED (device: 851006630, 3.197s )

Copy link

You can find the documentation preview for this PR here.

@nordic-segl nordic-segl force-pushed the NRFX-6832_Enable-i2c-tests-on-L20-cpuapp branch 3 times, most recently from 47e0ab0 to 53be2cb Compare April 1, 2025 06:11
@nordic-segl nordic-segl force-pushed the NRFX-6832_Enable-i2c-tests-on-L20-cpuapp branch from 53be2cb to a0eb97f Compare April 2, 2025 07:13
@NordicBuilder NordicBuilder removed the DNM label Apr 2, 2025
@nordic-segl nordic-segl force-pushed the NRFX-6832_Enable-i2c-tests-on-L20-cpuapp branch from a0eb97f to 4aa8f6e Compare April 2, 2025 11:22
… APP

Include:
tests: drivers: i2c: i2c_target_api: Enable test on nRF54L20 cpuapp
nrfconnect/sdk-zephyr#2671
zephyrproject-rtos/zephyr#87727

Disable nrf.extended variant of the i2c_target_api test on
nrf54l20pdk/nrf54l20/cpuapp.

Signed-off-by: Sebastian Głąb <sebastian.glab@nordicsemi.no>
@nordic-segl nordic-segl force-pushed the NRFX-6832_Enable-i2c-tests-on-L20-cpuapp branch from 4aa8f6e to 9234313 Compare April 3, 2025 07:02
@nordic-segl
Copy link
Contributor Author

Sdk-zephyr was updated in other PR.
This PR, removes nrf.extended variant of i2c test that is no longer needed (original test in sdk-zephyr is run instead).

@rlubos rlubos merged commit 0856720 into nrfconnect:main Apr 3, 2025
14 checks passed
@nordic-segl nordic-segl deleted the NRFX-6832_Enable-i2c-tests-on-L20-cpuapp branch April 3, 2025 08:55
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.

7 participants