diff options
| author | Dario Nieuwenhuis <[email protected]> | 2023-10-03 20:28:44 +0000 |
|---|---|---|
| committer | GitHub <[email protected]> | 2023-10-03 20:28:44 +0000 |
| commit | b012a67df7b5c95cce0e09eb66e3952bbc021bcc (patch) | |
| tree | 174c2e09b60a07c6998647d0e2c45d6b7e93ad97 | |
| parent | f197aa94193d3ed09811482c88d69da90624090a (diff) | |
| parent | 40f30fa4cd1d1273c1fe6c6520042d21e73a348e (diff) | |
Merge pull request #2007 from embassy-rs/unverbose
Remove a few ultra-verbose logs.
| -rw-r--r-- | embassy-rp/src/gpio.rs | 10 | ||||
| -rw-r--r-- | embassy-rp/src/uart/buffered.rs | 2 | ||||
| -rw-r--r-- | tests/stm32/src/bin/dac.rs | 2 |
3 files changed, 2 insertions, 12 deletions
diff --git a/embassy-rp/src/gpio.rs b/embassy-rp/src/gpio.rs index 33bc58a0f..ee7e03e95 100644 --- a/embassy-rp/src/gpio.rs +++ b/embassy-rp/src/gpio.rs | |||
| @@ -222,7 +222,6 @@ fn IO_IRQ_QSPI() { | |||
| 222 | #[must_use = "futures do nothing unless you `.await` or poll them"] | 222 | #[must_use = "futures do nothing unless you `.await` or poll them"] |
| 223 | struct InputFuture<'a, T: Pin> { | 223 | struct InputFuture<'a, T: Pin> { |
| 224 | pin: PeripheralRef<'a, T>, | 224 | pin: PeripheralRef<'a, T>, |
| 225 | level: InterruptTrigger, | ||
| 226 | } | 225 | } |
| 227 | 226 | ||
| 228 | impl<'d, T: Pin> InputFuture<'d, T> { | 227 | impl<'d, T: Pin> InputFuture<'d, T> { |
| @@ -249,7 +248,6 @@ impl<'d, T: Pin> InputFuture<'d, T> { | |||
| 249 | .inte((pin.pin() / 8) as usize) | 248 | .inte((pin.pin() / 8) as usize) |
| 250 | .write_set(|w| match level { | 249 | .write_set(|w| match level { |
| 251 | InterruptTrigger::LevelHigh => { | 250 | InterruptTrigger::LevelHigh => { |
| 252 | trace!("InputFuture::new enable LevelHigh for pin {}", pin.pin()); | ||
| 253 | w.set_level_high(pin_group, true); | 251 | w.set_level_high(pin_group, true); |
| 254 | } | 252 | } |
| 255 | InterruptTrigger::LevelLow => { | 253 | InterruptTrigger::LevelLow => { |
| @@ -267,7 +265,7 @@ impl<'d, T: Pin> InputFuture<'d, T> { | |||
| 267 | } | 265 | } |
| 268 | }); | 266 | }); |
| 269 | 267 | ||
| 270 | Self { pin, level } | 268 | Self { pin } |
| 271 | } | 269 | } |
| 272 | } | 270 | } |
| 273 | 271 | ||
| @@ -303,14 +301,8 @@ impl<'d, T: Pin> Future for InputFuture<'d, T> { | |||
| 303 | && !inte.level_high(pin_group) | 301 | && !inte.level_high(pin_group) |
| 304 | && !inte.level_low(pin_group) | 302 | && !inte.level_low(pin_group) |
| 305 | { | 303 | { |
| 306 | trace!( | ||
| 307 | "{:?} for pin {} was cleared, return Poll::Ready", | ||
| 308 | self.level, | ||
| 309 | self.pin.pin() | ||
| 310 | ); | ||
| 311 | return Poll::Ready(()); | 304 | return Poll::Ready(()); |
| 312 | } | 305 | } |
| 313 | trace!("InputFuture::poll return Poll::Pending"); | ||
| 314 | Poll::Pending | 306 | Poll::Pending |
| 315 | } | 307 | } |
| 316 | } | 308 | } |
diff --git a/embassy-rp/src/uart/buffered.rs b/embassy-rp/src/uart/buffered.rs index e57b72599..645d703d8 100644 --- a/embassy-rp/src/uart/buffered.rs +++ b/embassy-rp/src/uart/buffered.rs | |||
| @@ -490,8 +490,6 @@ impl<T: Instance> interrupt::typelevel::Handler<T::Interrupt> for BufferedInterr | |||
| 490 | w.set_oeic(ris.oeris()); | 490 | w.set_oeic(ris.oeris()); |
| 491 | }); | 491 | }); |
| 492 | 492 | ||
| 493 | trace!("on_interrupt ris={:#X}", ris.0); | ||
| 494 | |||
| 495 | // Errors | 493 | // Errors |
| 496 | if ris.feris() { | 494 | if ris.feris() { |
| 497 | warn!("Framing error"); | 495 | warn!("Framing error"); |
diff --git a/tests/stm32/src/bin/dac.rs b/tests/stm32/src/bin/dac.rs index 67a7d5b59..fb7a84b1b 100644 --- a/tests/stm32/src/bin/dac.rs +++ b/tests/stm32/src/bin/dac.rs | |||
| @@ -54,7 +54,7 @@ async fn main(_spawner: Spawner) { | |||
| 54 | // Calibrate and normalize the measurement to get close to the dac_output_val | 54 | // Calibrate and normalize the measurement to get close to the dac_output_val |
| 55 | let measured_normalized = ((measured as i32 - offset as i32) / normalization_factor) as i16; | 55 | let measured_normalized = ((measured as i32 - offset as i32) / normalization_factor) as i16; |
| 56 | 56 | ||
| 57 | info!("value / measured: {} / {}", dac_output_val, measured_normalized); | 57 | //info!("value / measured: {} / {}", dac_output_val, measured_normalized); |
| 58 | 58 | ||
| 59 | // The deviations are quite enormous but that does not matter since this is only a quick test | 59 | // The deviations are quite enormous but that does not matter since this is only a quick test |
| 60 | assert!((dac_output_val as i16 - measured_normalized).abs() < 15); | 60 | assert!((dac_output_val as i16 - measured_normalized).abs() < 15); |
