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

applications: sdp: mspi: extend MSPI API with reset functions #21076

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

Conversation

magp-nordic
Copy link
Contributor

@magp-nordic magp-nordic commented Mar 20, 2025

Extend MSPI API with custom functions handling reset pins.

Needs changes from #20870 (d033f90 commit), otherwise stack overflow in ICMsg happens.

@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 20, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Mar 20, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 49ed243cc6b9ff8df9a33d96ee564ef25c04c606

more details

sdk-nrf:

PR head: 49ed243cc6b9ff8df9a33d96ee564ef25c04c606
merge base: f9474f069603018f4b8d09a38d19af1e12248ce4
target head (main): 4a662ee237365bd35cf8f140a4706d84121259d3
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 (7)
CODEOWNERS
applications
│  ├── sdp
│  │  ├── mspi
│  │  │  ├── src
│  │  │  │  ├── hrt
│  │  │  │  │  ├── hrt-nrf54l15.s
│  │  │  │  │  │ hrt.c
│  │  │  │  │ main.c
drivers
│  ├── mspi
│  │  │ mspi_nrfe.c
include
│  ├── drivers
│  │  ├── mspi
│  │  │  ├── nrf_mspi.h
│  │  │  │ nrfe_mspi.h

Outputs:

Toolchain

Version:
Build docker image:

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

  • 🟠 Toolchain
  • 🟠 Build twister
  • 🟠 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

@magp-nordic magp-nordic force-pushed the NRFX-7213-handle-reset-gpios-property branch from 3022645 to 98eb5fc Compare March 26, 2025 12:10
Fix setting previous state of not used pins in RX path.

Signed-off-by: Magdalena Pastula <magdalena.pastula@nordicsemi.no>
Extend MSPI API with custom functions handling reset pins.

Signed-off-by: Magdalena Pastula <magdalena.pastula@nordicsemi.no>
@magp-nordic magp-nordic force-pushed the NRFX-7213-handle-reset-gpios-property branch from 796f200 to 3b18859 Compare March 27, 2025 18:50
@magp-nordic magp-nordic marked this pull request as ready for review March 27, 2025 18:52
@magp-nordic magp-nordic requested review from a team as code owners March 27, 2025 18:52
@magp-nordic magp-nordic force-pushed the NRFX-7213-handle-reset-gpios-property branch from 3b18859 to c4e432b Compare March 27, 2025 18:54
@github-actions github-actions bot removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Mar 27, 2025
@magp-nordic magp-nordic force-pushed the NRFX-7213-handle-reset-gpios-property branch 2 times, most recently from 957e5f1 to ce74520 Compare March 27, 2025 19:08
Change cast of opcode in ep_recv function to uint32_t instead
of uint8_t.

Signed-off-by: Magdalena Pastula <magdalena.pastula@nordicsemi.no>
@magp-nordic magp-nordic force-pushed the NRFX-7213-handle-reset-gpios-property branch from ce74520 to 49ed243 Compare March 28, 2025 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants