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

Published 20 hours ago • verified publisher icon rust-embedded-community

Add ADC support

Open liamdiprose opened this issue 6 years ago • 4 comments

Opening this issue to discuss the addition of ADC support.

Just as stated in my other issue #10 , I have an upcoming project that aims to process audio using Rust. For this project, I will need ADC support for audio input.

I am happy to add support for the ADC's, but unlike the PWM support, I do not know how much work is required.

Are there any points we need to discuss concerning the addition of ADC support?

liamdiprose avatar Aug 01 '18 10:08 liamdiprose

As with PWM, if there's a HAL, we should use it. If there isn't a HAL, we should propose one (keeping it in this tree for now, until it is accepted/merged up-stream).

I haven't had need for an ADC so I'm open to suggestions on the API.

thejpster avatar Aug 12 '18 20:08 thejpster

So, nrf52-hal has an API for their saadc peripheral. We could crib that?

thejpster avatar Nov 10 '19 16:11 thejpster

Hey, thanks for pinging me on this. I've just started getting back into Rust / Embedded.

Just so we're clear, by API you're talking about the Saadc::new function and SaadcExt trait?

liamdiprose avatar Nov 10 '19 21:11 liamdiprose

Hey, I`ve done some tinkering with the adc module, it´s very early tho. Suggestions and directions are welcome, there is a pull request #46

vipierozan99 avatar Jan 05 '21 14:01 vipierozan99