Skip to content

RP2 DMA Class - #10704

Closed
markb139 wants to merge 11 commits into
micropython:masterfrom
markb139:rp2_dma_release2
Closed

markb139 wants to merge 11 commits into
micropython:masterfrom
markb139:rp2_dma_release2

Conversation

@markb139

@markb139 markb139 commented Feb 9, 2023

Copy link
Copy Markdown

Hi,
Hope I haven't caused chaos trying to create this PR.
I've written a DMA class for the RP2040 microcontroller.
The class handles 3 scenarios at present

  1. write from a buffer to a peripheral
  2. 
read to a buffer from a peripheral
  3. 
copy one buffer to another


Transfers can happen at full bus speed or be paced by a DREQ channel.

This code includes a DMA timer class for pacing transfers at a fixed rate - see example code.


The methods can be given an optional handler for callback when a transfer completes when an IRQ is raised. Which IRQ is raised defaults to IRQ0, but can be passed in with the irq parameter.

Regards

Mark

@samskiter

Copy link
Copy Markdown

Would be amazing to see this merged!

@markb139

markb139 commented Oct 2, 2023

Copy link
Copy Markdown
Author

Hi,
I've been away from the Micropython world and the RP2040 over the summer.
I plan to update my project that makes use of the DMA code.

Looks like I have a couple of merge conflicts to fix

Regards

@samskiter

Copy link
Copy Markdown

I've

Hi, I've been away from the Micropython world and the RP2040 over the summer. I plan to update my project that makes use of the DMA code.

Looks like I have a couple of merge conflicts to fix

Regards

Might have you sorted there... markb139#1

@samskiter

Copy link
Copy Markdown

Just to let you know there's issues with the documentation and the stubs

Josverl/micropython-stubs#727

@samskiter

Copy link
Copy Markdown

@markb139 just FYI, there's a competing PR: #7641 can you evaluate and compare?

@markb139

markb139 commented Nov 3, 2023

Copy link
Copy Markdown
Author

I'll take a look.
I'm currently Pi-less as my Pi's are doing other things and my Pi5 hasn't arrived :)

@dpgeorge

Copy link
Copy Markdown
Member

Thanks for the contribution, but I'll close this in favour of the implementation made in #7641 (which is now merged).

@dpgeorge dpgeorge closed this Dec 22, 2023
@markb139

Copy link
Copy Markdown
Author

No problem.

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.

3 participants