search

rust-embedded-community / tm4c-hal

An Embedded HAL and general chip support for the TM4C123/LM4F120. Replaces the old lm4f120 crate.
Apache License 2.0
40 stars 26 forks source link

Add ADC support #11

Open liamdiprose opened 5 years ago

liamdiprose commented 5 years ago

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?

thejpster commented 5 years ago

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 commented 4 years ago

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

liamdiprose commented 4 years ago

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?

vipierozan99 commented 3 years ago

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