diff options
| author | xoviat <[email protected]> | 2021-03-17 21:24:04 -0500 |
|---|---|---|
| committer | GitHub <[email protected]> | 2021-03-17 21:24:04 -0500 |
| commit | f43813c42018cc33e7cb07481d28192d14a6d9e9 (patch) | |
| tree | 49fef6a16474365d2140e3974ba4c0c6761429b6 | |
| parent | 4a38e7eb2b71cc721ad16fdb814c1aeb3c2c5e79 (diff) | |
| parent | b204c0974c04b30092e026428b9560eb64fc9a33 (diff) | |
Merge pull request #83 from xoviat/cleanup
cleanup misc. warnings
| -rw-r--r-- | embassy-stm32f4-examples/src/bin/can.rs | 4 | ||||
| -rw-r--r-- | embassy-stm32f4/src/can.rs | 6 | ||||
| -rw-r--r-- | embassy-stm32f4/src/exti.rs | 1 | ||||
| -rw-r--r-- | embassy-stm32f4/src/qei.rs | 8 | ||||
| -rw-r--r-- | embassy-stm32f4/src/serial.rs | 3 |
5 files changed, 7 insertions, 15 deletions
diff --git a/embassy-stm32f4-examples/src/bin/can.rs b/embassy-stm32f4-examples/src/bin/can.rs index 27aef02b1..701a277ec 100644 --- a/embassy-stm32f4-examples/src/bin/can.rs +++ b/embassy-stm32f4-examples/src/bin/can.rs | |||
| @@ -14,13 +14,11 @@ use cortex_m_rt::entry; | |||
| 14 | use embassy::executor::{task, Executor}; | 14 | use embassy::executor::{task, Executor}; |
| 15 | use embassy::traits::gpio::*; | 15 | use embassy::traits::gpio::*; |
| 16 | use embassy::util::Forever; | 16 | use embassy::util::Forever; |
| 17 | use embassy_stm32f4::{can, exti, interrupt}; | 17 | use embassy_stm32f4::{can, interrupt}; |
| 18 | use futures::pin_mut; | 18 | use futures::pin_mut; |
| 19 | use stm32f4xx_hal::prelude::*; | 19 | use stm32f4xx_hal::prelude::*; |
| 20 | use stm32f4xx_hal::{can::Can, stm32}; | 20 | use stm32f4xx_hal::{can::Can, stm32}; |
| 21 | 21 | ||
| 22 | static EXTI: Forever<exti::ExtiManager> = Forever::new(); | ||
| 23 | |||
| 24 | #[task] | 22 | #[task] |
| 25 | async fn run(dp: stm32::Peripherals, _cp: cortex_m::Peripherals) { | 23 | async fn run(dp: stm32::Peripherals, _cp: cortex_m::Peripherals) { |
| 26 | let gpioa = dp.GPIOA.split(); | 24 | let gpioa = dp.GPIOA.split(); |
diff --git a/embassy-stm32f4/src/can.rs b/embassy-stm32f4/src/can.rs index a8a3ca164..7c5311a9d 100644 --- a/embassy-stm32f4/src/can.rs +++ b/embassy-stm32f4/src/can.rs | |||
| @@ -9,11 +9,9 @@ use bxcan::Interrupts; | |||
| 9 | use core::future::Future; | 9 | use core::future::Future; |
| 10 | use embassy::interrupt::Interrupt; | 10 | use embassy::interrupt::Interrupt; |
| 11 | use embassy::util::InterruptFuture; | 11 | use embassy::util::InterruptFuture; |
| 12 | use embassy::util::Signal; | ||
| 13 | use nb; | 12 | use nb; |
| 14 | use nb::block; | 13 | use nb::block; |
| 15 | 14 | ||
| 16 | use crate::hal::prelude::*; | ||
| 17 | use crate::interrupt; | 15 | use crate::interrupt; |
| 18 | 16 | ||
| 19 | /// Interface to the Serial peripheral | 17 | /// Interface to the Serial peripheral |
| @@ -53,9 +51,9 @@ impl<T: Instance> Can<T> { | |||
| 53 | /// Receive can frame. | 51 | /// Receive can frame. |
| 54 | /// | 52 | /// |
| 55 | /// This method async-blocks until the frame is received. | 53 | /// This method async-blocks until the frame is received. |
| 56 | pub fn receive<'a>(&'a mut self) -> impl Future<Output = (bxcan::Frame)> + 'a { | 54 | pub fn receive<'a>(&'a mut self) -> impl Future<Output = bxcan::Frame> + 'a { |
| 57 | async move { | 55 | async move { |
| 58 | let mut frame: Option<bxcan::Frame> = None; | 56 | let mut frame: Option<bxcan::Frame>; |
| 59 | 57 | ||
| 60 | loop { | 58 | loop { |
| 61 | let fut = InterruptFuture::new(&mut self.rx_int); | 59 | let fut = InterruptFuture::new(&mut self.rx_int); |
diff --git a/embassy-stm32f4/src/exti.rs b/embassy-stm32f4/src/exti.rs index 90f3ade39..660e3be33 100644 --- a/embassy-stm32f4/src/exti.rs +++ b/embassy-stm32f4/src/exti.rs | |||
| @@ -2,7 +2,6 @@ use core::future::Future; | |||
| 2 | use core::mem; | 2 | use core::mem; |
| 3 | use core::pin::Pin; | 3 | use core::pin::Pin; |
| 4 | 4 | ||
| 5 | use embassy::interrupt::Interrupt; | ||
| 6 | use embassy::traits::gpio::{WaitForFallingEdge, WaitForRisingEdge}; | 5 | use embassy::traits::gpio::{WaitForFallingEdge, WaitForRisingEdge}; |
| 7 | use embassy::util::InterruptFuture; | 6 | use embassy::util::InterruptFuture; |
| 8 | 7 | ||
diff --git a/embassy-stm32f4/src/qei.rs b/embassy-stm32f4/src/qei.rs index 1d4689e19..391c8f7c2 100644 --- a/embassy-stm32f4/src/qei.rs +++ b/embassy-stm32f4/src/qei.rs | |||
| @@ -1,21 +1,19 @@ | |||
| 1 | use crate::interrupt; | 1 | use crate::interrupt; |
| 2 | use core::future::Future; | 2 | use core::future::Future; |
| 3 | use core::pin::Pin; | 3 | use core::pin::Pin; |
| 4 | use embassy::interrupt::Interrupt; | ||
| 5 | use embassy::traits::qei::WaitForRotate; | 4 | use embassy::traits::qei::WaitForRotate; |
| 6 | use embedded_hal::Direction; | 5 | use embedded_hal::Direction; |
| 7 | use embedded_hal::Qei as THQei; | ||
| 8 | use stm32f4xx_hal::pac::TIM2; | 6 | use stm32f4xx_hal::pac::TIM2; |
| 9 | use stm32f4xx_hal::qei::{Pins, Qei as HalQei}; | 7 | use stm32f4xx_hal::{qei, qei::Pins}; |
| 10 | 8 | ||
| 11 | pub struct Qei<T: Instance, PINS> { | 9 | pub struct Qei<T: Instance, PINS> { |
| 12 | qei: HalQei<T, PINS>, | 10 | qei: qei::Qei<T, PINS>, |
| 13 | int: T::Interrupt, | 11 | int: T::Interrupt, |
| 14 | } | 12 | } |
| 15 | 13 | ||
| 16 | impl<PINS: Pins<TIM2>> Qei<TIM2, PINS> { | 14 | impl<PINS: Pins<TIM2>> Qei<TIM2, PINS> { |
| 17 | pub fn tim2(tim: TIM2, pins: PINS, interrupt: interrupt::TIM2) -> Self { | 15 | pub fn tim2(tim: TIM2, pins: PINS, interrupt: interrupt::TIM2) -> Self { |
| 18 | let qei = HalQei::tim2(tim, pins); | 16 | let qei = qei::Qei::tim2(tim, pins); |
| 19 | 17 | ||
| 20 | let tim = unsafe { | 18 | let tim = unsafe { |
| 21 | &mut *(stm32f4xx_hal::stm32::TIM2::ptr() | 19 | &mut *(stm32f4xx_hal::stm32::TIM2::ptr() |
diff --git a/embassy-stm32f4/src/serial.rs b/embassy-stm32f4/src/serial.rs index 2ca82ac50..97b365a11 100644 --- a/embassy-stm32f4/src/serial.rs +++ b/embassy-stm32f4/src/serial.rs | |||
| @@ -6,10 +6,9 @@ | |||
| 6 | 6 | ||
| 7 | use core::future::Future; | 7 | use core::future::Future; |
| 8 | use core::marker::PhantomData; | 8 | use core::marker::PhantomData; |
| 9 | use core::ptr; | ||
| 10 | use core::sync::atomic::{self, Ordering}; | 9 | use core::sync::atomic::{self, Ordering}; |
| 11 | 10 | ||
| 12 | use embassy::interrupt::{Interrupt, InterruptExt}; | 11 | use embassy::interrupt::Interrupt; |
| 13 | use embassy::traits::uart::{Error, Uart}; | 12 | use embassy::traits::uart::{Error, Uart}; |
| 14 | use embassy::util::InterruptFuture; | 13 | use embassy::util::InterruptFuture; |
| 15 | 14 | ||
