Skip to content

Conversation

@magp-nordic
Copy link
Contributor

@magp-nordic magp-nordic commented Jan 27, 2025

Added path for receiving data on SPI (SINGLE mode only, there will be another PR for QUAD).
For now, data is transceived byte by byte, and receiving starts right after transmission. This has its disadvantages, for example:

  • INB register needs to be read when the transmission is still ongoing, otherwise, the clock will not be generated when a peripheral device is sending data (not sure why)
  • Because data is transceived in bytes, there is less time between writing to OUTB and reading from INB. Although I checked that the current solution works for higher frequencies, it can still be a problem when processing a lot of requests from the APP.

There will be another ticket for fixing the above issues. Right now I am adding this so that we would have at least some working solution, even if it's not ideal.

Requires changes from #20114 to be able to receive data from nRF54L15 DK external flash.

@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 Jan 27, 2025
@magp-nordic magp-nordic force-pushed the NRFX-6706-tx-rx-with-dk-flash branch from 658de91 to 1970b8f Compare January 27, 2025 15:34
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Jan 27, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: bf4777d92e9c0eb010a0b715f202cea7f8ab7dc1

more details

sdk-nrf:

PR head: bf4777d92e9c0eb010a0b715f202cea7f8ab7dc1
merge base: 1758d0b850a68d342b6aecb7b518e3e4b91b0ac9
target head (main): 61f1dbc8f163cc5131b69cad148971ee072e7671
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 (6)
applications
│  ├── sdp
│  │  ├── mspi
│  │  │  ├── Kconfig
│  │  │  ├── src
│  │  │  │  ├── hrt
│  │  │  │  │  ├── hrt-nrf54l15.s
│  │  │  │  │  ├── hrt.c
│  │  │  │  │  │ hrt.h
│  │  │  │  │ main.c
drivers
│  ├── mspi
│  │  │ mspi_nrfe.c

Outputs:

Toolchain

Version: 342151af73
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:342151af73_bbe5b33786

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

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

@masz-nordic masz-nordic changed the title Nrfx 6706 TX RX with DK flash applications: sdp: mspi: RX Jan 27, 2025
@NordicBuilder
Copy link
Contributor

You can find the documentation preview for this PR at this link.

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

@magp-nordic magp-nordic force-pushed the NRFX-6706-tx-rx-with-dk-flash branch 2 times, most recently from e0d55f3 to 43c7343 Compare January 28, 2025 17:23
@magp-nordic magp-nordic marked this pull request as ready for review January 28, 2025 17:23
@magp-nordic magp-nordic requested review from a team as code owners January 28, 2025 17:23
@masz-nordic masz-nordic added this to the 3.0.0 milestone Jan 29, 2025
@magp-nordic magp-nordic force-pushed the NRFX-6706-tx-rx-with-dk-flash branch 3 times, most recently from ea7cde6 to 30de4a6 Compare January 29, 2025 14:49
@magp-nordic
Copy link
Contributor Author

@nordicjm I applied your suggestions. I also moved changes to snippets to another PR (#20114), since they are not needed for RX implementation, only for checking it.

@magp-nordic magp-nordic force-pushed the NRFX-6706-tx-rx-with-dk-flash branch 2 times, most recently from 303a260 to 07c8ad5 Compare January 29, 2025 16:06
Copy link
Contributor

@mif1-nordic mif1-nordic left a comment

Choose a reason for hiding this comment

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

Please rebase on #19877 as it will be merged first

@magp-nordic magp-nordic force-pushed the NRFX-6706-tx-rx-with-dk-flash branch 2 times, most recently from 79afd53 to eac0063 Compare February 4, 2025 16:09
@magp-nordic magp-nordic force-pushed the NRFX-6706-tx-rx-with-dk-flash branch 3 times, most recently from 1282342 to 8fb77a9 Compare February 5, 2025 14:28
@magp-nordic
Copy link
Contributor Author

PR rebased after #19877 was merged, @masz-nordic @mif1-nordic , please, review again.

Fixed hard fault that occurs when receiving data caused by
memory access violation and changed logging so that all
received data is printed, not only an opcode.

Signed-off-by: Magdalena Pastula <[email protected]>
Add path for receiving data on SPI.

Signed-off-by: Magdalena Pastula <[email protected]>
@magp-nordic magp-nordic force-pushed the NRFX-6706-tx-rx-with-dk-flash branch from 8fb77a9 to bf4777d Compare February 6, 2025 16:43
@masz-nordic masz-nordic merged commit 61b8072 into nrfconnect:main Feb 7, 2025
13 checks passed
@magp-nordic magp-nordic deleted the NRFX-6706-tx-rx-with-dk-flash branch February 10, 2025 09:03
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.

5 participants