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

drivers: mspi: mspi_nrfe: cleanup #20447

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

Conversation

masz-nordic
Copy link
Contributor

@masz-nordic masz-nordic commented Feb 18, 2025

This commit:

  • removes support for CONFIG_MULTITHREADING=n
  • removes support for CONFIG_SYS_CLOCK_EXISTS=n
  • removes unnecessary decriptions for functions
  • renames functions to make them more descriptive and remove namespacing where unnecessary
  • removes arguments that where unnecessary
  • moves functions around to make it more readable
  • removes code for CONFIG_PM_DEVICE
  • other minor changes

@masz-nordic masz-nordic requested review from a team as code owners February 18, 2025 11:26
@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 Feb 18, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Feb 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: e00803accdb848f6144c9c4b1d004219aad3fa22

more details

sdk-nrf:

PR head: e00803accdb848f6144c9c4b1d004219aad3fa22
merge base: 3b3ec99f1b13330b624ed2d68d8682d8102c20a8
target head (main): 834a4fd3a5d44c078743ea9b7fa08645525e6078
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 (2)
.github
│  │ test-spec.yml
drivers
│  ├── mspi
│  │  │ mspi_nrfe.c

Outputs:

Toolchain

Version: 0f22b642ed
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:0f22b642ed_bbe5b33786

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 35
  • ✅ 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_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-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

This commit:
- removes support for `CONFIG_MULTITHREADING=n`
- removes support for `CONFIG_SYS_CLOCK_EXISTS=n`
- removes unnecessary decriptions for functions
- renames functions to make them more descriptive
  and remove namespacing where unnecessary
- removes arguments where unnecessary
- moves functions around to make it more readable
- removes code for `CONFIG_PM_DEVICE`
- other minor changes

Signed-off-by: Marcin Szymczyk <[email protected]>
@masz-nordic masz-nordic added this to the 3.0.0 milestone Feb 18, 2025
Update directories for SDP.

Signed-off-by: Marcin Szymczyk <[email protected]>
@masz-nordic masz-nordic requested a review from a team as a code owner February 18, 2025 12:26
@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 Feb 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants