aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDario Nieuwenhuis <[email protected]>2023-10-03 20:28:44 +0000
committerGitHub <[email protected]>2023-10-03 20:28:44 +0000
commitb012a67df7b5c95cce0e09eb66e3952bbc021bcc (patch)
tree174c2e09b60a07c6998647d0e2c45d6b7e93ad97
parentf197aa94193d3ed09811482c88d69da90624090a (diff)
parent40f30fa4cd1d1273c1fe6c6520042d21e73a348e (diff)
Merge pull request #2007 from embassy-rs/unverbose
Remove a few ultra-verbose logs.
-rw-r--r--embassy-rp/src/gpio.rs10
-rw-r--r--embassy-rp/src/uart/buffered.rs2
-rw-r--r--tests/stm32/src/bin/dac.rs2
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"]
223struct InputFuture<'a, T: Pin> { 223struct InputFuture<'a, T: Pin> {
224 pin: PeripheralRef<'a, T>, 224 pin: PeripheralRef<'a, T>,
225 level: InterruptTrigger,
226} 225}
227 226
228impl<'d, T: Pin> InputFuture<'d, T> { 227impl<'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);