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

Nrfx 6118 add icbmsg to egpio #17321

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

Conversation

magp-nordic
Copy link
Contributor

Based on #16592, please, review only the last 5 commits.

@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 Sep 13, 2024
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Sep 13, 2024

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@8005d4e nrfconnect/sdk-zephyr#2003 nrfconnect/sdk-zephyr#2003/files

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Sep 13, 2024

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 6ef095265b8dda221f29d24864c57daa063a75ff
zephyr: PR head: ed9919efda39eb692a8f78f601bdc4d60332b0b7

more details

sdk-nrf:

PR head: 6ef095265b8dda221f29d24864c57daa063a75ff
merge base: 547aaab19c52cc03660d57943a963480a042e14f
target head (main): f7258d0ad63ff6f736b08f60b9d4c2e28398de68
Diff

zephyr:

PR head: ed9919efda39eb692a8f78f601bdc4d60332b0b7
merge base: 8005d4e87b9bcba25f441a51f4f5d29b541022b9
target head (main): 3f835fec4f01a10a055c23760f6a0f40aa77eaab
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 (18)
applications
│  ├── sdp
│  │  ├── gpio
│  │  │  ├── CMakeLists.txt
│  │  │  ├── boards
│  │  │  │  │ nrf54l15dk_nrf54l15_cpuflpr_icbmsg.overlay
│  │  │  ├── include
│  │  │  │  │ main.h
│  │  │  │ sample.yaml
cmake
│  ├── sysbuild
│  │  │ sdp.cmake
drivers
│  ├── gpio
│  │  ├── CMakeLists.txt
│  │  ├── Kconfig
│  │  │ gpio_nrfe.h
scripts
│  ├── twister
│  │  ├── alt
│  │  │  ├── zephyr
│  │  │  │  ├── samples
│  │  │  │  │  ├── basic
│  │  │  │  │  │  ├── blinky
│  │  │  │  │  │  │  │ sample.yaml
snippets
│  ├── emulated-gpio
│  │  ├── icbmsg
│  │  │  ├── boards
│  │  │  │  │ nrf54l15dk_nrf54l15_cpuapp.overlay
│  │  │  ├── emulated-gpio.overlay
│  │  │  │ snippet.yml
sysbuild
│  │ Kconfig.sdp
west.yml
zephyr
│  ├── samples
│  │  ├── subsys
│  │  │  ├── ipc
│  │  │  │  ├── ipc_service
│  │  │  │  │  ├── icmsg
│  │  │  │  │  │  ├── boards
│  │  │  │  │  │  │  │ nrf54l15pdk_nrf54l15_cpuapp_icbmsg.overlay
│  │  │  │  │  │  ├── remote
│  │  │  │  │  │  │  ├── boards
│  │  │  │  │  │  │  │  │ nrf54l15pdk_nrf54l15_cpuflpr_icbmsg.overlay
│  │  │  │  │  │  │ sample.yaml
│  ├── subsys
│  │  ├── ipc
│  │  │  ├── ipc_service
│  │  │  │  ├── backends
│  │  │  │  │  │ ipc_icbmsg.c

Outputs:

Toolchain

Version: 9583beca34
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:9583beca34_81ed5a52d6

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 1473
    • sdk-zephyr test count: 5652
  • ❌ Integration tests
    • ✅ test-fw-nrfconnect-boot
    • ❌ test-fw-nrfconnect-nfc
    • ✅ test-fw-nrfconnect-rpc
    • ❌ test-fw-nrfconnect-rs
    • ❌ test-fw-nrfconnect-fem
    • ✅ test-sdk-find-my
    • ✅ test-low-level
    • ✅ test-sdk-mcuboot
    • ✅ 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-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_nrf_provisioning
    • 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-tfm
    • test-fw-nrfconnect-thread
    • test-fw-nrfconnect-zigbee
    • test-sdk-audio
    • 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

cmake/sysbuild/sdp.cmake Outdated Show resolved Hide resolved
cmake/sysbuild/sdp.cmake Outdated Show resolved Hide resolved
cmake/sysbuild/sdp.cmake Outdated Show resolved Hide resolved
cmake/sysbuild/sdp.cmake Outdated Show resolved Hide resolved
Update sdk-zephyr revision with icbmsg alignment to
non-multithreading.

Signed-off-by: Magdalena Pastula <[email protected]>
@magp-nordic magp-nordic requested review from a team as code owners October 3, 2024 18:24
@magp-nordic
Copy link
Contributor Author

magp-nordic commented Oct 3, 2024

Now based on #17256 (first commit)
Please, do not review yet, needs some alignment after #16592 was merged.

@NordicBuilder
Copy link
Contributor

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publishing GitHub Action.

Copy link
Contributor

@jaz1-nordic jaz1-nordic left a comment

Choose a reason for hiding this comment

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

nitpicks. Additionally please resolve complaints from CI.

snippets/emulated-gpio/icbmsg/snippet.yml Outdated Show resolved Hide resolved
Add icbmsg as a possible backend for eGPIO.

Signed-off-by: Magdalena Pastula <[email protected]>
Add icbmsg as possible backend for eGPIO.

Signed-off-by: Magdalena Pastula <[email protected]>
Add icbmsg as possible backend for eGPIO.

Signed-off-by: Magdalena Pastula <[email protected]>
Add icbmsg as possible backend for eGPIO.

Signed-off-by: Magdalena Pastula <[email protected]>
Add option of ICBMSG as a backend for eGPIO.

Signed-off-by: Magdalena Pastula <[email protected]>
Add eGPIO testcase with icbmsg backend.

Signed-off-by: Magdalena Pastula <[email protected]>
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.

4 participants