From f944cce39d55a10f31a03828b0bee144d3bbbec2 Mon Sep 17 00:00:00 2001
From: cscherr <Christoph.Scherr@newtec.de>
Date: Wed, 6 Aug 2025 09:43:04 +0200
Subject: [PATCH] refactor: remove the software timing measures since they are
 not reliable

---
 src/bench.rs | 28 +++++-----------------------
 1 file changed, 5 insertions(+), 23 deletions(-)

diff --git a/src/bench.rs b/src/bench.rs
index bcb226a..f00abf3 100644
--- a/src/bench.rs
+++ b/src/bench.rs
@@ -6,15 +6,6 @@ use hal::{
     prelude::*,
 };
 
-macro_rules! display_results {
-    ($name:expr, $rust_cycles:expr, $c_cycles:expr) => {
-        info!(
-            "Benchmark {}:\nRust\t: {}\nC\t: {}",
-            $name, $rust_cycles, $c_cycles
-        )
-    };
-}
-
 #[inline]
 pub fn bench(pin: &mut Pin<Output<PushPull>>, delay: &mut Delay) {
     bench_delay(pin, delay);
@@ -22,40 +13,31 @@ pub fn bench(pin: &mut Pin<Output<PushPull>>, delay: &mut Delay) {
 }
 
 fn bench_delay(pin: &mut Pin<Output<PushPull>>, delay: &mut Delay) {
-    let short_sleep = time_exec(pin, || delay.delay_ms(20u32));
-    let long_sleep = time_exec(pin, || delay.delay_ms(50u32));
-    info!("Bench for short sleep: {}", short_sleep);
-    info!("Bench for long sleep: {}", long_sleep);
+    time_exec(pin, || delay.delay_ms(20u32));
+    time_exec(pin, || delay.delay_ms(50u32));
 }
 
-fn time_exec<F>(pin: &mut Pin<Output<PushPull>>, mut f: F) -> u32
+fn time_exec<F>(pin: &mut Pin<Output<PushPull>>, mut f: F)
 where
     F: FnMut(),
 {
     info!("Running the requested function 50 times and returning the average");
     const RUNS: u32 = 50;
-    let start = cortex_m::peripheral::SYST::get_current();
     pin.set_high().expect("could not set pun to high");
     for _ in 0..RUNS {
         f();
     }
     pin.set_low().expect("could not set pun to low");
-    let end = cortex_m::peripheral::SYST::get_current();
-    // the timer counts down
-    info!("Start: {}\nEnd: {}", start, end);
-    let total = start.wrapping_sub(end);
-    total / RUNS
 }
 
 fn bench_crc(pin: &mut Pin<Output<PushPull>>, delay: &mut Delay) {
     const DATA: &[u8] = b"hello world AAAAAAAAAAAAAAAAAAAAAAA";
     delay.delay_ms(50_u32);
-    let native = time_exec(pin, || {
+    time_exec(pin, || {
         Crc32Rust::checksum(DATA);
     });
     delay.delay_ms(50_u32);
-    let c_code = time_exec(pin, || {
+    time_exec(pin, || {
         Crc32C::checksum(DATA);
     });
-    display_results!("CRC", native, c_code);
 }