From daf9bcf77f8f9c46d01160f7cd8d685d2640cbe4 Mon Sep 17 00:00:00 2001 From: Ed Beroset Date: Tue, 3 Mar 2015 15:01:38 -0500 Subject: [PATCH] startup: Fixed measure_clk to return kHz instead of MHz. The original code had iterated 10,000,000 times and taken the elapsed time divided by 10,000, to yield kHz which is mathematically correct only if we were measuring time in seconds, but we are measuring time in milliseconds, so the correct divisor is actually 10,000,000. Previous code would report 0.500 for actual measured speed of 500 kHz. Change-Id: Iba4c4961fe3973e7ccfa6dfa11d606a966ceb50c Signed-off-by: Ed Beroset Reviewed-on: http://openocd.zylin.com/2573 Tested-by: jenkins Reviewed-by: Jens Bauer Reviewed-by: Paul Fertser --- src/jtag/startup.tcl | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/jtag/startup.tcl b/src/jtag/startup.tcl index 6f9c000d..f88ef1fa 100644 --- a/src/jtag/startup.tcl +++ b/src/jtag/startup.tcl @@ -81,8 +81,9 @@ proc srst_asserted {} { # measure actual JTAG clock proc measure_clk {} { set start_time [ms]; - runtest 10000000; - echo "Running at more than [expr 10000.0 / ([ms]-$start_time)] kHz"; + set iterations 10000000; + runtest $iterations; + echo "Running at more than [expr $iterations.0 / ([ms]-$start_time)] kHz"; } add_help_text measure_clk "Runs a test to measure the JTAG clk. Useful with RCLK / RTCK." -- 2.39.5