Skip to content

[TI cc23x0 SoC] Add DMA mode to SPI driver #90585

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

Merged

Conversation

jpanisbl
Copy link
Contributor

This series adds DMA mode support to SPI driver for the TI cc23x0 SoC.

Datasheet: https://www.ti.com/lit/ds/symlink/cc2340r5.pdf

jpanisbl added 2 commits May 26, 2025 10:23
Two DMA channels are assigned to TX and RX respectively:
- A TX DMA single request is asserted when there is space in the FIFO.
- A RX DMA single request is asserted when data is in the FIFO.

Signed-off-by: Julien Panis <[email protected]>
Add support to use DMA mode with cc23x0 SPI module. This consists in
specifying the DMA channels and peripherals.

Signed-off-by: Julien Panis <[email protected]>
@github-actions github-actions bot added platform: TI SimpleLink Texas Instruments SimpleLink MCU area: SPI SPI bus labels May 26, 2025
@jpanisbl
Copy link
Contributor Author

jpanisbl commented Jun 6, 2025

@tbursztyka @teburd @vaishnavachath Could you please review when you have time for this ?

vaishnavachath
vaishnavachath previously approved these changes Jun 10, 2025
tbursztyka
tbursztyka previously approved these changes Jun 16, 2025
Copy link
Collaborator

@JarmouniA JarmouniA left a comment

Choose a reason for hiding this comment

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

A test case is needed in 1 of the SPI tests (loopback for ex) so this code gets at least built in CI.

jpanisbl added 2 commits June 17, 2025 10:12
When loopback mode is enabled, the output of transmit serial shifter
is connected to the input of receive serial shifter internally.

Signed-off-by: Julien Panis <[email protected]>
Enable DMA for the test.

Signed-off-by: Julien Panis <[email protected]>
@jpanisbl jpanisbl dismissed stale reviews from tbursztyka and vaishnavachath via 0527e60 June 17, 2025 08:19
Copy link

@jpanisbl
Copy link
Contributor Author

jpanisbl commented Jun 17, 2025

A test case is needed in 1 of the SPI tests (loopback for ex) so this code gets at least built in CI.

@JarmouniA Done (I added 2 new commits: 1 in the driver to enable loopback optionally, and 1 for the test).
I also re-requested review from @vaishnavachath @tbursztyka.
Thank you all for your time.

@jpanisbl
Copy link
Contributor Author

jpanisbl commented Jun 25, 2025

Hi @vaishnavachath @tbursztyka Could you please review this PR ? Nothing has changed in the 2 commits you approved 2 weeks ago. Only 2 new commits (XS size) were added, and they are now approved by JarmouniA who requested the SPI test.

@kartben kartben merged commit a631df5 into zephyrproject-rtos:main Jun 27, 2025
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: SPI SPI bus platform: TI SimpleLink Texas Instruments SimpleLink MCU
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants