rp-hal icon indicating copy to clipboard operation
rp-hal copied to clipboard

DMA on the way!

Open AsafFisher opened this issue 4 years ago • 14 comments

Implement the DMA api for all the peripherals :memo:

AsafFisher avatar Feb 02 '21 22:02 AsafFisher

Things I had on my list for the DMA:

  • Example: https://docs.rs/stm32f1xx-hal/0.7.0/stm32f1xx_hal/dma/index.html
  • Traits: https://crates.io/crates/embedded-dma

tdittr avatar Feb 03 '21 10:02 tdittr

Thank you for the guidance! I will look in to it (:

AsafFisher avatar Feb 03 '21 13:02 AsafFisher

I dont like https://crates.io/crates/embedded-dma's traits why should I follow them?

AsafFisher avatar Mar 11 '21 17:03 AsafFisher

Well, I guess it would allow more interoperation, similar to the embedded-hal traits.

tdittr avatar Mar 11 '21 20:03 tdittr

What is the status of this implementation?

maximeborges avatar Nov 10 '21 15:11 maximeborges

@maximeborges The branch where this work took place is https://github.com/rp-rs/rp-hal/tree/rp-dma It has not been touched in 7 months. I would call the status of this implementation "abandoned". Use it as a starting point if you wish, but if you a need for DMA I suggest starting a new branch and working on it rather than waiting for this to be finished.

9names avatar Nov 13 '21 09:11 9names

See #209 for a more complete take on DMA which is already usable.

mgottschlag avatar Dec 15 '21 11:12 mgottschlag

How is it going? Is there any working on progress?

lightsing avatar Aug 21 '22 15:08 lightsing

No progress

9names avatar Aug 22 '22 07:08 9names

@AsafFisher any news on this one?

eloraiby avatar Sep 22 '22 23:09 eloraiby

AsafFisher has not been active on this project in over a year. No progress to report

9names avatar Sep 23 '22 00:09 9names

I have been using the PR #209 in my project (details in this comment: https://github.com/rp-rs/rp-hal/pull/209#issuecomment-1156582772 ) I have rebased the change to the recent branch in https://github.com/slint-ui/rp-hal/tree/dma-0.6

ogoffart avatar Sep 23 '22 06:09 ogoffart

I did a rebase of your rebase earlier today. https://github.com/9names/rp-hal/tree/dma-0.6 The plan was going to write + run some tests to see what works, maybe clean up the implementation and write some docs

9names avatar Sep 23 '22 06:09 9names

That would be nice to see that merged. I also tried to upgrade to embedded_dma 0.2 in https://github.com/slint-ui/rp-hal/tree/embedded_dma2 (which i just pushed now), but i haven't tried it yet.

ogoffart avatar Sep 23 '22 07:09 ogoffart

Closed via #457

9names avatar Jun 24 '23 07:06 9names