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

[dma] Port DMA tests from integrated_dev branch #26348

Merged
merged 6 commits into from
Feb 20, 2025

Conversation

jwnrt
Copy link
Contributor

@jwnrt jwnrt commented Feb 18, 2025

I've cherry picked the commits but dropped the changes to DV as that's not currently working for DJ on master.

I've only made the minimal changes required to get this compiling, I haven't yet ported the tests to device-tables.

…functions

When using the DMA in hardware handshake mode, we need to configure the SPI host first
and then let the DMA read or write the FIFOs. This PR splits up the single function
that configures the DMA and accesses the FIFOs into two, without changing the API
of this DIF.

Signed-off-by: Robert Schilling <[email protected]>
(cherry picked from commit cec5b06)
@jwnrt jwnrt requested review from pamaury and Razer6 February 18, 2025 14:48
@jwnrt jwnrt requested a review from a team as a code owner February 18, 2025 14:48
@jwnrt jwnrt force-pushed the dma-inline-hashing branch from c4ca5a9 to 4e3242c Compare February 18, 2025 15:00
@jwnrt jwnrt changed the title [dma] Port dma_inline_hashing test from integrated_dev branch [dma] Port DMA tests from integrated_dev branch Feb 18, 2025
@jwnrt
Copy link
Contributor Author

jwnrt commented Feb 18, 2025

I've added the dma_abort test as well since the cherry-pick follows on easily.

@jwnrt jwnrt force-pushed the dma-inline-hashing branch from 4e3242c to 890ebdd Compare February 18, 2025 15:07
@jwnrt jwnrt force-pushed the dma-inline-hashing branch 4 times, most recently from a5a2399 to b71252f Compare February 18, 2025 16:32
@jwnrt jwnrt requested review from nbdd0121 and AlexJones0 and removed request for a team February 18, 2025 16:39
Razer6 and others added 2 commits February 19, 2025 11:57
Co-authored-by: James Wainwright <[email protected]>
Signed-off-by: Robert Schilling <[email protected]>
Co-authored-by: James Wainwright <[email protected]>
Signed-off-by: Robert Schilling <[email protected]>
(cherry picked from commit f012ecf)
@jwnrt jwnrt force-pushed the dma-inline-hashing branch from b71252f to a777b13 Compare February 19, 2025 11:57
@pamaury
Copy link
Contributor

pamaury commented Feb 19, 2025

I would suggest to port it to the DT as part of the same PR but I leave that up to your appreciation.

@jwnrt jwnrt merged commit 1483fa4 into lowRISC:master Feb 20, 2025
41 of 54 checks passed
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.

4 participants